Changeset - 1de3e5d104bf
Parent rev.
Child rev.
[Not reviewed]
default
0 3 0
Mads Kiilerich (mads) - 6 years ago 2020-06-24 17:12:28
mads@kiilerich.com
Grafted from: 0e14bbd2a0e7
tests: consistent use of more helpful raise_exception "fixture"

"Total Crash" was too misleading when debugging problems.
3 files changed with 29 insertions and 32 deletions:
kallithea/tests/api/api_base.py
25
28
kallithea/tests/fixture.py
2
2
kallithea/tests/functional/test_admin_repos.py
2
2
0 comments (0 inline, 0 general)
kallithea/tests/api/api_base.py
➞
Show inline comments
 
@@ -33,13 +33,14 @@ from kallithea.model.meta import Session
 
from kallithea.model.repo import RepoModel
 
from kallithea.model.repo_group import RepoGroupModel
 
from kallithea.model.scm import ScmModel
 
from kallithea.model.user import UserModel
 
from kallithea.model.user_group import UserGroupModel
 
from kallithea.tests import base
 
from kallithea.tests.fixture import Fixture
 
from kallithea.tests.fixture import Fixture, raise_exception
 

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
API_URL = '/_admin/api'

			




	
	
	
		
 
TEST_USER_GROUP = 'test_user_group'

			




	
	
	
		
 
TEST_REPO_GROUP = 'test_repo_group'

			




	
	
	
		
 

			




	
	
		
 
@@ -60,16 +61,12 @@ def _build_data(apikey, method, **kw):

			




	
	
	
		
 
    })

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
jsonify = lambda obj: ext_json.loads(ext_json.dumps(obj))

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
def crash(*args, **kwargs):

			




	
	
	
		
 
    raise Exception('Total Crash !')

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
def api_call(test_obj, params):

			




	
	
	
		
 
    response = test_obj.app.post(API_URL, content_type='application/json',

			




	
	
	
		
 
                                 params=params)

			




	
	
	
		
 
    return response

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
		
 
@@ -346,13 +343,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'rescan_repos')

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = {'added': [], 'removed': []}

			




	
	
	
		
 
        self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(ScmModel, 'repo_scan', crash)

			




	
	
	
		
 
    @mock.patch.object(ScmModel, 'repo_scan', raise_exception)

			




	
	
	
		
 
    def test_api_rescann_error(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'rescan_repos', )

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'Error occurred during rescan repositories action'

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
		
 
@@ -436,13 +433,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            expected = ret

			




	
	
	
		
 
            self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 
        finally:

			




	
	
	
		
 
            fixture.destroy_user(usr.user_id)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'create_or_update', crash)

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'create_or_update', raise_exception)

			




	
	
	
		
 
    def test_api_create_user_when_exception_happened(self):

			




	
	
	
		
 

			




	
	
	
		
 
        username = 'test_new_api_user'

			




	
	
	
		
 
        email = username + "@example.com"

			




	
	
	
		
 

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'create_user',

			




	
	
		
 
@@ -470,13 +467,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 

			




	
	
	
		
 
        ret = {'msg': 'deleted user ID:%s %s' % (usr_id, username),

			




	
	
	
		
 
               'user': None}

			




	
	
	
		
 
        expected = ret

			




	
	
	
		
 
        self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'delete', crash)

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'delete', raise_exception)

			




	
	
	
		
 
    def test_api_delete_user_when_exception_happened(self):

			




	
	
	
		
 
        usr = UserModel().create_or_update(username='test_user',

			




	
	
	
		
 
                                           password='qweqwe',

			




	
	
	
		
 
                                           email='u232@example.com',

			




	
	
	
		
 
                                           firstname='u1', lastname='u1')

			




	
	
	
		
 
        Session().commit()

			




	
	
		
 
@@ -558,13 +555,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  userid=usr.user_id)

			




	
	
	
		
 

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 
        expected = 'editing default user is forbidden'

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'update_user', crash)

			




	
	
	
		
 
    @mock.patch.object(UserModel, 'update_user', raise_exception)

			




	
	
	
		
 
    def test_api_update_user_when_exception_happens(self):

			




	
	
	
		
 
        usr = User.get_by_username(base.TEST_USER_ADMIN_LOGIN)

			




	
	
	
		
 
        ret = jsonify(usr.get_api_data())

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'update_user',

			




	
	
	
		
 
                                  userid=usr.user_id)

			




	
	
	
		
 

			




	
	
		
 
