Stop new Pyflakes complaints.

10 files changed, 39 insertions, 64 deletions

diff --git a/repoze/bfg/tests/test_configuration.py b/repoze/bfg/tests/test_configuration.py
index 416df3086..43d2cb153 100644
--- a/repoze/bfg/tests/test_configuration.py
+++ b/repoze/bfg/tests/test_configuration.py
@@ -125,8 +125,8 @@ class ConfiguratorTests(unittest.TestCase):
     def test_ctor_authorization_policy_only(self):
         from zope.configuration.exceptions import ConfigurationError
         policy = object()
-        config = self.assertRaises(ConfigurationError,
-                                   self._makeOne, authorization_policy=policy)
+        self.assertRaises(ConfigurationError,
+                          self._makeOne, authorization_policy=policy)
 
     def test_ctor_no_root_factory(self):
         from repoze.bfg.interfaces import IRootFactory
@@ -327,7 +327,7 @@ class ConfiguratorTests(unittest.TestCase):
     def test_load_zcml_lock_and_unlock(self):
         config = self._makeOne()
         dummylock = DummyLock()
-        registry = config.load_zcml(
+        config.load_zcml(
             'repoze.bfg.tests.fixtureapp:configure.zcml',
             lock=dummylock)
         self.assertEqual(dummylock.acquired, True)
@@ -818,7 +818,6 @@ class ConfiguratorTests(unittest.TestCase):
         config = self._makeOne()
         config.add_view(view=view, containment=IDummy)
         wrapper = self._getViewCallable(config)
-        request = self._makeRequest(config)
         context = DummyContext()
         directlyProvides(context, IDummy)
         self.assertEqual(wrapper(context, None), 'OK')
@@ -828,7 +827,6 @@ class ConfiguratorTests(unittest.TestCase):
         config = self._makeOne()
         config.add_view(view=view, containment=IDummy)
         wrapper = self._getViewCallable(config)
-        request = self._makeRequest(config)
         context = DummyContext()
         self._assertNotFound(wrapper, context, None)
 
@@ -875,9 +873,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_xhr(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', xhr=True)
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -889,9 +885,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_request_method(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', request_method='GET')
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -903,9 +897,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_path_info(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', path_info='/foo')
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -917,9 +909,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_request_param(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', request_param='abc')
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -931,9 +921,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_header(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', header='Host')
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -945,9 +933,7 @@ class ConfiguratorTests(unittest.TestCase):
 
     def test_add_route_with_accept(self):
         config = self._makeOne()
-        view = lambda *arg: 'OK'
         config.add_route('name', 'path', accept='text/xml')
-        request_type = self._getRouteRequestIface(config, 'name')
         route = self._assertRoute(config, 'name', 'path', 1)
         predicate = route.predicates[0]
         request = self._makeRequest(config)
@@ -994,7 +980,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_request_method='GET')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         request = self._makeRequest(config)
         request.method = 'GET'
         self.assertEqual(wrapper(None, request), 'OK')
@@ -1008,7 +994,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_header='Host')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         request = self._makeRequest(config)
         request.headers = {'Host':'abc'}
         self.assertEqual(wrapper(None, request), 'OK')
@@ -1022,7 +1008,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_xhr=True)
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         request = self._makeRequest(config)
         request.is_xhr = True
         self.assertEqual(wrapper(None, request), 'OK')
@@ -1036,7 +1022,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_path_info='/foo')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         request = self._makeRequest(config)
         request.path_info = '/foo'
         self.assertEqual(wrapper(None, request), 'OK')
@@ -1050,7 +1036,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_accept='text/xml')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         request = self._makeRequest(config)
         request.accept = ['text/xml']
         self.assertEqual(wrapper(None, request), 'OK')
@@ -1065,7 +1051,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_containment=IDummy)
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         context = DummyContext()
         directlyProvides(context, IDummy)
         self.assertEqual(wrapper(context, None), 'OK')
@@ -1079,7 +1065,7 @@ class ConfiguratorTests(unittest.TestCase):
                          view_renderer='fixtures/minimal.txt')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         self.assertEqual(wrapper(None, None).body, 'Hello!')
 
     def test_add_route_with_view_renderer_alias(self):
@@ -1090,7 +1076,7 @@ class ConfiguratorTests(unittest.TestCase):
                          renderer='fixtures/minimal.txt')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         self.assertEqual(wrapper(None, None).body, 'Hello!')
 
     def test_add_route_with_view_permission(self):
@@ -1104,7 +1090,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, view_permission='edit')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         self.failUnless(hasattr(wrapper, '__call_permissive__'))
 
     def test_add_route_with_view_permission_alias(self):
@@ -1118,7 +1104,7 @@ class ConfiguratorTests(unittest.TestCase):
         config.add_route('name', 'path', view=view, permission='edit')
         request_type = self._getRouteRequestIface(config, 'name')
         wrapper = self._getViewCallable(config, None, request_type)
-        route = self._assertRoute(config, 'name', 'path')
+        self._assertRoute(config, 'name', 'path')
         self.failUnless(hasattr(wrapper, '__call_permissive__'))
 
     def test__override_not_yet_registered(self):
@@ -1485,7 +1471,7 @@ class ConfiguratorTests(unittest.TestCase):
         config = self._makeOne()
         self._registerSettings(config,
                                debug_authorization=True, reload_templates=True)
-        logger = self._registerLogger(config)
+        self._registerLogger(config)
         self._registerSecurityPolicy(config, False)
         result = config._derive_view(view, permission='view')
         self.assertEqual(view.__module__, result.__module__)
@@ -1582,7 +1568,7 @@ class ConfiguratorTests(unittest.TestCase):
         request.registry = config.registry
         wrapped = config._derive_view(
             inner_view, viewname='inner', wrapper_viewname='owrap')
-        result = self.assertRaises(ValueError, wrapped, None, request)
+        self.assertRaises(ValueError, wrapped, None, request)
 
     def test_override_resource_samename(self):
         from zope.configuration.exceptions import ConfigurationError
@@ -2277,8 +2263,6 @@ class TestMultiView(unittest.TestCase):
         def view(context, request):
             """ """
         mv.views = [(100, view)]
-        context = DummyContext()
-        request = DummyRequest()
         self.assertEqual(mv.__permitted__(None, None), True)
         
     def test_permitted(self):
diff --git a/repoze/bfg/tests/test_paster.py b/repoze/bfg/tests/test_paster.py
index 6091aec44..cabde215e 100644
--- a/repoze/bfg/tests/test_paster.py
+++ b/repoze/bfg/tests/test_paster.py
@@ -34,7 +34,6 @@ class TestBFGShellCommand(unittest.TestCase):
 
     def test_command_ipython_enabled(self):
         command = self._makeOne()
-        interact = DummyInteractor()
         app = DummyApp()
         loadapp = DummyLoadApp(app)
         command.loadapp = (loadapp,)
diff --git a/repoze/bfg/tests/test_path.py b/repoze/bfg/tests/test_path.py
index 8522eb387..043392311 100644
--- a/repoze/bfg/tests/test_path.py
+++ b/repoze/bfg/tests/test_path.py
@@ -24,7 +24,6 @@ class TestCallerPath(unittest.TestCase):
         import os
         from repoze.bfg.tests import test_path
         test_path.__bfg_abspath__ = '/foo/bar'
-        here = os.path.abspath(os.path.dirname(__file__))
         result = self._callFUT('a/b/c')
         self.assertEqual(result, os.path.join('/foo/bar', 'a/b/c'))
 
@@ -103,7 +102,7 @@ class TestPackagePath(unittest.TestCase):
     def test_memoization_success(self):
         from repoze.bfg.tests import test_path
         module = DummyPackageOrModule(test_path)
-        result = self._callFUT(module)
+        self._callFUT(module)
         self.assertEqual(module.__bfg_abspath__, module.package_path)
         
     def test_memoization_fail(self):
diff --git a/repoze/bfg/tests/test_resource.py b/repoze/bfg/tests/test_resource.py
index 2e55d1044..39f883ba1 100644
--- a/repoze/bfg/tests/test_resource.py
+++ b/repoze/bfg/tests/test_resource.py
@@ -204,7 +204,6 @@ class TestPackageOverrides(unittest.TestCase):
         self.assertRaises(TypeError, self._makeOne, package)
 
     def test_ctor_package_already_has_loader_of_same_type(self):
-        dummy_pkg_resources = DummyPkgResources()
         package = DummyPackage('package')
         package.__loader__ = self._makeOne(package)
         po = self._makeOne(package)
