Fix tests - package admin role has gone.

ckan · Apr 15, 2015 · b4a99a1 · b4a99a1
1 parent 274aae4
commit b4a99a1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/ckan/tests/legacy/__init__.py b/ckan/tests/legacy/__init__.py
@@ -95,9 +95,9 @@ def _paster(cls, cmd, config_path_rel):
 class CommonFixtureMethods(BaseCase):
 
     @classmethod
-    def create_package(self, data={}, admins=[], **kwds):
+    def create_package(self, data={}, **kwds):
         # Todo: A simpler method for just creating a package.
-        CreateTestData.create_arbitrary(package_dicts=[data or kwds], admins=admins)
+        CreateTestData.create_arbitrary(package_dicts=[data or kwds])
 
     @classmethod
     def create_user(cls, **kwds):

diff --git a/ckan/tests/legacy/functional/api/model/test_package.py b/ckan/tests/legacy/functional/api/model/test_package.py
@@ -366,7 +366,7 @@ def test_09_update_package_entity_not_found(self):
     def create_package_with_admin_user(self, package_data):
         '''Creates a package with self.user as admin and provided package_data.
         '''
-        self.create_package(admins=[self.user], data=package_data)
+        self.create_package(data=package_data)
 
     def assert_package_update_ok(self, package_ref_attribute,
                                  method_str):
@@ -695,7 +695,7 @@ def test_package_update_delete_resource(self):
     def test_entity_delete_ok(self):
         # create a package with package_fixture_data
         if not self.get_package_by_name(self.package_fixture_data['name']):
-            self.create_package(admins=[self.user], name=self.package_fixture_data['name'])
+            self.create_package(name=self.package_fixture_data['name'])
         assert self.get_package_by_name(self.package_fixture_data['name'])
         # delete it
         offset = self.package_offset(self.package_fixture_data['name'])
@@ -708,7 +708,7 @@ def test_entity_delete_ok(self):
     def test_entity_delete_ok_without_request_headers(self):
         # create a package with package_fixture_data
         if not self.get_package_by_name(self.package_fixture_data['name']):
-            self.create_package(admins=[self.user], name=self.package_fixture_data['name'])
+            self.create_package(name=self.package_fixture_data['name'])
         assert self.get_package_by_name(self.package_fixture_data['name'])
         # delete it
         offset = self.package_offset(self.package_fixture_data['name'])

diff --git a/ckan/tests/legacy/functional/test_package.py b/ckan/tests/legacy/functional/test_package.py
@@ -409,7 +409,6 @@ def _reset_data(self):
              'resources':[{'url':u'url escape: & umlaut: \xfc quote: "',
                           'description':u'description escape: & umlaut: \xfc quote "',
                           }],
-             'admins':[u'testadmin'],
              })
 
         self.editpkg = model.Package.by_name(self.editpkg_name)