@@ -1017,13 +1014,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  repo_type=self.REPO_TYPE,)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 
        expected = 'repo group `%s` not found' % group_name

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 
        fixture.destroy_repo(repo_name)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'create', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'create', raise_exception)

			




	
	
	
		
 
    def test_api_create_repo_exception_occurred(self):

			




	
	
	
		
 
        repo_name = 'api-repo'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'create_repo',

			




	
	
	
		
 
                                  repo_name=repo_name,

			




	
	
	
		
 
                                  owner=base.TEST_USER_ADMIN_LOGIN,

			




	
	
	
		
 
                                  repo_type=self.REPO_TYPE,)

			




	
	
		
 
@@ -1137,13 +1134,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            expected = 'repository `%s` does not exist' % repo_name

			




	
	
	
		
 
            self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 
        finally:

			




	
	
	
		
 
            fixture.destroy_repo(repo_name)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'update', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'update', raise_exception)

			




	
	
	
		
 
    def test_api_update_repo_exception_occurred(self):

			




	
	
	
		
 
        repo_name = 'api_update_me'

			




	
	
	
		
 
        fixture.create_repo(repo_name, repo_type=self.REPO_TYPE)

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'update_repo',

			




	
	
	
		
 
                                  repoid=repo_name, owner=base.TEST_USER_ADMIN_LOGIN,)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
		
 
@@ -1261,13 +1258,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
            fixture.destroy_repo(repo_name)

			




	
	
	
		
 

			




	
	
	
		
 
    def test_api_delete_repo_exception_occurred(self):

			




	
	
	
		
 
        repo_name = 'api_delete_me'

			




	
	
	
		
 
        fixture.create_repo(repo_name, repo_type=self.REPO_TYPE)

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            with mock.patch.object(RepoModel, 'delete', crash):

			




	
	
	
		
 
            with mock.patch.object(RepoModel, 'delete', raise_exception):

			




	
	
	
		
 
                id_, params = _build_data(self.apikey, 'delete_repo',

			




	
	
	
		
 
                                          repoid=repo_name, )

			




	
	
	
		
 
                response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
                expected = 'failed to delete repository `%s`' % repo_name

			




	
	
	
		
 
                self._compare_error(id_, expected, given=response.body)

			




	
	
		
 
@@ -1409,13 +1406,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        )

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = "repo `%s` already exist" % fork_name

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'create_fork', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'create_fork', raise_exception)

			




	
	
	
		
 
    def test_api_fork_repo_exception_occurred(self):

			




	
	
	
		
 
        fork_name = 'api-repo-fork'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'fork_repo',

			




	
	
	
		
 
                                  repoid=self.REPO,

			




	
	
	
		
 
                                  fork_name=fork_name,

			




	
	
	
		
 
                                  owner=base.TEST_USER_ADMIN_LOGIN,

			




	
	
		
 
@@ -1481,13 +1478,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  group_name=TEST_USER_GROUP)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = "user group `%s` already exist" % TEST_USER_GROUP

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'create', crash)

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'create', raise_exception)

			




	
	
	
		
 
    def test_api_get_user_group_exception_occurred(self):

			




	
	
	
		
 
        group_name = 'exception_happens'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'create_user_group',

			




	
	
	
		
 
                                  group_name=group_name)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
		
 
@@ -1517,13 +1514,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        finally:

			




	
	
	
		
 
            if changing_attr == 'group_name':

			




	
	
	
		
 
                # switch to updated name for proper cleanup

			




	
	
	
		
 
                gr_name = updates['group_name']

			




	
	
	
		
 
            fixture.destroy_user_group(gr_name)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'update', crash)

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'update', raise_exception)

			




	
	
	
		
 
    def test_api_update_user_group_exception_occurred(self):

			




	
	
	
		
 
        gr_name = 'test_group'

			




	
	
	
		
 
        fixture.create_user_group(gr_name)

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            id_, params = _build_data(self.apikey, 'update_user_group',

			




	
	
	
		
 
                                      usergroupid=gr_name)

			




	
	
		
 