diff --git a/repoze/bfg/tests/test_router.py b/repoze/bfg/tests/test_router.py
index 76c61f1e2..45e66b8fe 100644
--- a/repoze/bfg/tests/test_router.py
+++ b/repoze/bfg/tests/test_router.py
@@ -109,7 +109,6 @@ class TestRouter(unittest.TestCase):
         return environ
 
     def test_root_policy(self):
-        environ = self._makeEnviron()
         context = DummyContext()
         self._registerTraverserFactory(context)
         rootfactory = self._registerRootFactory('abc')
@@ -125,7 +124,6 @@ class TestRouter(unittest.TestCase):
         self.assertEqual(router.forbidden_view, app)
 
     def test_iforbiddenview_nooverride(self):
-        context = DummyContext()
         router = self._makeOne()
         from repoze.bfg.view import default_forbidden_view
         self.assertEqual(router.forbidden_view, default_forbidden_view)
@@ -139,14 +137,12 @@ class TestRouter(unittest.TestCase):
         self.assertEqual(router.notfound_view, app)
 
     def test_inotfoundview_nooverride(self):
-        context = DummyContext()
         router = self._makeOne()
         from repoze.bfg.view import default_notfound_view
         self.assertEqual(router.notfound_view, default_notfound_view)
 
     def test_call_traverser_default(self):
         environ = self._makeEnviron()
-        context = DummyContext()
         logger = self._registerLogger()
         router = self._makeOne()
         start_response = DummyStartResponse()
@@ -249,7 +245,7 @@ class TestRouter(unittest.TestCase):
         view = DummyView(response)
         environ = self._makeEnviron()
         self._registerView(view, '', None, None)
-        rootfactory = self._registerRootFactory(context)
+        self._registerRootFactory(context)
         router = self._makeOne()
         start_response = DummyStartResponse()
         result = router(environ, start_response)
@@ -267,7 +263,7 @@ class TestRouter(unittest.TestCase):
         self._registerTraverserFactory(context, view_name='foo',
                                        subpath=['bar'],
                                        traversed=['context'])
-        rootfactory = self._registerRootFactory(context)
+        self._registerRootFactory(context)
         response = DummyResponse()
         response.app_iter = ['Hello world']
         view = DummyView(response)
@@ -294,7 +290,7 @@ class TestRouter(unittest.TestCase):
         context = DummyContext()
         directlyProvides(context, IContext)
         self._registerTraverserFactory(context)
-        rootfactory = self._registerRootFactory(context)
+        self._registerRootFactory(context)
         response = DummyResponse()
         response.app_iter = ['Hello world']
         view = DummyView(response)
@@ -416,6 +412,7 @@ class TestRouter(unittest.TestCase):
         self.assertEqual(aftertraversal_events[0].request.environ, environ)
         self.assertEqual(len(response_events), 1)
         self.assertEqual(response_events[0].response, response)
+        self.assertEqual(result, response.app_iter)
 
     def test_call_pushes_and_pops_threadlocal_manager(self):
         context = DummyContext()
@@ -428,7 +425,7 @@ class TestRouter(unittest.TestCase):
         router = self._makeOne()
         start_response = DummyStartResponse()
         router.threadlocal_manager = DummyThreadLocalManager()
-        result = router(environ, start_response)
+        router(environ, start_response)
         self.assertEqual(len(router.threadlocal_manager.pushed), 1)
         self.assertEqual(len(router.threadlocal_manager.popped), 1)
 
@@ -445,7 +442,7 @@ class TestRouter(unittest.TestCase):
         view = DummyView(response)
         environ = self._makeEnviron(PATH_INFO='/archives/action1/article1')
         self._registerView(view, '', None, None)
-        rootfactory = self._registerRootFactory(context)
+        self._registerRootFactory(context)
         router = self._makeOne()
         start_response = DummyStartResponse()
         result = router(environ, start_response)
@@ -486,7 +483,7 @@ class TestRouter(unittest.TestCase):
         view = DummyView(response)
         environ = self._makeEnviron(PATH_INFO='/archives/action1/article1')
         self._registerView(view, '', None, None)
-        rootfactory = self._registerRootFactory(context)
+        self._registerRootFactory(context)
         router = self._makeOne()
         start_response = DummyStartResponse()
         result = router(environ, start_response)
