diff options
Diffstat (limited to 'repoze/bfg/tests/test_configuration.py')
| -rw-r--r-- | repoze/bfg/tests/test_configuration.py | 46 |
1 files changed, 15 insertions, 31 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): |