@@ -1556,13 +1553,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  userid=base.TEST_USER_ADMIN_LOGIN)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'user group `%s` does not exist' % 'false-group'

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'add_user_to_group', crash)

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'add_user_to_group', raise_exception)

			




	
	
	
		
 
    def test_api_add_user_to_user_group_exception_occurred(self):

			




	
	
	
		
 
        gr_name = 'test_group'

			




	
	
	
		
 
        fixture.create_user_group(gr_name)

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            id_, params = _build_data(self.apikey, 'add_user_to_user_group',

			




	
	
	
		
 
                                      usergroupid=gr_name,

			




	
	
		
 
@@ -1589,13 +1586,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                ),

			




	
	
	
		
 
                'success': True}

			




	
	
	
		
 
            self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 
        finally:

			




	
	
	
		
 
            fixture.destroy_user_group(gr_name)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'remove_user_from_group', crash)

			




	
	
	
		
 
    @mock.patch.object(UserGroupModel, 'remove_user_from_group', raise_exception)

			




	
	
	
		
 
    def test_api_remove_user_from_user_group_exception_occurred(self):

			




	
	
	
		
 
        gr_name = 'test_group_3'

			




	
	
	
		
 
        gr = fixture.create_user_group(gr_name)

			




	
	
	
		
 
        UserGroupModel().add_user_to_group(gr, user=base.TEST_USER_ADMIN_LOGIN)

			




	
	
	
		
 
        Session().commit()

			




	
	
	
		
 
        try:

			




	
	
		
 
@@ -1648,13 +1645,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        ugroup = fixture.create_user_group(gr_name)

			




	
	
	
		
 
        gr_id = ugroup.users_group_id

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'delete_user_group',

			




	
	
	
		
 
                                  usergroupid=gr_name)

			




	
	
	
		
 

			




	
	
	
		
 
        try:

			




	
	
	
		
 
            with mock.patch.object(UserGroupModel, 'delete', crash):

			




	
	
	
		
 
            with mock.patch.object(UserGroupModel, 'delete', raise_exception):

			




	
	
	
		
 
                response = api_call(self, params)

			




	
	
	
		
 
                expected = 'failed to delete user group ID:%s %s' % (gr_id, gr_name)

			




	
	
	
		
 
                self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 
        finally:

			




	
	
	
		
 
            fixture.destroy_user_group(gr_name)

			




	
	
	
		
 

			




	
	
		
 
@@ -1690,13 +1687,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  perm=perm)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'permission `%s` does not exist' % perm

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'grant_user_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'grant_user_permission', raise_exception)

			




	
	
	
		
 
    def test_api_grant_user_permission_exception_when_adding(self):

			




	
	
	
		
 
        perm = 'repository.read'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'grant_user_permission',

			




	
	
	
		
 
                                  repoid=self.REPO,

			




	
	
	
		
 
                                  userid=base.TEST_USER_ADMIN_LOGIN,

			




	
	
		
 
@@ -1720,13 +1717,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                base.TEST_USER_ADMIN_LOGIN, self.REPO

			




	
	
	
		
 
            ),

			




	
	
	
		
 
            'success': True

			




	
	
	
		
 
        }

			




	
	
	
		
 
        self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'revoke_user_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'revoke_user_permission', raise_exception)

			




	
	
	
		
 
    def test_api_revoke_user_permission_exception_when_adding(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'revoke_user_permission',

			




	
	
	
		
 
                                  repoid=self.REPO,

			




	
	
	
		
 
                                  userid=base.TEST_USER_ADMIN_LOGIN, )

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
		
 
@@ -1768,13 +1765,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  perm=perm)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'permission `%s` does not exist' % perm

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'grant_user_group_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'grant_user_group_permission', raise_exception)

			




	
	
	
		
 
    def test_api_grant_user_group_permission_exception_when_adding(self):

			




	
	
	
		
 
        perm = 'repository.read'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'grant_user_group_permission',

			




	
	
	
		
 
                                  repoid=self.REPO,

			




	
	
	
		
 
                                  usergroupid=TEST_USER_GROUP,

			




	
	
		
 
