【发布时间】:2012-07-23 19:03:12
【问题描述】:
我想做什么
我正在尝试编写单元测试来检查我的正则表达式是否正确构造:
# mediamanager/models.py
import re
from django.conf import settings
filetypes_re = {}
for key, exts in settings.MM_FILETYPES.items():
filetypes_re[key] = re.compile(r'({})'.format('|'.join(exts)))
注意:实际上,我完全不知道为什么要编写这个单元测试,因为这段代码非常简单……但这不是重点。
如您所见,最终的正则表达式取决于用户可以设置的变量 settings.MM_FILETYPES。我需要测试特定的输入,对于这种情况,Django 提供了装饰器@override_settings,它可以临时覆盖设置值:
# mediamanager/tests.py
import unittest
from django.test.utils import override_settings
from mediamanager.models import (filetypes_re, …) # import everything we want to test
class ModelsTestCase(unittest.TestCase):
@override_settings(MM_FILETYPES={'image': ['jpg', 'png', 'gif'],
'document': ['pdf', 'txt'],
'audio': ['mp3', 'wav']})
def test_filetype_re(self):
filetypes_re_exp = {'image': '(jpg|png|gif)',
'document': '(pdf|txt)',
'audio': '(mp3|wav)'}
for key, value in filetypes_re_exp.items():
self.assertEqual(value, filetypes_re[key].pattern)
很遗憾,这个测试没有通过。在覆盖设置之前加载模块 mediamanager.models,因此使用旧设置编译 filetypes_re。我需要(以某种方式)重新加载它以使新设置生效。
问题
我以这种方式更改了单元测试:
@override_settings(MM_FILETYPES={'image': ['jpg', 'png', 'gif'],
'document': ['pdf', 'txt'],
'audio': ['mp3', 'wav']})
def test_filetype_re(self):
import mediamanager.models # obtaining module object from sys.modules have the same result
reload(mediamanager.models)
filetypes_re = mediamanager.models.filetypes_re
filetypes_re_exp = {'image': '(jpg|png|gif)',
'document': '(pdf|txt)',
'audio': '(mp3|wav)'}
for key, value in filetypes_re_exp.items():
self.assertEqual(value, filetypes_re[key].pattern)
测试通过了。但可能是因为我从 mediamanager.models 模块导入了其他对象,这个测试用例中的其他测试失败了。不是全部,只有两个(这很奇怪)。 编辑:一点也不奇怪。只有在 test_filetyes_re 和 reload() 调用之后运行的测试才会失败。
问题
如何“重新加载”模块 mediamanager.models:
- filetypes_re 使用新设置进行评估?
- 从同一模型导入的所有其他对象不受影响?
我是否应该仅仅因为重新加载后变得不稳定而重写一段代码(我是指来自 mediamanager.models 的其他那些在重新加载后未通过测试的对象)?我读过一些关于重新加载模块的文章,通常这不是一个好主意。
有没有更好的方法来定义模块级对象,比如这个正则表达式,以使测试更容易?
【问题讨论】:
-
你能解释一下为什么你需要重新加载模块吗?为什么要更改测试的第一个版本?
-
那是因为用户自己定义了 MM_FILETYPES。因此我不知道我应该测试什么。我临时覆盖了这些设置(使用装饰器),但由于 mediamanager.models 已经加载,因此使用旧设置评估 filetypes_re。我需要重新加载他以使新设置生效。
-
啊,好的-您需要清除一些东西-您问题中的第一块代码-来自您的
settings.py?或者别的地方?其次,您的所有单元测试正在检查mediamanager.models.filetypes_re是否符合您的预期。为什么? mediamanager 是您的应用吗? -
不,settings.py 无关紧要。每个设置都应该通过测试,这就是我使用 override_settings 装饰器的原因——创建已知输入。在这个unittest中,只有这个方法(test_filetypes_re)检查filetypes_re。其他方法检查来自 mediamanager.models 的其他对象,这就是这些测试失败的原因。模块重新加载后,从该模块导入的所有对象仍指向旧对象。
-
什么旧的?
mediamanager.models是否发生了某种变化?
标签: python django unit-testing testing