[#707] Remove some unnecessary test code

These actions are not what's being tested here
ckan · Jun 27, 2013 · f7f4955 · f7f4955
1 parent 7a9f64b
commit f7f4955
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 41 deletions.
diff --git a/ckan/tests/functional/api/model/test_group.py b/ckan/tests/functional/api/model/test_group.py
@@ -302,26 +302,6 @@ def _group_create(self, group_name):
                                           {'id': self.package['name']}],
                                       )
 
-        # Let's just make sure that worked.
-        package = tests.call_action_api(self.app, 'package_show',
-                                        id=self.package['id'])
-        assert group['name'] in [group_['name']
-                                 for group_ in package['groups']]
-        group = tests.call_action_api(self.app, 'group_show', id=group['id'])
-        assert self.package['name'] in [package_['name'] for package_ in
-                                        group['packages']]
-        assert self.group_visitor['name'] not in [user['name']
-                                                  for user in group['users']]
-        assert self.group_member['name'] in [user['name']
-                                             for user in group['users']
-                                             if user['capacity'] == 'member']
-        assert self.group_editor['name'] in [user['name']
-                                             for user in group['users']
-                                             if user['capacity'] == 'editor']
-        assert self.group_admin['name'] in [user['name']
-                                            for user in group['users']
-                                            if user['capacity'] == 'admin']
-
         return group
 
     def _test_group_purge(self, group_name, by_id):

diff --git a/ckan/tests/functional/api/model/test_organization.py b/ckan/tests/functional/api/model/test_organization.py
@@ -84,27 +84,6 @@ def _organization_create(self, organization_name):
                               owner_org=organization['name'],
                               apikey=self.sysadmin.apikey)
 
-        # Let's just make sure that worked.
-        package = tests.call_action_api(self.app, 'package_show',
-                                        id=self.package['id'])
-        assert package['organization']['name'] == organization['name']
-        organization = tests.call_action_api(self.app, 'organization_show',
-                                             id=organization['id'])
-        assert self.package['name'] in [package_['name']
-                                        for package_ in
-                                        organization['packages']]
-        user_names = [user['name'] for user in organization['users']]
-        assert self.org_visitor['name'] not in user_names
-        members = [user['name'] for user in organization['users']
-                   if user['capacity'] == 'member']
-        assert self.org_member['name'] in members
-        editors = [user['name'] for user in organization['users']
-                   if user['capacity'] == 'editor']
-        assert self.org_editor['name'] in editors
-        admins = [user['name'] for user in organization['users']
-                  if user['capacity'] == 'admin']
-        assert self.org_admin['name'] in admins
-
         return organization
 
     def _test_organization_purge(self, org_name, by_id):