Merge lp:~ricardokirkner/configglue/update-django-openid-auth-schema into lp:configglue

[Ricardo Kirkner]

Updated django-openid-auth schema in contrib, to support new OPENID_LAUNCHPAD_TEAMS_REQUIRED setting.

Updated django-openid-auth schema in contrib, to support new OPENID_LAUNCHPAD_TEAMS_REQUIRED setting.

[Natalia Bidart (nataliabidart)]

Looks good! Thanks

review: Approve

[Natalia Bidart (nataliabidart)]

Thanks for the test.

review: Approve

[Ubuntu One Auto Pilot (otto-pilot)]

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0, Needs Fixing == 0, Needs Information == 0, Resubmit == 0, Pending == 0. Got: 1 Pending.

[Ricardo Kirkner (ricardokirkner)]

Make tarmac happy

review: Approve

=== modified file 'configglue/contrib/schema/django_openid_auth.py'
--- configglue/contrib/schema/django_openid_auth.py	2012-05-10 12:45:19 +0000
+++ configglue/contrib/schema/django_openid_auth.py	2013-05-03 21:43:26 +0000
@@ -11,7 +11,7 @@
 class DjangoOpenIdAuthSchema(Schema):
     """Configglue schema for django-openid-auth."""
 
-    __version__ = '0.4'
+    __version__ = '0.5'
 
     class openid(Section):
         openid_use_as_admin_login = BoolOption(
@@ -37,6 +37,8 @@
         openid_launchpad_teams_mapping = DictOption()
         openid_launchpad_staff_teams = ListOption(
             item=StringOption())
+        openid_launchpad_teams_required = ListOption(
+            item=StringOption())
         openid_disallow_inames = BoolOption(
             default=False)
         allowed_external_openid_redirect_domains = ListOption(

=== added file 'configglue/tests/test_contrib_schema.py'
--- configglue/tests/test_contrib_schema.py	1970-01-01 00:00:00 +0000
+++ configglue/tests/test_contrib_schema.py	2013-05-03 21:43:26 +0000
@@ -0,0 +1,13 @@
+from unittest import TestCase
+
+from configglue.schema import ListOption, StringOption
+from configglue.contrib.schema import DjangoOpenIdAuthSchema
+
+
+class DjangoOpenIdAuthSchemaTestCase(TestCase):
+
+    def test_openid_launchpad_teams_required_option(self):
+        schema = DjangoOpenIdAuthSchema()
+        option = schema.openid.openid_launchpad_teams_required
+        self.assertTrue(isinstance(option, ListOption))
+        self.assertTrue(isinstance(option.item, StringOption))