Added some OAuth plugin tests

diff --git a/mediagoblin/tests/test_mgoblin_app.ini b/mediagoblin/tests/test_mgoblin_app.ini
index 3b979ff7f2f87db49b004169a7f15ba201ddb0bc..b112d298e5bf42ad887e87bb74230ba5800d5465 100644 (file)
--- a/mediagoblin/tests/test_mgoblin_app.ini
+++ b/mediagoblin/tests/test_mgoblin_app.ini
@@ -31,3 +31,6 @@ lock_dir = %(here)s/test_user_dev/beaker/cache/lock
 CELERY_ALWAYS_EAGER = true
 CELERY_RESULT_DBURI = "sqlite:///%(here)s/test_user_dev/celery.db"
 BROKER_HOST = "sqlite:///%(here)s/test_user_dev/kombu.db"
+
+[plugins]
+[[mediagoblin.plugins.oauth]]

diff --git a/mediagoblin/tests/test_oauth.py b/mediagoblin/tests/test_oauth.py
new file mode 100644 (file)
index 0000000..ced2ef7
--- /dev/null
+++ b/mediagoblin/tests/test_oauth.py
@@ -0,0 +1,133 @@
+# GNU MediaGoblin -- federated, autonomous media hosting
+# Copyright (C) 2011, 2012 MediaGoblin contributors.  See AUTHORS.
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+import logging
+
+from mediagoblin import mg_globals
+from mediagoblin.init.plugins import setup_plugins
+from mediagoblin.tools import template, pluginapi
+from mediagoblin.tests.tools import get_test_app, fixture_add_user
+from mediagoblin.tests.test_pluginapi import with_cleanup, build_config
+
+
+_log = logging.getLogger(__name__)
+
+
+class TestOAuth(object):
+    def setUp(self):
+        self.app = get_test_app()
+        self.db = mg_globals.database
+
+        self.pman = pluginapi.PluginManager()
+
+        self.user_password = '4cc355_70k3N'
+        self.user = fixture_add_user('joauth', self.user_password)
+
+        self.login()
+
+    def login(self):
+        self.app.post(
+                '/auth/login/', {
+                    'username': self.user.username,
+                    'password': self.user_password})
+
+    def register_client(self, name, client_type, description=None,
+            redirect_uri=''):
+        return self.app.post(
+                '/oauth/client/register', {
+                    'name': name,
+                    'description': description,
+                    'type': client_type,
+                    'redirect_uri': redirect_uri})
+
+    def get_context(self, template_name):
+        return template.TEMPLATE_TEST_CONTEXT[template_name]
+
+    def test_1_public_client_registration_without_redirect_uri(self):
+        ''' Test 'public' OAuth client registration without any redirect uri '''
+        response = self.register_client('OMGOMGOMG', 'public',
+                'OMGOMG Apache License v2')
+
+        ctx = self.get_context('oauth/client/register.html')
+
+        client = self.db.OAuthClient.query.filter(
+                self.db.OAuthClient.name == 'OMGOMGOMG').first()
+
+        assert response.status_int == 200
+
+        # Should display an error
+        assert ctx['form'].redirect_uri.errors
+
+        # Should not pass through
+        assert not client
+
+    def test_2_successful_public_client_registration(self):
+        ''' Successfully register a public client '''
+        self.login()
+        self.register_client('OMGOMG', 'public', 'OMG!',
+                'http://foo.example')
+
+        client = self.db.OAuthClient.query.filter(
+                self.db.OAuthClient.name == 'OMGOMG').first()
+
+        # Client should have been registered
+        assert client
+
+    def test_3_successful_confidential_client_reg(self):
+        ''' Register a confidential OAuth client '''
+        response = self.register_client('GMOGMO', 'confidential', 'NO GMO!')
+
+        assert response.status_int == 302
+
+        client = self.db.OAuthClient.query.filter(
+                self.db.OAuthClient.name == 'GMOGMO').first()
+
+        # Client should have been registered
+        assert client
+
+        return client
+
+    def test_4_authorize_confidential_client(self):
+        ''' Authorize a confidential client as a logged in user '''
+        client = self.test_3_successful_confidential_client_reg()
+
+        redirect_uri = 'https://foo.example'
+        response = self.app.get('/oauth/authorize', {
+                'client_id': client.identifier,
+                'scope': 'admin',
+                'redirect_uri': redirect_uri})
+
+        # User-agent should NOT be redirected
+        assert response.status_int == 200
+
+        ctx = self.get_context('oauth/authorize.html')
+
+        form = ctx['form']
+
+        # Short for client authorization post reponse
+        capr = self.app.post(
+                '/oauth/client/authorize', {
+                    'client_id': form.client_id.data,
+                    'allow': 'Allow',
+                    'next': form.next.data})
+
+        assert capr.status_int == 302
+
+        authorization_response = capr.follow()
+
+        assert authorization_response.location.startswith(redirect_uri)
+
+        return authorization_response

diff --git a/mediagoblin/tests/tools.py b/mediagoblin/tests/tools.py
index 6fd11fc2843cd1c19adf4e50155d2e4e2d4fa2aa..b9b0fcd2842ba3db6a29898a1398a103df22ab03 100644 (file)
--- a/mediagoblin/tests/tools.py
+++ b/mediagoblin/tests/tools.py
@@ -60,7 +60,7 @@ class BadCeleryEnviron(Exception): pass
 class TestingMeddleware(BaseMeddleware):
     """
     Meddleware for the Unit tests
-    
+
     It might make sense to perform some tests on all
     requests/responses. Or prepare them in a special
     manner. For example all html responses could be tested
@@ -100,7 +100,7 @@ def suicide_if_bad_celery_environ():
     if not os.environ.get('CELERY_CONFIG_MODULE') == \
             'mediagoblin.init.celery.from_tests':
         raise BadCeleryEnviron(BAD_CELERY_MESSAGE)
-    
+
 
 def get_test_app(dump_old_app=True):
     suicide_if_bad_celery_environ()