diff --git a/ckan/tests/functional/test_group.py b/ckan/tests/functional/test_group.py index 59d745f2df4..9707e8889af 100644 --- a/ckan/tests/functional/test_group.py +++ b/ckan/tests/functional/test_group.py @@ -7,7 +7,7 @@ import ckan.model as model from ckan.lib.create_test_data import CreateTestData from ckan.logic import check_access, NotAuthorized - + from pylons import config from ckan.tests import * @@ -17,11 +17,11 @@ class MockGroupControllerPlugin(SingletonPlugin): implements(IGroupController) - + def __init__(self): from collections import defaultdict self.calls = defaultdict(int) - + def read(self, entity): self.calls['read'] += 1 @@ -77,7 +77,7 @@ def test_index(self): self.check_named_element(res, 'tr', group_title, group_packages_count, group_description) res = res.click(group_title) assert groupname in res - + def test_read_non_existent(self): name = u'group_does_not_exist' offset = url_for(controller='group', action='read', id=name) @@ -189,7 +189,7 @@ def test_2_edit(self): res = form.submit('save', status=302, extra_environ={'REMOTE_USER': 'russianfan'}) # should be read page # assert 'Groups - %s' % self.groupname in res, res - + model.Session.remove() group = model.Group.by_name(self.groupname) assert group.title == newtitle, group @@ -214,11 +214,11 @@ def test_3_edit_form_has_new_package(self): user = model.User.by_name(u'russianfan') model.setup_default_user_roles(pkg, [user]) model.repo.commit_and_remove() - + res = self.app.get(offset, status=200, extra_environ={'REMOTE_USER': 'russianfan'}) assert 'annakarenina' in res, res assert 'newone' in res - + def test_4_new_duplicate_package(self): prefix = '' @@ -265,14 +265,14 @@ def test_delete(self): CreateTestData.create_groups([{'name': group_name, 'packages': [self.packagename]}], admin_user_name='russianfan') - + group = model.Group.by_name(group_name) offset = url_for(controller='group', action='edit', id=group_name) res = self.app.get(offset, status=200, extra_environ={'REMOTE_USER': 'russianfan'}) main_res = self.main_div(res) assert 'Edit: %s' % group.title in main_res, main_res assert 'value="active" selected' in main_res, main_res - + # delete form = res.forms['group-edit'] form['state'] = 'deleted' @@ -291,7 +291,7 @@ class TestNew(FunctionalTestCase): def setup_class(self): model.Session.remove() CreateTestData.create() - + self.packagename = u'testpkg' model.repo.new_revision() model.Session.add(model.Package(name=self.packagename)) @@ -372,7 +372,7 @@ def test_3_new_duplicate_group(self): assert 'Group name already exists' in res, res self.check_tag(res, '