3

我的应用程序试图用一些基本的审计/日志功能来包装 django.contrib.auth.views 登录和注销视图。我遵循django-axes项目中描述的处方，并在服务器上运行和其他一些测试，它按预期工作，透明而没有问题。

代码如下：

from django.contrib.auth import views as auth_views
from myapp.watchers import watch_login

class WatcherMiddleware(object):
    def __init__(self):
        auth_views.login = watch_login(auth_views.login)

和

def watch_login(func):
    def decorated_login(request, *args, **kwargs):
        #do some stuff
        response = func(request, *args, **kwargs)
        #more stuff
        return response
    return decorated_login

网址：

#Edit: Added project's urls - just using vanilla django's auth login
(r'^accounts/login/$', 'django.contrib.auth.views.login',{"template_name":settings.LOGIN_TEMPLATE }),

然而，在我们的构建工作流程中，我们在 django.contrib.auth.tests.views 中遇到了一些问题。

具体来说，这些是在 django.contrib.auth 中失败的测试：

ERROR: test_current_site_in_context_after_login (django.contrib.auth.tests.views.LoginTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python26\lib\site-packages\django\contrib\auth\tests\views.py", line 192, in test_current_site_in_context_after_login
    response = self.client.get(reverse('django.contrib.auth.views.login'))
  File "C:\Python26\lib\site-packages\django\core\urlresolvers.py", line 351, in reverse
    *args, **kwargs)))
  File "C:\Python26\lib\site-packages\django\core\urlresolvers.py", line 297, in reverse
    "arguments '%s' not found." % (lookup_view_s, args, kwargs))
NoReverseMatch: Reverse for 'myapp.watchers.decorated_login' with arguments '()' and keyword arguments '{}' not found.

======================================================================
ERROR: test_security_check (django.contrib.auth.tests.views.LoginTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python26\lib\site-packages\django\contrib\auth\tests\views.py", line 204, in test_security_check
    login_url = reverse('django.contrib.auth.views.login')
  File "C:\Python26\lib\site-packages\django\core\urlresolvers.py", line 351, in reverse
    *args, **kwargs)))
  File "C:\Python26\lib\site-packages\django\core\urlresolvers.py", line 297, in reverse
    "arguments '%s' not found." % (lookup_view_s, args, kwargs))
NoReverseMatch: Reverse for 'myapp.watchers.decorated_login' with arguments '()' and keyword arguments '{}' not found.

只有这两个测试因包含包装的登录猴子补丁而失败。
似乎 django auth 测试中的 reverse() 调用的行为与未修补函数的行为方式不同。

我们之所以走这条路线来包装日志记录与使用 django 1.3 的新身份验证信号的原因是因为那里提供的日志记录方法只会告诉您是否发生了错误尝试 - 它不会让您访问请求对象记录有关该不当请求的其他信息。在这种情况下修补身份验证表单不会有帮助，因此我们需要包装登录功能。

我的登录功能包装有问题吗？或者，尽管整体功能没有变化，但由于其他副作用导致测试失败，这是否符合预期？

编辑：我正在运行 python 2.6.4，django 1.2.5

4

2 回答 2

5

你不能简单地将它包装在另一个视图中吗？

urls：

url(
    r'^accounts/login/$',
    'accounts.views.login',
    {"template_name":settings.LOGIN_TEMPLATE }
),

accounts.views：

from django.contrib.auth import views as auth_views   

def login(request, *args, **kwars):
    # do some stuff    
    response = auth_views.login(request, *args, **kwars)
    # more stuff
    return response

像这样，django.contrib.auth.tests将测试他们为之编写的视图，您可以为您需要的“更多东西”编写自己的测试。

于 2011-05-07T17:43:42.160 回答
3

我怀疑这与影响 django-registration的潜在问题相同，因为测试运行程序仅导入当时正在测试的应用程序的 URL——即，contrib.auth而不是关于与此问题类似的事情myapp的各种 票证，但快速扫描其中暗示解决方案是将事物解耦，我猜这对于您的解决方案是不可行的。

另一种解决方法是使用 Fabric 文件或 Makefile 来触发一个测试子集，避免因为你的 monkeypatch 而失败的两个测试，然后添加两个替代的猿友好测试来替换它们。

于 2011-05-06T09:19:23.677 回答