Ticket #1762: webtest-ken.patch

setup.py

@@ -46 +46 @@
-
-testtools =  [
-    "nose >= 0.9.3, <= 0.10.0a1",
+    "WebTest",
-]
-
-tgtesttools =  [
-    "nose >= 0.9.3, <= 0.10.0a1",
+    "WebTest",
-]
-
-# python 2.5 compatible list

turbogears/identity/tests/test_identity.py

@@ -55 +55 @@
-
-    @expose()
-    def index(self):
-pass
+return 'identityroot index'
-
-    @expose()
-    def identity_failed(self, **kw):
@@ -69 +69 @@
-    @expose()
-    @identity.require(in_group('peon'))
-    def in_peon_group(self):
+        user = TG_User.by_user_name("samIam") 
+        group_ids = set((TG_Group.by_group_name("peon").id,
+            TG_Group.by_group_name("other").id))
+        assert identity.current.user == user
+        assert identity.current.user_name == user.user_name
+        assert identity.current.user_id == user.id
+        assert identity.current.groups == set(('peon', 'other'))
+        assert identity.current.group_ids == group_ids
+        assert "samIam" == cherrypy.serving.request.identity.user_name
+
-        return 'in_peon_group'
-
-    @expose()
+    @identity.require(in_group('peon'))
+    def identity_current_set(self):
+        user = TG_User.by_user_name("samIam")
+        group_ids = set((TG_Group.by_group_name("peon").id,
+            TG_Group.by_group_name("other").id))
+        assert identity.current.user == user
+        assert identity.current.user_name == user.user_name
+        assert identity.current.user_id == user.id
+        assert identity.current.groups == set(('peon', 'other'))
+        assert identity.current.group_ids == group_ids
+        assert "samIam" == cherrypy.serving.request.identity.user_name
+
+        return 'identity_current_set'
+
+
+    @expose()
-    def test_exposed_require(self):
-        if not hasattr(self.in_peon_group, '_require'):
-            return 'no _require attr'
@@ -136 +162 @@
-            return 'wrong params: %s\nexpected unicode objects' \
-                ' for all strings' % cherrypy.request.params
-
+    @expose()
+    def is_anonymous(self):
+        assert cherrypy.serving.request.identity.user_name == None
+        assert cherrypy.serving.request.identity.anonymous
+        return 'is_anonymous'
-
-class TestIdentity(unittest.TestCase):
-
@@ -151 +182 @@
-        self._original_config = original_config
-        config.configure_loggers(test_config)
-        config.update(test_config['global'])
-
-
+
-        self.init_model()
-
-    def tearDown(self):
-startup.stopTurboGears
+testutil.stop_server
-        config.update(self._original_config)
-
-    def init_model(self):
@@ -181 +211 @@
-
-    def test_user_password_parameters(self):
-        """Controller can receive user_name and password parameters."""
-
-
-
+
+
-
-    def test_user_exists(self):
-        u = TG_User.by_user_name('samIam')
@@ -203 +232 @@
-        """Test if we can set a user password which is encoded as unicode
-        (no encryption algorithm)."""
-        config.update({'identity.soprovider.encryption_algorithm': None})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = u'garçon'
@@ -217 +243 @@
-    def test_user_password_hashed_sha(self):
-        """Test if a sha hashed password is stored in the database."""
-        config.update({'identity.soprovider.encryption_algorithm': 'sha1'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = 'password'
@@ -232 +255 @@
-        """Test if a sha hashed password with unicode characters is stored
-        in the database."""
-        config.update({'identity.soprovider.encryption_algorithm': 'sha1'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = u'garçon'
@@ -246 +266 @@
-    def test_user_password_hashed_md5(self):
-        """Test if a md5 hashed password is stored in the database."""
-        config.update({'identity.soprovider.encryption_algorithm': 'md5'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = 'password'
@@ -261 +278 @@
-        """Test if a md5 hashed password with unicode characters is stored
-        in the database."""
-        config.update({'identity.soprovider.encryption_algorithm': 'md5'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = u'garçon'
@@ -278 +292 @@
-        test ensures that the encryption algorithm does handle non-unicode
-        parameters gracefully."""
-        config.update({'identity.soprovider.encryption_algorithm': 'md5'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = u'garçon'.encode('UTF-8')
@@ -293 +304 @@
-        """Test that we can store raw values in the password field
-        (without being hashed)."""
-        config.update({'identity.soprovider.encryption_algorithm':'sha1'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.set_password_raw('password')
@@ -306 +314 @@
-
-    def test_user_password_raw_unicode(self):
-        config.update({'identity.soprovider.encryption_algorithm':'sha1'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.set_password_raw(u'garçon')
@@ -322 +327 @@
-        config.update({'identity.soprovider.encryption_algorithm': 'custom',
-            'identity.custom_encryption':
-                'identity.tests.test_identity.mycustomencrypt'})
-        # force new config values to load
-        startup.startTurboGears()
-        testutil.create_request('/')
-        hub.begin()
-        u = TG_User.by_user_name('samIam')
-        u.password = 'password'
@@ -335 +337 @@
-
-    def test_anonymous_browsing(self):
-        """Test if we can show up anonymously."""
-testutil.create_request('/
-identity.current.anonymous
+response = testutil.go('/is_anonymous
+'is_anonymous' in response
-
-    def test_deny_anonymous(self):
-        """Test that we have secured an url from anonymous users."""
-
-
-
+
+
-
-    def test_deny_anonymous_viewable(self):
-        """Test that a logged in user can see an resource blocked
-        from anonymous users."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_logout(self):
-gets 
-testutil.create_request('/in_peon_group
+
+response = testutil.go('/identity_current_set
-            'user_name=samIam&password=secret&login=Login')
-
-
-
-
-
-
+
+
+
+
-
-    def test_logout_with_set_identity(self):
-        """Test that logout works even when there is no visit_key
-        (e.g. when testutils.set_identity_user is used)."""
-        request = testutil.DummyRequest()
-        old_user = testutil.test_user
-        user = TG_User.by_user_name("samIam")
-        testutil.set_identity_user(user)
-        testutil.attach_identity(request)
-        testutil.set_identity_user(old_user)
-        testutil.call_with_request(cherrypy.root.logout, request)
-        assert request.identity.anonymous
-
-    def test_require_group(self):
-        """Test that a anonymous user"""
-
-
-
+
+
-
-    def test_require_expose_required_permission(self):
-        """Test that the decorator exposes the correct permissions via _require
-        attribute on the actual method."""
-
-
-
+
+
-
-    def test_require_group_viewable(self):
-        """Test that a user with proper group membership can see a restricted url."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
-
+
-
-    def test_require_group_viewable(self):
-        """Test that the current user and group properties are set correctly."""
-testutil.create_request('/in_peon_group
+response = testutil.go('/identity_current_set
-            'user_name=samIam&password=secret&login=Login')
-
-
-
-
-
-
-
-
+
-
-    def test_user_not_in_right_group(self):
-        """Test that a user is denied access if they aren't in the right group."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_require_permission(self):
-        """Test that an anonymous user is denied access to a permission restricted url."""
-
-
-
+
+
-
-    def test_require_permission_viewable(self):
-        """Test that a user with proper permissions can see a restricted url."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_user_lacks_permission(self):
-        """Test that a user is denied acces if they don't have the proper permission."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_user_info_available(self):
-        """Test that we can see user information inside our controller."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_bad_login(self):
-        """Test that we are denied access if we provide a bad login."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=wrong&login=Login')
-
-
+
-
-    def test_restricted_subdirectory(self):
-        """Test that we can restrict access to a whole subdirectory."""
-
-
-
+
+
-
-    def test_restricted_subdirectory_viewable(self):
-        """Test that we can access a restricted subdirectory
-        if we have proper credentials."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_decoratator_in_restricted_subdirectory(self):
-        """Test that we can require a different permission
-        in a protected subdirectory."""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_decoratator_failure_in_restricted_subdirectory(self):
-        """Test that we can get an identity failure from a decorator
-        in a restricted subdirectory"""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_explicit_checks_in_restricted_subdirectory(self):
-        """Test that explicit permission checks in a protected
-        directory is handled as expected"""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_throwing_identity_exception_in_restricted_subdirectory(self):
-        """Test that throwing an IdentityException in a protected
-        directory is handled as expected"""
-testutil.create_request
+response = testutil.go
-            'user_name=samIam&password=secret&login=Login')
-
-
+
-
-    def test_decode_filter_whenidfails(self):
-        """Test that the decode filter doesn't break with nested
-        variables and Identity"""
-        params = urllib.quote(IdentityRoot._test_encoded_params.decode(
-            'utf-8').encode('latin-1'), '=&')
-
-
-
+
+
-
-    def test_decode_filter_whenidworks(self):
-        """Test that the decode filter doesn't break with nested
@@ -511 +473 @@
-        params = urllib.quote(IdentityRoot._test_encoded_params.decode(
-            'utf-8').encode('latin-1'), '=&')
-        params += '&user_name=samIam&password=secret&login=Login'
-
-
-
+
+
-
-
-class TestTGUser(testutil.DBTest):
@@ -522 +483 @@
-    def setUp(self):
-        self._identity_on = config.get('identity.on', False)
-        config.update({'identity.on': False})
-
-
-
-
-
-
+
-        testutil.DBTest.setUp(self)
-
-    def tearDown(self):
-        testutil.DBTest.tearDown(self)
-
-
+
-        config.update({'identity.on': self._identity_on})
-
-    def test_create_user(self):

turbogears/identity/tests/test_visit.py

@@ -2 +2 @@
-from unittest import TestCase
-import cherrypy
-from turbogears import config, controllers, expose, startup, testutil, visit
+from Cookie import SimpleCookie
-
-
-morsel
+response
-    """Returns a dict containing cookie information to pass to a server."""
-{'Cookie': morsel.output(header="")[1:]}
+dict(Cookie=response.headers['Set-Cookie'])
-
-
-class VisitRoot(controllers.RootController):
-
-    @expose()
-    def index(self):
-
+
+
+
+
+
-
-
-class TestVisit(TestCase):
-
-    def setUp(self):
@@ -27 +31 @@
-        cherrypy.root = VisitRoot()
-
-    def tearDown(self):
-startup.stopTurboGears
+testutil.stop_server
-        config.update({'visit.timeout': self._visit_timeout})
-        config.update({'visit.on': self._visit_on})
-
-    def test_visit_response(self):
-        """Test if the visit cookie is set in cherrypy.response."""
-testutil.create_request
-cherrypy.response.simple_cookie
+response = testutil.go
+response.cookies_set
-
-    def test_new_visit(self):
-        """Test that we can see a new visit on the server."""
-testutil.create_request
-visit.current().is_new
+response = testutil.go
+response.raw['new']
-
-    def test_old_visit(self):
-        """Test if we can track a visitor over time."""
-testutil.create_request
+response = testutil.go
-        # first visit's cookie
-cherrypy.response.simple_cookie
-testutil.create_request("/", headers=cookie_header(morsel
-visit.current().is_new
+response.cookies_set
+response = testutil.go("/", headers=cookie_header(response
+response.raw['new']
-
-    def test_cookie_expires(self):
-        """Test if the visit timeout mechanism works."""
@@ -55 +59 @@
-        try:
-            # set expiration to one second for this test only
-            config.update({'visit.timeout': 1.0/60})
-testutil.create_request
-cherrypy.response.simple_cookie
+response = testutil.go
+response.cookies_set
-            time.sleep(2) # 2 seconds
-testutil.create_request("/", headers=cookie_header(morsel
+response = testutil.go("/", headers=cookie_header(response
-        finally:
-            config.update({'visit.timeout': timeout})
-cherrypy.response.simple_cookie
-.value != morsel.value
+response.cookies_set
+ != morsel
-            'cookie values should not match'
-visit.current().is_new
+response.raw['new']
-            'this should be a new visit, as the cookie has expired'
-
-    def test_cookie_not_permanent(self):
-        """Check that by default the visit cookie is not permanent."""
-
-
+
+
+
-        assert not morsel['expires'] and not morsel['max-age']
-
-    def test_cookie_permanent(self):
@@ -80 +85 @@
-            # set cookie permanent for this test only (needs restart)
-            startup.stopTurboGears()
-            config.update({'visit.cookie.permanent': True})
-
-
-
+
+
+
+
-        finally:
-            config.update({'visit.cookie.permanent': permanent})
-3000
+'3000'
-        expires = time.mktime(time.strptime(morsel['expires'],
-            '%a, %d-%b-%Y %H:%M:%S GMT')[:8] + (0,))
-
-
+
+
+

turbogears/visit/api.py

@@ -251 +251 @@
-        max_age = self.cookie_max_age
-        if max_age:
-            # use 'expires' because MSIE ignores 'max-age'
-
+'"%s"' % 
-                "%a, %d-%b-%Y %H:%M:%S GMT",
-                time.gmtime(time.time() + max_age))
-            # 'max-age' takes precedence on standard conformant browsers

turbogears/tests/test_form_controllers.py

@@ -31 +31 @@
-    def testform(self, name, date, age, tg_errors=None):
-        if tg_errors:
-            self.has_errors = True
-
-
-
+
-
-    @expose()
-    @validate(form=myform)
-    def testform_new_style(self, name, date, age):
-        if cherrypy.request.validation_errors:
-            self.has_errors = True
-
-
-
+
-
-def test_form_translation():
-    """Form input is translated into properly converted parameters"""
-
-
-
-
-
+
+
+
+
-
-def test_form_translation_new_style():
-    """Form input is translated into properly converted parameters"""
-
-
-
-
-
+
+
+
+
-
-def test_invalid_form_with_error_handling():
-    """Invalid forms can be handled by the method"""
-
-
-
+
-
-def test_css_should_appear():
-    """CSS should appear when asked for"""
-
-
+
+
+
-
-def test_javascript_should_appear():
-    """JavaScript should appear when asked for"""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-def test_include_mochikit():
-    """JSLinks (and MochiKit especially) can be included easily"""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-def test_suppress_mochikit():
-    """MochiKit inclusion can be suppressed"""
-
-
-
+
+
-    # repair the fixture
-
-
-
-
-
+
-
+    included = testutil.go("/usemochi")
+    assert "MochiKit.js" not in suppressed.body
+    assert "MochiKit.js" in included.body
+
-def test_mochikit_everywhere():
-    """MochiKit can be included everywhere by setting tg.mochikit_all"""
- {"tg.mochikit_all"
-testutil.create_request
- {"tg.mochikit_all"
-cherrypy.response.body[0]
+{"tg.mochikit_all" 
+response = testutil.go
+{"tg.mochikit_all" 
+response
-
-def test_mochikit_nowhere():
-    """Setting tg.mochikit_suppress will prevent including it everywhere"""
-    config.update({"global": {"tg.mochikit_all": True}})
-    config.update({"global": {"tg.mochikit_suppress": True}})
-testutil.create_request
+response = testutil.go
-    config.update({"global": {"tg.mochikit_all": False}})
-    config.update({"global": {"tg.mochikit_suppress": False}})
-cherrypy.response.body[0]
+response
-
-def test_include_widgets():
-    """Any widget can be included everywhere by setting tg.include_widgets"""
-    config.update({"global": {"tg.include_widgets": ["mochikit"]}})
-testutil.create_request
+response = testutil.go
-    config.update({"global": {"tg.include_widgets": []}})
-cherrypy.response.body[0]
+response
-
-
-class State(object):
@@ -146 +138 @@
-        super(TestValidationState, self).__init__(*args, **kw)
-
-    def test_counter_is_incremented(self):
-cherrypy.root = self.Controller(
+testutil.start_server(root = self.Controller()
-        # parameter values are irrelevant
-        url = '/validation?a=1&b=2&c.a=3&c.b=4'
-
-
+
-        msg = "Validation state is not handled properly"
-        # 4 == 1 (a) + 1(b) + 1(c.a) + 1(c.b)
-body
+response

turbogears/tests/test_errorhandling.py

@@ -1 +1 @@
-import unittest
-
-import cherrypy
-
-from turbogears.controllers import error_handler, exception_handler, \
-                                   expose, validate, RootController, Controller
-from turbogears.errorhandling import FailsafeSchema
@@ -94 +91 @@
-        "second": validators.Int(not_empty=True)})
-    @error_handler(defaulterrorhandler)
-    def positionalargs(self, first, second, *args, **kw):
-        self.first = first
-        self.second = second
-        self.third = args[0]
-        return dict(title="Positional arguments", first=first, second=second,
-
+third=args[0], 
-
-    @expose()
-    @validate(validators={"bar": validators.Int(not_empty=True)})
@@ -141 +135 @@
-        return self.notexposed(bar)
-
-    def continuation(self, tg_source):
-
-
+
+
+
-
-    @expose()
-    @validate(validators={"bar": validators.Int(not_empty=True)})
@@ -211 +206 @@
-class TestErrorHandler(unittest.TestCase):
-
-    def setUp(self):
-
-
+
+
+
-
-    def test_defaultErrorHandler(self):
-        """Default error handler."""
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
+response = testutil.go
+response
-
-    def test_specialisedErrorHandler(self):
-        """Error handler specialisation."""
-
-
-
-
-
-
-
-
+
+
+
+
+
+
-
-    def test_exceptionErrorHandler(self):
-        """Error handler for exceptions."""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-    def test_recursiveErrorHandler(self):
-        """Recursive error handler."""
-
-
-
-
-
+
+
+
+
-
-    def test_implicitErrorHandler(self):
-        """Implicit error handling."""
-
-
-
-
-
-
+
+
+
+
-
-    def test_normalMethodErrorHandler(self):
-        """Normal method as an error handler."""
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
+response = testutil.go
+response
-
-    def test_infiniteRecursionPrevention(self):
-        """Infinite recursion prevention."""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-    def test_positionalArgs(self):
-        """Positional argument validation."""
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
-cherrypy.root.first
-cherrypy.root.second
-cherrypy.root.third
+response = testutil.go
+response
+response = testutil.go
+response
+response = testutil.go
+response
+response = testutil.go
+response
+response.raw['first']
+response.raw['second']
+response.raw['third']
-
-    def test_missingArgs(self):
-        """Arguments required in validation missing."""
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
+response = testutil.go
+response
-
-    def test_nohandler(self):
-        """No error hanlder declared."""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-    def test_bindArgs(self):
-        """Arguments can be bond to an error handler."""
-testutil.create_request
-cherrypy.response.body[0]
+response = testutil.go
+response
-
-    def test_notExposed(self):
-        """Validation error handling is decoupled from expose."""
-testutil.create_request
-cherrypy.response.body[0]
-cherrypy.response.body[0]
+response = testutil.go
+response
+response
-
-    def test_continuations(self):
-        """Continuations via error handling mechanism."""
-testutil.create_request
-cherrypy.response.body[0]
-cherrypy.root.continuation
+response = testutil.go
+response
+response.raw['continuation']
-
-    def test_nested(self):
-        """Potentially ambiguous cases."""
-testutil.create_request
-cherrypy.response.body[0]
-testutil.create_request