From 61dc3a4197e559322e632ce61c7441d0f99b5fbd Mon Sep 17 00:00:00 2001 From: Vitor Baptista Date: Tue, 8 Apr 2014 15:45:58 -0300 Subject: [PATCH] [#1505] Remove assertions that group_member_create was called in the tests If user_invite fails when group_member_create fails, and succeeds when group_member_create succeeds, there's no need to assert that group_member_create was actually called. --- ckan/new_tests/logic/auth/test_create.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/ckan/new_tests/logic/auth/test_create.py b/ckan/new_tests/logic/auth/test_create.py index 039ba3d96c1..f5ced243ed0 100644 --- a/ckan/new_tests/logic/auth/test_create.py +++ b/ckan/new_tests/logic/auth/test_create.py @@ -25,8 +25,6 @@ def test_user_invite_delegates_correctly_to_group_member_create(self, gmc): 'auth_user_obj': user } data_dict = {'group_id': 42} - group_member_create_data_dict = data_dict.copy() - group_member_create_data_dict['id'] = data_dict['group_id'] gmc.return_value = {'success': False} nose.tools.assert_raises(logic.NotAuthorized, helpers.call_auth, @@ -35,5 +33,3 @@ def test_user_invite_delegates_correctly_to_group_member_create(self, gmc): gmc.return_value = {'success': True} result = helpers.call_auth('user_invite', context=context, **data_dict) assert result is True - - gmc.assert_called_twice_with(context, group_member_create_data_dict)