@@ -1802,13 +1799,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                TEST_USER_GROUP, self.REPO

			




	
	
	
		
 
            ),

			




	
	
	
		
 
            'success': True

			




	
	
	
		
 
        }

			




	
	
	
		
 
        self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'revoke_user_group_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, 'revoke_user_group_permission', raise_exception)

			




	
	
	
		
 
    def test_api_revoke_user_group_permission_exception_when_adding(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'revoke_user_group_permission',

			




	
	
	
		
 
                                  repoid=self.REPO,

			




	
	
	
		
 
                                  usergroupid=TEST_USER_GROUP, )

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
		
 
@@ -1904,13 +1901,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  perm=perm)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'permission `%s` does not exist' % perm

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'grant_user_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'grant_user_permission', raise_exception)

			




	
	
	
		
 
    def test_api_grant_user_permission_to_repo_group_exception_when_adding(self):

			




	
	
	
		
 
        perm = 'group.read'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'grant_user_permission_to_repo_group',

			




	
	
	
		
 
                                  repogroupid=TEST_REPO_GROUP,

			




	
	
	
		
 
                                  userid=base.TEST_USER_ADMIN_LOGIN,

			




	
	
		
 
@@ -1989,13 +1986,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
            }

			




	
	
	
		
 
            self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 
        else:

			




	
	
	
		
 
            expected = 'repository group `%s` does not exist' % TEST_REPO_GROUP

			




	
	
	
		
 
            self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'revoke_user_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'revoke_user_permission', raise_exception)

			




	
	
	
		
 
    def test_api_revoke_user_permission_from_repo_group_exception_when_adding(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'revoke_user_permission_from_repo_group',

			




	
	
	
		
 
                                  repogroupid=TEST_REPO_GROUP,

			




	
	
	
		
 
                                  userid=base.TEST_USER_ADMIN_LOGIN, )

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
		
 
@@ -2093,13 +2090,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                                  perm=perm)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
	
		
 
        expected = 'permission `%s` does not exist' % perm

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'grant_user_group_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'grant_user_group_permission', raise_exception)

			




	
	
	
		
 
    def test_api_grant_user_group_permission_exception_when_adding_to_repo_group(self):

			




	
	
	
		
 
        perm = 'group.read'

			




	
	
	
		
 
        id_, params = _build_data(self.apikey,

			




	
	
	
		
 
                                  'grant_user_group_permission_to_repo_group',

			




	
	
	
		
 
                                  repogroupid=TEST_REPO_GROUP,

			




	
	
	
		
 
                                  usergroupid=TEST_USER_GROUP,

			




	
	
		
 
@@ -2177,13 +2174,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
            }

			




	
	
	
		
 
            self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 
        else:

			




	
	
	
		
 
            expected = 'repository group `%s` does not exist' % TEST_REPO_GROUP

			




	
	
	
		
 
            self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'revoke_user_group_permission', crash)

			




	
	
	
		
 
    @mock.patch.object(RepoGroupModel, 'revoke_user_group_permission', raise_exception)

			




	
	
	
		
 
    def test_api_revoke_user_group_permission_from_repo_group_exception_when_adding(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'revoke_user_group_permission_from_repo_group',

			




	
	
	
		
 
                                  repogroupid=TEST_REPO_GROUP,

			




	
	
	
		
 
                                  usergroupid=TEST_USER_GROUP,)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 

			




	
	
		
 
@@ -2298,13 +2295,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
                'url': response.json['result']['gist']['url']

			




	
	
	
		
 
            },

			




	
	
	
		
 
            'msg': 'created new gist'

			




	
	
	
		
 
        }

			




	
	
	
		
 
        self._compare_ok(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(GistModel, 'create', crash)

			




	
	
	
		
 
    @mock.patch.object(GistModel, 'create', raise_exception)

			




	
	
	
		
 
    def test_api_create_gist_exception_occurred(self):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey_regular, 'create_gist',

			




	
	
	
		
 
                                  files={})

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 
        expected = 'failed to create gist'

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
		
 
@@ -2330,13 +2327,13 @@ class _BaseTestApi(object):

			




	
	
	
		
 
        id_, params = _build_data(self.apikey_regular, 'delete_gist',

			




	
	
	
		
 
                                  gistid=gist_id)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 
        expected = 'gist `%s` does not exist' % (gist_id,)

			




	
	
	
		
 
        self._compare_error(id_, expected, given=response.body)

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(GistModel, 'delete', crash)

			




	
	
	
		
 
    @mock.patch.object(GistModel, 'delete', raise_exception)

			




	
	
	
		
 
    def test_api_delete_gist_exception_occurred(self):

			




	
	
	
		
 
        gist_id = fixture.create_gist().gist_access_id

			




	
	
	
		
 
        id_, params = _build_data(self.apikey, 'delete_gist',

			




	
	
	
		
 
                                  gistid=gist_id)

			




	
	
	
		
 
        response = api_call(self, params)

			




	
	
	
		
 
        expected = 'failed to delete gist ID:%s' % (gist_id,)

			




        

    


    
    

    

        

                

                    kallithea/tests/fixture.py
                

                

                  
                      
                        

                      ➞

                      
                        
                  

                  
                      
                  
                      
                  
                      
                  
                      
                  
                  
                

                

                    Show inline comments
                    
                

        

        

            



	
	
		
 
