Commit 50b6bf27 by Omar Khan

Registration: don't require terms of service if checkbox is hidden

Fixes test failures in e4613749
parent d46ea873
...@@ -1628,7 +1628,11 @@ def create_account_with_params(request, params): ...@@ -1628,7 +1628,11 @@ def create_account_with_params(request, params):
not do_external_auth not do_external_auth
) )
# Can't have terms of service for certain SHIB users, like at Stanford # Can't have terms of service for certain SHIB users, like at Stanford
registration_fields = getattr(settings, 'REGISTRATION_EXTRA_FIELDS', {})
tos_required = ( tos_required = (
registration_fields.get('terms_of_service') != 'hidden' or
registration_fields.get('honor_code') != 'hidden'
) and (
not settings.FEATURES.get("AUTH_USE_SHIB") or not settings.FEATURES.get("AUTH_USE_SHIB") or
not settings.FEATURES.get("SHIB_DISABLE_TOS") or not settings.FEATURES.get("SHIB_DISABLE_TOS") or
not do_external_auth or not do_external_auth or
......
...@@ -1683,6 +1683,16 @@ class RegistrationViewTest(ThirdPartyAuthTestMixin, ApiTestCase): ...@@ -1683,6 +1683,16 @@ class RegistrationViewTest(ThirdPartyAuthTestMixin, ApiTestCase):
} }
) )
@override_settings(REGISTRATION_EXTRA_FIELDS={"honor_code": "hidden", "terms_of_service": "hidden"})
def test_register_hidden_honor_code_and_terms_of_service(self):
response = self.client.post(self.url, {
"email": self.EMAIL,
"name": self.NAME,
"username": self.USERNAME,
"password": self.PASSWORD,
})
self.assertHttpOK(response)
def test_missing_fields(self): def test_missing_fields(self):
response = self.client.post( response = self.client.post(
self.url, self.url,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment