[#1226] Return the old behaviour for create_user

ckan · Oct 14, 2013 · 3273ebf · 3273ebf
1 parent c269023
commit 3273ebf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/ckan/logic/auth/create.py b/ckan/logic/auth/create.py
@@ -105,7 +105,6 @@ def rating_create(context, data_dict):
 
 
 @logic.auth_allow_anonymous_access
-@logic.auth_sysadmins_check
 def user_create(context, data_dict=None):
     using_api = 'api_version' in context
     create_user_via_api = new_authz.check_config_permission(

diff --git a/ckan/tests/functional/api/test_user.py b/ckan/tests/functional/api/test_user.py
@@ -83,7 +83,7 @@ def teardown_class(cls):
 
         model.repo.rebuild_db()
 
-    def test_user_create_api_disabled_sysadmin(self):
+    def test_user_create_api_enabled_sysadmin(self):
         params = {
             'name': 'testinganewusersysadmin',
             'email': 'testinganewuser@ckan.org',
@@ -93,7 +93,7 @@ def test_user_create_api_disabled_sysadmin(self):
                 extra_environ={'Authorization': str(self.sysadmin_user.apikey)},
                 expect_errors=True)
         res_dict = res.json
-        assert res_dict['success'] is False
+        assert res_dict['success'] is True
 
     def test_user_create_api_disabled_anon(self):
         params = {
@@ -190,7 +190,6 @@ def test_user_create_api_disabled(self):
             'password': 'random',
         }
         res = self.app.post('/api/3/action/user_create', json.dumps(params),
-                extra_environ={'Authorization': str(self.sysadmin_user.apikey)},
                 expect_errors=True)
         res_dict = res.json
         assert res_dict['success'] is False
@@ -229,7 +228,6 @@ def test_user_create_api_disabled(self):
             'password': 'random',
         }
         res = self.app.post('/api/3/action/user_create', json.dumps(params),
-                extra_environ={'Authorization': str(self.sysadmin_user.apikey)},
                 expect_errors=True)
         res_dict = res.json
         assert res_dict['success'] is False