@@ -47,14 +47,14 @@ from kallithea.tests.base import (GIT_RE

			




	
	
	
		
 

			




	
	
	
		
 
log = logging.getLogger(__name__)

			




	
	
	
		
 

			




	
	
	
		
 
FIXTURES = os.path.join(dirname(dirname(os.path.abspath(__file__))), 'tests', 'fixtures')

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
def error_function(*args, **kwargs):

			




	
	
	
		
 
    raise Exception('Total Crash !')

			




	
	
	
		
 
def raise_exception(*args, **kwargs):

			




	
	
	
		
 
    raise Exception('raise_exception raised exception')

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
class Fixture(object):

			




	
	
	
		
 

			




	
	
	
		
 
    def __init__(self):

			




	
	
	
		
 
        pass

			




        

    


    
    

    

        

                

                    kallithea/tests/functional/test_admin_repos.py
                

                

                  
                      
                        

                      ➞

                      
                        
                  

                  
                      
                  
                      
                  
                      
                  
                      
                  
                  
                

                

                    Show inline comments
                    
                

        

        

            



	
	
		
 
@@ -11,13 +11,13 @@ from kallithea.model import db

			




	
	
	
		
 
from kallithea.model.db import Permission, Repository, Ui, User, UserRepoToPerm

			




	
	
	
		
 
from kallithea.model.meta import Session

			




	
	
	
		
 
from kallithea.model.repo import RepoModel

			




	
	
	
		
 
from kallithea.model.repo_group import RepoGroupModel

			




	
	
	
		
 
from kallithea.model.user import UserModel

			




	
	
	
		
 
from kallithea.tests import base

			




	
	
	
		
 
from kallithea.tests.fixture import Fixture, error_function

			




	
	
	
		
 
from kallithea.tests.fixture import Fixture, raise_exception

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
fixture = Fixture()

			




	
	
	
		
 

			




	
	
	
		
 

			




	
	
	
		
 
def _get_permission_for_user(user, repo):

			




	
	
		
 
@@ -589,13 +589,13 @@ class _BaseTestCase(base.TestController)

			




	
	
	
		
 

			




	
	
	
		
 
        response.mustcontain('<span class="error-message">Invalid value</span>')

			




	
	
	
		
 

			




	
	
	
		
 
        RepoModel().delete(repo_name)

			




	
	
	
		
 
        Session().commit()

			




	
	
	
		
 

			




	
	
	
		
 
    @mock.patch.object(RepoModel, '_create_filesystem_repo', error_function)

			




	
	
	
		
 
    @mock.patch.object(RepoModel, '_create_filesystem_repo', raise_exception)

			




	
	
	
		
 
    def test_create_repo_when_filesystem_op_fails(self):

			




	
	
	
		
 
        self.log_user()

			




	
	
	
		
 
        repo_name = self.NEW_REPO

			




	
	
	
		
 
        description = 'description for newly created repo'

			




	
	
	
		
 

			




	
	
	
		
 
        response = self.app.post(base.url('repos'),

			




        

    



    


    



    



      

      





    
    0 comments (0 inline, 0 general)
    





    


  

  







  


    




    








    
        Server instance: clio-30092
    
    
        This site is powered by
            Kallithea 0.7.0,
        which is
        © 2010–2025 by various authors & licensed under GPLv3.
            – Support