diff --git a/repoze/bfg/tests/test_security.py b/repoze/bfg/tests/test_security.py
index 08347f49f..0a15831b7 100644
--- a/repoze/bfg/tests/test_security.py
+++ b/repoze/bfg/tests/test_security.py
@@ -151,7 +151,7 @@ class TestViewExecutionPermitted(unittest.TestCase):
             pass
         context = DummyContext()
         directlyProvides(context, IContext)
-        checker = self._registerSecuredView('', True)
+        self._registerSecuredView('', True)
         request = DummyRequest({})
         directlyProvides(request, IRequest)
         result = self._callFUT(context, request, '')
@@ -290,7 +290,6 @@ class TestRemember(unittest.TestCase):
         return remember(*arg)
 
     def test_no_authentication_policy(self):
-        context = DummyContext()
         request = _makeRequest()
         result = self._callFUT(request, 'me')
         self.assertEqual(result, [])
diff --git a/repoze/bfg/tests/test_testing.py b/repoze/bfg/tests/test_testing.py
index 0a9da3e85..7c8b7ad30 100644
--- a/repoze/bfg/tests/test_testing.py
+++ b/repoze/bfg/tests/test_testing.py
@@ -67,7 +67,7 @@ class Test_registerTemplateRenderer(TestBase):
         from repoze.bfg.testing import DummyTemplateRenderer
         self.failUnless(isinstance(renderer, DummyTemplateRenderer))
         from repoze.bfg.chameleon_zpt import render_template_to_response
-        response = render_template_to_response('templates/foo', foo=1, bar=2)
+        render_template_to_response('templates/foo', foo=1, bar=2)
         self.assertEqual(dict(foo=1, bar=2), renderer._received)
 
     def test_registerTemplateRenderer_explicitrenderer(self):
@@ -194,7 +194,7 @@ class Test_registerView(TestBase):
         from zope.interface import Interface
         from repoze.bfg.interfaces import IViewPermission
         from repoze.bfg import testing
-        view = testing.registerViewPermission('moo.html')
+        testing.registerViewPermission('moo.html')
         result = self.registry.getMultiAdapter(
             (Interface, Interface), IViewPermission, 'moo.html')
         self.assertEqual(result, True)
@@ -203,7 +203,7 @@ class Test_registerView(TestBase):
         from zope.interface import Interface
         from repoze.bfg.interfaces import IViewPermission
         from repoze.bfg import testing
-        view = testing.registerViewPermission('moo.html', result=False)
+        testing.registerViewPermission('moo.html', result=False)
         result = self.registry.getMultiAdapter(
             (Interface, Interface), IViewPermission, 'moo.html')
         self.assertEqual(result, False)
@@ -621,7 +621,6 @@ class Test_tearDown(unittest.TestCase):
             raise TypeError
         registry.__init__ = raiseit
         old = {'registry':registry}
-        hook = lambda *arg: None
         try:
             manager.push(old)
             self._callFUT() # doesn't blow up
diff --git a/repoze/bfg/tests/test_traversal.py b/repoze/bfg/tests/test_traversal.py
index 307c241b4..023b82643 100644
--- a/repoze/bfg/tests/test_traversal.py
+++ b/repoze/bfg/tests/test_traversal.py
@@ -259,7 +259,6 @@ class ModelGraphTraverserTests(unittest.TestCase):
     def test_withroute_nothingfancy(self):
         model = DummyContext()
         traverser = self._makeOne(model)
-        routing_args = ((), {})
         environ = {'bfg.routes.matchdict': {}}
         result = traverser(environ)
         self.assertEqual(result['context'], model)
@@ -340,7 +339,6 @@ class FindInterfaceTests(unittest.TestCase):
         bar.__name__ = 'bar'
         baz.__parent__ = bar
         baz.__name__ = 'baz'
-        request = DummyRequest()
         from zope.interface import directlyProvides
         from zope.interface import Interface
         class IFoo(Interface):
@@ -365,7 +363,6 @@ class FindInterfaceTests(unittest.TestCase):
         bar.__name__ = 'bar'
         baz.__parent__ = bar
         baz.__name__ = 'baz'
-        request = DummyRequest()
         result = self._callFUT(baz, DummyRoot)
         self.assertEqual(result.__name__, 'root')
 
diff --git a/repoze/bfg/tests/test_view.py b/repoze/bfg/tests/test_view.py
index 2b33a4ac7..6fbedb934 100644
--- a/repoze/bfg/tests/test_view.py
+++ b/repoze/bfg/tests/test_view.py
@@ -232,7 +232,6 @@ class TestStaticView(BaseTest, unittest.TestCase):
         self.assertEqual(response.directory, path)
 
     def test_relpath(self):
-        import os
         path = 'fixtures'
         view = self._makeOne(path)
         context = DummyContext()
@@ -241,15 +240,12 @@ class TestStaticView(BaseTest, unittest.TestCase):
         request.environ = self._makeEnviron()
         response = view(context, request)
         self.assertEqual(request.copied, True)
-        here = os.path.abspath(os.path.dirname(__file__))
         self.assertEqual(response.root_resource, 'fixtures')
         self.assertEqual(response.resource_name, 'fixtures')
         self.assertEqual(response.package_name, 'repoze.bfg.tests')
         self.assertEqual(response.cache_max_age, 3600)
 
     def test_relpath_withpackage(self):
-        import os
-        path = 'fixtures'
         view = self._makeOne('another:fixtures')
         context = DummyContext()
         request = DummyRequest()
@@ -257,15 +253,12 @@ class TestStaticView(BaseTest, unittest.TestCase):
         request.environ = self._makeEnviron()
         response = view(context, request)
         self.assertEqual(request.copied, True)
-        here = os.path.abspath(os.path.dirname(__file__))
         self.assertEqual(response.root_resource, 'fixtures')
         self.assertEqual(response.resource_name, 'fixtures')
         self.assertEqual(response.package_name, 'another')
         self.assertEqual(response.cache_max_age, 3600)
 
     def test_relpath_withpackage_name(self):
-        import os
-        path = 'fixtures'
         view = self._makeOne('fixtures', package_name='another')
         context = DummyContext()
         request = DummyRequest()
@@ -273,7 +266,6 @@ class TestStaticView(BaseTest, unittest.TestCase):
         request.environ = self._makeEnviron()
         response = view(context, request)
         self.assertEqual(request.copied, True)
-        here = os.path.abspath(os.path.dirname(__file__))
         self.assertEqual(response.root_resource, 'fixtures')
         self.assertEqual(response.resource_name, 'fixtures')
         self.assertEqual(response.package_name, 'another')
diff --git a/repoze/bfg/tests/test_zcml.py b/repoze/bfg/tests/test_zcml.py
index f69afc03b..148b8cac6 100644
--- a/repoze/bfg/tests/test_zcml.py
+++ b/repoze/bfg/tests/test_zcml.py
@@ -68,8 +68,17 @@ class TestViewDirective(unittest.TestCase):
         self.assertEqual(perm, None)
 
     def test_with_dotted_renderer(self):
+        from repoze.bfg.threadlocal import get_current_registry
         from repoze.bfg.interfaces import IView
+        from repoze.bfg.interfaces import IRendererFactory
+        from repoze.bfg.interfaces import IRequest
         context = DummyContext()
+        reg = get_current_registry()
+        def factory(path):
+            def foo(*arg):
+                return 'OK'
+            return foo
+        reg.registerUtility(factory, IRendererFactory, name='.pt')
         view = lambda *arg: None
         self._callFUT(context, 'repoze.view', IDummy, view=view,
                       renderer='foo/template.pt')
@@ -79,6 +88,9 @@ class TestViewDirective(unittest.TestCase):
                    None, False, None, None, None)
         self.assertEqual(actions[0]['discriminator'], discrim)
         register = actions[0]['callable']
+        register()
+        regview = reg.adapters.lookup((IDummy, IRequest), IView, name='')
+        self.assertEqual(regview(None, None).body, 'OK')
 
 class TestNotFoundDirective(unittest.TestCase):
     def setUp(self):
@@ -272,7 +284,6 @@ class TestRemoteUserAuthenticationPolicyDirective(unittest.TestCase):
         actions = context.actions
         self.assertEqual(len(actions), 1)
         regadapt = actions[0]
-        regadapt_discriminator = 'authentication_policy'
         self.assertEqual(regadapt['discriminator'], IAuthenticationPolicy)
         self.assertEqual(regadapt['callable'], None)
         self.assertEqual(regadapt['args'], ())
@@ -714,7 +725,6 @@ class TestAdapterDirective(unittest.TestCase):
 
     def test_no_factories_multiple_for(self):
         context = DummyContext()
-        factory = DummyFactory()
         self.assertRaises(ValueError, self._callFUT, context,
                           factory=[],
                           provides=IFactory,