Mercurial > kallithea
annotate rhodecode/tests/api/api_base.py @ 3457:08e8115585bd beta
calling lock function without lock attribute, will return lock state
| author | Marcin Kuzminski <marcin@python-works.com> |
|---|---|
| date | Tue, 05 Mar 2013 20:03:33 +0100 |
| parents | fa6ba6727475 |
| children | 7cde75eac0fe |
| rev | line source |
|---|---|
|
2529
40b3a54391f9
Added functional test create repo with a group
Marcin Kuzminski <marcin@python-works.com>
parents:
2526
diff
changeset
|
1 from __future__ import with_statement |
| 2526 | 2 import random |
| 3 import mock | |
| 4 | |
| 5 from rhodecode.tests import * | |
| 6 from rhodecode.lib.compat import json | |
| 7 from rhodecode.lib.auth import AuthUser | |
| 8 from rhodecode.model.user import UserModel | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
9 from rhodecode.model.users_group import UserGroupModel |
| 2526 | 10 from rhodecode.model.repo import RepoModel |
|
2529
40b3a54391f9
Added functional test create repo with a group
Marcin Kuzminski <marcin@python-works.com>
parents:
2526
diff
changeset
|
11 from rhodecode.model.meta import Session |
|
2697
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
12 from rhodecode.model.scm import ScmModel |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
13 from rhodecode.model.db import Repository |
| 2526 | 14 |
| 15 API_URL = '/_admin/api' | |
| 16 | |
| 17 | |
| 18 def _build_data(apikey, method, **kw): | |
| 19 """ | |
| 20 Builds API data with given random ID | |
| 21 | |
| 22 :param random_id: | |
| 23 :type random_id: | |
| 24 """ | |
| 25 random_id = random.randrange(1, 9999) | |
| 26 return random_id, json.dumps({ | |
| 27 "id": random_id, | |
| 28 "api_key": apikey, | |
| 29 "method": method, | |
| 30 "args": kw | |
| 31 }) | |
| 32 | |
| 33 jsonify = lambda obj: json.loads(json.dumps(obj)) | |
| 34 | |
| 35 | |
| 36 def crash(*args, **kwargs): | |
| 37 raise Exception('Total Crash !') | |
| 38 | |
| 39 | |
| 2738 | 40 def api_call(test_obj, params): |
| 41 response = test_obj.app.post(API_URL, content_type='application/json', | |
| 42 params=params) | |
| 43 return response | |
| 44 | |
| 45 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
46 TEST_USER_GROUP = 'test_users_group' |
| 2526 | 47 |
| 48 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
49 def make_users_group(name=TEST_USER_GROUP): |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
50 gr = UserGroupModel().create(name=name) |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
51 UserGroupModel().add_user_to_group(users_group=gr, |
| 2526 | 52 user=TEST_USER_ADMIN_LOGIN) |
| 53 Session().commit() | |
| 54 return gr | |
| 55 | |
| 56 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
57 def destroy_users_group(name=TEST_USER_GROUP): |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
58 UserGroupModel().delete(users_group=name, force=True) |
| 2526 | 59 Session().commit() |
| 60 | |
| 61 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
62 def create_repo(repo_name, repo_type, owner=None): |
| 2526 | 63 # create new repo |
|
3056
6104dfd35b16
Implemented #379 defaults settings page for creation of repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2738
diff
changeset
|
64 form_data = _get_repo_create_params( |
|
6104dfd35b16
Implemented #379 defaults settings page for creation of repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2738
diff
changeset
|
65 repo_name_full=repo_name, |
|
6104dfd35b16
Implemented #379 defaults settings page for creation of repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2738
diff
changeset
|
66 repo_description='description %s' % repo_name, |
|
6104dfd35b16
Implemented #379 defaults settings page for creation of repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2738
diff
changeset
|
67 ) |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
68 cur_user = UserModel().get_by_username(owner or TEST_USER_ADMIN_LOGIN) |
| 2526 | 69 r = RepoModel().create(form_data, cur_user) |
| 70 Session().commit() | |
| 71 return r | |
| 72 | |
| 73 | |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
74 def create_fork(fork_name, fork_type, fork_of): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
75 fork = RepoModel(Session())._get_repo(fork_of) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
76 r = create_repo(fork_name, fork_type) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
77 r.fork = fork |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
78 Session().add(r) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
79 Session().commit() |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
80 return r |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
81 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
82 |
| 2526 | 83 def destroy_repo(repo_name): |
| 84 RepoModel().delete(repo_name) | |
| 85 Session().commit() | |
| 86 | |
| 87 | |
| 88 class BaseTestApi(object): | |
| 89 REPO = None | |
| 90 REPO_TYPE = None | |
| 91 | |
| 92 @classmethod | |
| 93 def setUpClass(self): | |
| 94 self.usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) | |
| 95 self.apikey = self.usr.api_key | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
96 self.test_user = UserModel().create_or_update( |
| 2526 | 97 username='test-api', |
| 98 password='test', | |
| 99 email='test@api.rhodecode.org', | |
| 100 firstname='first', | |
| 101 lastname='last' | |
| 102 ) | |
| 103 Session().commit() | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
104 self.TEST_USER_LOGIN = self.test_user.username |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
105 self.apikey_regular = self.test_user.api_key |
| 2526 | 106 |
| 107 @classmethod | |
| 108 def teardownClass(self): | |
| 109 pass | |
| 110 | |
| 111 def setUp(self): | |
| 112 self.maxDiff = None | |
| 113 make_users_group() | |
| 114 | |
| 115 def tearDown(self): | |
| 116 destroy_users_group() | |
| 117 | |
| 118 def _compare_ok(self, id_, expected, given): | |
| 119 expected = jsonify({ | |
| 120 'id': id_, | |
| 121 'error': None, | |
| 122 'result': expected | |
| 123 }) | |
| 124 given = json.loads(given) | |
| 125 self.assertEqual(expected, given) | |
| 126 | |
| 127 def _compare_error(self, id_, expected, given): | |
| 128 expected = jsonify({ | |
| 129 'id': id_, | |
| 130 'error': expected, | |
| 131 'result': None | |
| 132 }) | |
| 133 given = json.loads(given) | |
| 134 self.assertEqual(expected, given) | |
| 135 | |
| 136 # def test_Optional(self): | |
| 137 # from rhodecode.controllers.api.api import Optional | |
| 138 # option1 = Optional(None) | |
| 139 # self.assertEqual('<Optional:%s>' % None, repr(option1)) | |
| 140 # | |
| 141 # self.assertEqual(1, Optional.extract(Optional(1))) | |
| 142 # self.assertEqual('trololo', Optional.extract('trololo')) | |
| 143 | |
| 144 def test_api_wrong_key(self): | |
| 145 id_, params = _build_data('trololo', 'get_user') | |
| 2738 | 146 response = api_call(self, params) |
| 2526 | 147 |
| 148 expected = 'Invalid API KEY' | |
| 149 self._compare_error(id_, expected, given=response.body) | |
| 150 | |
| 151 def test_api_missing_non_optional_param(self): | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
152 id_, params = _build_data(self.apikey, 'get_repo') |
| 2738 | 153 response = api_call(self, params) |
| 2526 | 154 |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
155 expected = 'Missing non optional `repoid` arg in JSON DATA' |
| 2526 | 156 self._compare_error(id_, expected, given=response.body) |
| 157 | |
|
3165
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
158 def test_api_missing_non_optional_param_args_null(self): |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
159 id_, params = _build_data(self.apikey, 'get_repo') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
160 params = params.replace('"args": {}', '"args": null') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
161 response = api_call(self, params) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
162 |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
163 expected = 'Missing non optional `repoid` arg in JSON DATA' |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
164 self._compare_error(id_, expected, given=response.body) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
165 |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
166 def test_api_missing_non_optional_param_args_bad(self): |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
167 id_, params = _build_data(self.apikey, 'get_repo') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
168 params = params.replace('"args": {}', '"args": 1') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
169 response = api_call(self, params) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
170 |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
171 expected = 'Missing non optional `repoid` arg in JSON DATA' |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
172 self._compare_error(id_, expected, given=response.body) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
173 |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
174 def test_api_args_is_null(self): |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
175 id_, params = _build_data(self.apikey, 'get_users',) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
176 params = params.replace('"args": {}', '"args": null') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
177 response = api_call(self, params) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
178 self.assertEqual(response.status, '200 OK') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
179 |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
180 def test_api_args_is_bad(self): |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
181 id_, params = _build_data(self.apikey, 'get_users',) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
182 params = params.replace('"args": {}', '"args": 1') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
183 response = api_call(self, params) |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
184 self.assertEqual(response.status, '200 OK') |
|
e1baadec6217
fixes issue #702 API methods without arguments fail when "args":null
Marcin Kuzminski <marcin@python-works.com>
parents:
3163
diff
changeset
|
185 |
| 2526 | 186 def test_api_get_users(self): |
| 187 id_, params = _build_data(self.apikey, 'get_users',) | |
| 2738 | 188 response = api_call(self, params) |
| 2526 | 189 ret_all = [] |
| 190 for usr in UserModel().get_all(): | |
| 191 ret = usr.get_api_data() | |
| 192 ret_all.append(jsonify(ret)) | |
| 193 expected = ret_all | |
| 194 self._compare_ok(id_, expected, given=response.body) | |
| 195 | |
| 196 def test_api_get_user(self): | |
| 197 id_, params = _build_data(self.apikey, 'get_user', | |
| 198 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 199 response = api_call(self, params) |
| 2526 | 200 |
| 201 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) | |
| 202 ret = usr.get_api_data() | |
| 203 ret['permissions'] = AuthUser(usr.user_id).permissions | |
| 204 | |
| 205 expected = ret | |
| 206 self._compare_ok(id_, expected, given=response.body) | |
| 207 | |
| 208 def test_api_get_user_that_does_not_exist(self): | |
| 209 id_, params = _build_data(self.apikey, 'get_user', | |
| 210 userid='trololo') | |
| 2738 | 211 response = api_call(self, params) |
| 2526 | 212 |
| 213 expected = "user `%s` does not exist" % 'trololo' | |
| 214 self._compare_error(id_, expected, given=response.body) | |
| 215 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
216 def test_api_get_user_without_giving_userid(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
217 id_, params = _build_data(self.apikey, 'get_user') |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
218 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
219 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
220 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
221 ret = usr.get_api_data() |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
222 ret['permissions'] = AuthUser(usr.user_id).permissions |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
223 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
224 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
225 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
226 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
227 def test_api_get_user_without_giving_userid_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
228 id_, params = _build_data(self.apikey_regular, 'get_user') |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
229 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
230 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
231 usr = UserModel().get_by_username(self.TEST_USER_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
232 ret = usr.get_api_data() |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
233 ret['permissions'] = AuthUser(usr.user_id).permissions |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
234 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
235 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
236 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
237 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
238 def test_api_get_user_with_giving_userid_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
239 id_, params = _build_data(self.apikey_regular, 'get_user', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
240 userid=self.TEST_USER_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
241 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
242 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
243 expected = 'userid is not the same as your user' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
244 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
245 |
| 2526 | 246 def test_api_pull(self): |
| 247 #TODO: issues with rhodecode_extras here.. not sure why ! | |
| 248 pass | |
| 249 | |
| 250 # repo_name = 'test_pull' | |
| 251 # r = create_repo(repo_name, self.REPO_TYPE) | |
| 252 # r.clone_uri = TEST_self.REPO | |
| 253 # Session.add(r) | |
| 254 # Session.commit() | |
| 255 # | |
| 256 # id_, params = _build_data(self.apikey, 'pull', | |
| 257 # repoid=repo_name,) | |
| 258 # response = self.app.post(API_URL, content_type='application/json', | |
| 259 # params=params) | |
| 260 # | |
| 261 # expected = 'Pulled from `%s`' % repo_name | |
| 262 # self._compare_ok(id_, expected, given=response.body) | |
| 263 # | |
| 264 # destroy_repo(repo_name) | |
| 265 | |
| 266 def test_api_pull_error(self): | |
| 267 id_, params = _build_data(self.apikey, 'pull', | |
| 268 repoid=self.REPO,) | |
| 2738 | 269 response = api_call(self, params) |
| 2526 | 270 |
| 271 expected = 'Unable to pull changes from `%s`' % self.REPO | |
| 272 self._compare_error(id_, expected, given=response.body) | |
| 273 | |
|
2697
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
274 def test_api_rescan_repos(self): |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
275 id_, params = _build_data(self.apikey, 'rescan_repos') |
| 2738 | 276 response = api_call(self, params) |
|
2697
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
277 |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
278 expected = {'added': [], 'removed': []} |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
279 self._compare_ok(id_, expected, given=response.body) |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
280 |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
281 @mock.patch.object(ScmModel, 'repo_scan', crash) |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
282 def test_api_rescann_error(self): |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
283 id_, params = _build_data(self.apikey, 'rescan_repos',) |
| 2738 | 284 response = api_call(self, params) |
|
2697
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
285 |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
286 expected = 'Error occurred during rescan repositories action' |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
287 self._compare_error(id_, expected, given=response.body) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
288 |
|
3235
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
289 def test_api_invalidate_cache(self): |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
290 id_, params = _build_data(self.apikey, 'invalidate_cache', |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
291 repoid=self.REPO) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
292 response = api_call(self, params) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
293 |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
294 expected = ("Cache for repository `%s` was invalidated: " |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
295 "invalidated cache keys: %s" % (self.REPO, |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
296 [unicode(self.REPO)])) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
297 self._compare_ok(id_, expected, given=response.body) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
298 |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
299 @mock.patch.object(ScmModel, 'mark_for_invalidation', crash) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
300 def test_api_invalidate_cache_error(self): |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
301 id_, params = _build_data(self.apikey, 'invalidate_cache', |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
302 repoid=self.REPO) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
303 response = api_call(self, params) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
304 |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
305 expected = 'Error occurred during cache invalidation action' |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
306 self._compare_error(id_, expected, given=response.body) |
|
d6029dacbcc4
API invalidate_cache function ref #733
Marcin Kuzminski <marcin@python-works.com>
parents:
3216
diff
changeset
|
307 |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
308 def test_api_lock_repo_lock_aquire(self): |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
309 id_, params = _build_data(self.apikey, 'lock', |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
310 userid=TEST_USER_ADMIN_LOGIN, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
311 repoid=self.REPO, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
312 locked=True) |
| 2738 | 313 response = api_call(self, params) |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
314 expected = ('User `%s` set lock state for repo `%s` to `%s`' |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
315 % (TEST_USER_ADMIN_LOGIN, self.REPO, True)) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
316 self._compare_ok(id_, expected, given=response.body) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
317 |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
318 def test_api_lock_repo_lock_aquire_by_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
319 repo_name = 'api_delete_me' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
320 create_repo(repo_name, self.REPO_TYPE, owner=self.TEST_USER_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
321 try: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
322 id_, params = _build_data(self.apikey_regular, 'lock', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
323 repoid=repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
324 locked=True) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
325 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
326 expected = ('User `%s` set lock state for repo `%s` to `%s`' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
327 % (self.TEST_USER_LOGIN, repo_name, True)) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
328 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
329 finally: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
330 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
331 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
332 def test_api_lock_repo_lock_aquire_non_admin_with_userid(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
333 repo_name = 'api_delete_me' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
334 create_repo(repo_name, self.REPO_TYPE, owner=self.TEST_USER_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
335 try: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
336 id_, params = _build_data(self.apikey_regular, 'lock', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
337 userid=TEST_USER_ADMIN_LOGIN, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
338 repoid=repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
339 locked=True) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
340 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
341 expected = 'userid is not the same as your user' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
342 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
343 finally: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
344 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
345 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
346 def test_api_lock_repo_lock_aquire_non_admin_not_his_repo(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
347 id_, params = _build_data(self.apikey_regular, 'lock', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
348 repoid=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
349 locked=True) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
350 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
351 expected = 'repository `%s` does not exist' % (self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
352 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
353 |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
354 def test_api_lock_repo_lock_release(self): |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
355 id_, params = _build_data(self.apikey, 'lock', |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
356 userid=TEST_USER_ADMIN_LOGIN, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
357 repoid=self.REPO, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
358 locked=False) |
| 2738 | 359 response = api_call(self, params) |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
360 expected = ('User `%s` set lock state for repo `%s` to `%s`' |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
361 % (TEST_USER_ADMIN_LOGIN, self.REPO, False)) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
362 self._compare_ok(id_, expected, given=response.body) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
363 |
|
3161
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
364 def test_api_lock_repo_lock_aquire_optional_userid(self): |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
365 id_, params = _build_data(self.apikey, 'lock', |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
366 repoid=self.REPO, |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
367 locked=True) |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
368 response = api_call(self, params) |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
369 expected = ('User `%s` set lock state for repo `%s` to `%s`' |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
370 % (TEST_USER_ADMIN_LOGIN, self.REPO, True)) |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
371 self._compare_ok(id_, expected, given=response.body) |
|
3563c47e52fd
Implemented API calls for non-admin users for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
3056
diff
changeset
|
372 |
|
3457
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
373 def test_api_lock_repo_lock_optional_locked(self): |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
374 from rhodecode.lib import helpers |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
375 from rhodecode.lib.utils2 import time_to_datetime |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
376 _locked_since = helpers.fmt_date(time_to_datetime(Repository\ |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
377 .get_by_repo_name(self.REPO).locked[1])) |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
378 id_, params = _build_data(self.apikey, 'lock', |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
379 repoid=self.REPO) |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
380 response = api_call(self, params) |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
381 expected = ('Repo `%s` locked by `%s`. Locked=`True`. Locked since: `%s`' |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
382 % (self.REPO, TEST_USER_ADMIN_LOGIN, _locked_since)) |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
383 self._compare_ok(id_, expected, given=response.body) |
|
08e8115585bd
calling lock function without lock attribute, will return lock state
Marcin Kuzminski <marcin@python-works.com>
parents:
3417
diff
changeset
|
384 |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
385 @mock.patch.object(Repository, 'lock', crash) |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
386 def test_api_lock_error(self): |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
387 id_, params = _build_data(self.apikey, 'lock', |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
388 userid=TEST_USER_ADMIN_LOGIN, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
389 repoid=self.REPO, |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
390 locked=True) |
| 2738 | 391 response = api_call(self, params) |
|
2737
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
392 |
|
e21cb7b1a4a5
added API call for locking/unlocking repositories
Marcin Kuzminski <marcin@python-works.com>
parents:
2697
diff
changeset
|
393 expected = 'Error occurred locking repository `%s`' % self.REPO |
|
2697
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
394 self._compare_error(id_, expected, given=response.body) |
|
4565e655ea2a
API: Added option to rescann repositories via api call
Marcin Kuzminski <marcin@python-works.com>
parents:
2673
diff
changeset
|
395 |
| 2526 | 396 def test_api_create_existing_user(self): |
| 397 id_, params = _build_data(self.apikey, 'create_user', | |
| 398 username=TEST_USER_ADMIN_LOGIN, | |
| 399 email='test@foo.com', | |
| 400 password='trololo') | |
| 2738 | 401 response = api_call(self, params) |
| 2526 | 402 |
| 403 expected = "user `%s` already exist" % TEST_USER_ADMIN_LOGIN | |
| 404 self._compare_error(id_, expected, given=response.body) | |
| 405 | |
| 406 def test_api_create_user_with_existing_email(self): | |
| 407 id_, params = _build_data(self.apikey, 'create_user', | |
| 408 username=TEST_USER_ADMIN_LOGIN + 'new', | |
| 409 email=TEST_USER_REGULAR_EMAIL, | |
| 410 password='trololo') | |
| 2738 | 411 response = api_call(self, params) |
| 2526 | 412 |
| 413 expected = "email `%s` already exist" % TEST_USER_REGULAR_EMAIL | |
| 414 self._compare_error(id_, expected, given=response.body) | |
| 415 | |
| 416 def test_api_create_user(self): | |
| 417 username = 'test_new_api_user' | |
| 418 email = username + "@foo.com" | |
| 419 | |
| 420 id_, params = _build_data(self.apikey, 'create_user', | |
| 421 username=username, | |
| 422 email=email, | |
| 423 password='trololo') | |
| 2738 | 424 response = api_call(self, params) |
| 2526 | 425 |
| 426 usr = UserModel().get_by_username(username) | |
| 427 ret = dict( | |
| 428 msg='created new user `%s`' % username, | |
| 429 user=jsonify(usr.get_api_data()) | |
| 430 ) | |
| 431 | |
| 432 expected = ret | |
| 433 self._compare_ok(id_, expected, given=response.body) | |
| 434 | |
| 435 UserModel().delete(usr.user_id) | |
| 2738 | 436 Session().commit() |
| 2526 | 437 |
| 438 @mock.patch.object(UserModel, 'create_or_update', crash) | |
| 439 def test_api_create_user_when_exception_happened(self): | |
| 440 | |
| 441 username = 'test_new_api_user' | |
| 442 email = username + "@foo.com" | |
| 443 | |
| 444 id_, params = _build_data(self.apikey, 'create_user', | |
| 445 username=username, | |
| 446 email=email, | |
| 447 password='trololo') | |
| 2738 | 448 response = api_call(self, params) |
| 2526 | 449 expected = 'failed to create user `%s`' % username |
| 450 self._compare_error(id_, expected, given=response.body) | |
| 451 | |
| 452 def test_api_delete_user(self): | |
| 453 usr = UserModel().create_or_update(username=u'test_user', | |
| 454 password=u'qweqwe', | |
| 455 email=u'u232@rhodecode.org', | |
| 456 firstname=u'u1', lastname=u'u1') | |
| 2738 | 457 Session().commit() |
| 2526 | 458 username = usr.username |
| 459 email = usr.email | |
| 460 usr_id = usr.user_id | |
| 461 ## DELETE THIS USER NOW | |
| 462 | |
| 463 id_, params = _build_data(self.apikey, 'delete_user', | |
| 464 userid=username,) | |
| 2738 | 465 response = api_call(self, params) |
| 2526 | 466 |
| 467 ret = {'msg': 'deleted user ID:%s %s' % (usr_id, username), | |
| 468 'user': None} | |
| 469 expected = ret | |
| 470 self._compare_ok(id_, expected, given=response.body) | |
| 471 | |
| 472 @mock.patch.object(UserModel, 'delete', crash) | |
| 473 def test_api_delete_user_when_exception_happened(self): | |
| 474 usr = UserModel().create_or_update(username=u'test_user', | |
| 475 password=u'qweqwe', | |
| 476 email=u'u232@rhodecode.org', | |
| 477 firstname=u'u1', lastname=u'u1') | |
| 2738 | 478 Session().commit() |
| 2526 | 479 username = usr.username |
| 480 | |
| 481 id_, params = _build_data(self.apikey, 'delete_user', | |
| 482 userid=username,) | |
| 2738 | 483 response = api_call(self, params) |
| 2526 | 484 ret = 'failed to delete ID:%s %s' % (usr.user_id, |
| 485 usr.username) | |
| 486 expected = ret | |
| 487 self._compare_error(id_, expected, given=response.body) | |
| 488 | |
| 489 @parameterized.expand([('firstname', 'new_username'), | |
| 490 ('lastname', 'new_username'), | |
| 491 ('email', 'new_username'), | |
| 492 ('admin', True), | |
| 493 ('admin', False), | |
| 494 ('ldap_dn', 'test'), | |
| 495 ('ldap_dn', None), | |
| 496 ('active', False), | |
| 497 ('active', True), | |
| 498 ('password', 'newpass') | |
| 499 ]) | |
| 500 def test_api_update_user(self, name, expected): | |
| 501 usr = UserModel().get_by_username(self.TEST_USER_LOGIN) | |
| 502 kw = {name: expected, | |
| 503 'userid': usr.user_id} | |
| 504 id_, params = _build_data(self.apikey, 'update_user', **kw) | |
| 2738 | 505 response = api_call(self, params) |
| 2526 | 506 |
| 507 ret = { | |
| 508 'msg': 'updated user ID:%s %s' % (usr.user_id, self.TEST_USER_LOGIN), | |
| 509 'user': jsonify(UserModel()\ | |
| 510 .get_by_username(self.TEST_USER_LOGIN)\ | |
| 511 .get_api_data()) | |
| 512 } | |
| 513 | |
| 514 expected = ret | |
| 515 self._compare_ok(id_, expected, given=response.body) | |
| 516 | |
| 517 def test_api_update_user_no_changed_params(self): | |
| 518 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) | |
| 519 ret = jsonify(usr.get_api_data()) | |
| 520 id_, params = _build_data(self.apikey, 'update_user', | |
| 521 userid=TEST_USER_ADMIN_LOGIN) | |
| 522 | |
| 2738 | 523 response = api_call(self, params) |
| 2526 | 524 ret = { |
| 525 'msg': 'updated user ID:%s %s' % (usr.user_id, TEST_USER_ADMIN_LOGIN), | |
| 526 'user': ret | |
| 527 } | |
| 528 expected = ret | |
| 529 self._compare_ok(id_, expected, given=response.body) | |
| 530 | |
| 531 def test_api_update_user_by_user_id(self): | |
| 532 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) | |
| 533 ret = jsonify(usr.get_api_data()) | |
| 534 id_, params = _build_data(self.apikey, 'update_user', | |
| 535 userid=usr.user_id) | |
| 536 | |
| 2738 | 537 response = api_call(self, params) |
| 2526 | 538 ret = { |
| 539 'msg': 'updated user ID:%s %s' % (usr.user_id, TEST_USER_ADMIN_LOGIN), | |
| 540 'user': ret | |
| 541 } | |
| 542 expected = ret | |
| 543 self._compare_ok(id_, expected, given=response.body) | |
| 544 | |
|
2657
001c7e2ae986
fixed api issue with changing username during update_user
Marcin Kuzminski <marcin@python-works.com>
parents:
2653
diff
changeset
|
545 @mock.patch.object(UserModel, 'update_user', crash) |
| 2526 | 546 def test_api_update_user_when_exception_happens(self): |
| 547 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN) | |
| 548 ret = jsonify(usr.get_api_data()) | |
| 549 id_, params = _build_data(self.apikey, 'update_user', | |
| 550 userid=usr.user_id) | |
| 551 | |
| 2738 | 552 response = api_call(self, params) |
| 2526 | 553 ret = 'failed to update user `%s`' % usr.user_id |
| 554 | |
| 555 expected = ret | |
| 556 self._compare_error(id_, expected, given=response.body) | |
| 557 | |
| 558 def test_api_get_repo(self): | |
| 559 new_group = 'some_new_group' | |
| 560 make_users_group(new_group) | |
| 561 RepoModel().grant_users_group_permission(repo=self.REPO, | |
| 562 group_name=new_group, | |
| 563 perm='repository.read') | |
| 2738 | 564 Session().commit() |
| 2526 | 565 id_, params = _build_data(self.apikey, 'get_repo', |
| 566 repoid=self.REPO) | |
| 2738 | 567 response = api_call(self, params) |
| 2526 | 568 |
| 569 repo = RepoModel().get_by_repo_name(self.REPO) | |
| 570 ret = repo.get_api_data() | |
| 571 | |
| 572 members = [] | |
| 3215 | 573 followers = [] |
| 2526 | 574 for user in repo.repo_to_perm: |
| 575 perm = user.permission.permission_name | |
| 576 user = user.user | |
| 577 user_data = user.get_api_data() | |
| 578 user_data['type'] = "user" | |
| 579 user_data['permission'] = perm | |
| 580 members.append(user_data) | |
| 581 | |
| 582 for users_group in repo.users_group_to_perm: | |
| 583 perm = users_group.permission.permission_name | |
| 584 users_group = users_group.users_group | |
| 585 users_group_data = users_group.get_api_data() | |
| 586 users_group_data['type'] = "users_group" | |
| 587 users_group_data['permission'] = perm | |
| 588 members.append(users_group_data) | |
| 589 | |
| 3215 | 590 for user in repo.followers: |
| 591 followers.append(user.user.get_api_data()) | |
| 592 | |
| 2526 | 593 ret['members'] = members |
| 3215 | 594 ret['followers'] = followers |
| 2526 | 595 |
| 596 expected = ret | |
| 597 self._compare_ok(id_, expected, given=response.body) | |
| 598 destroy_users_group(new_group) | |
| 599 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
600 def test_api_get_repo_by_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
601 id_, params = _build_data(self.apikey, 'get_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
602 repoid=self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
603 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
604 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
605 repo = RepoModel().get_by_repo_name(self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
606 ret = repo.get_api_data() |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
607 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
608 members = [] |
| 3215 | 609 followers = [] |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
610 for user in repo.repo_to_perm: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
611 perm = user.permission.permission_name |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
612 user = user.user |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
613 user_data = user.get_api_data() |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
614 user_data['type'] = "user" |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
615 user_data['permission'] = perm |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
616 members.append(user_data) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
617 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
618 for users_group in repo.users_group_to_perm: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
619 perm = users_group.permission.permission_name |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
620 users_group = users_group.users_group |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
621 users_group_data = users_group.get_api_data() |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
622 users_group_data['type'] = "users_group" |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
623 users_group_data['permission'] = perm |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
624 members.append(users_group_data) |
|
3216
81bbb8086f03
whitespace cleanup
Marcin Kuzminski <marcin@python-works.com>
parents:
3215
diff
changeset
|
625 |
| 3215 | 626 for user in repo.followers: |
| 627 followers.append(user.user.get_api_data()) | |
|
3216
81bbb8086f03
whitespace cleanup
Marcin Kuzminski <marcin@python-works.com>
parents:
3215
diff
changeset
|
628 |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
629 ret['members'] = members |
| 3215 | 630 ret['followers'] = followers |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
631 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
632 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
633 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
634 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
635 def test_api_get_repo_by_non_admin_no_permission_to_repo(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
636 RepoModel().grant_user_permission(repo=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
637 user=self.TEST_USER_LOGIN, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
638 perm='repository.none') |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
639 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
640 id_, params = _build_data(self.apikey_regular, 'get_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
641 repoid=self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
642 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
643 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
644 expected = 'repository `%s` does not exist' % (self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
645 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
646 |
| 2526 | 647 def test_api_get_repo_that_doesn_not_exist(self): |
| 648 id_, params = _build_data(self.apikey, 'get_repo', | |
| 649 repoid='no-such-repo') | |
| 2738 | 650 response = api_call(self, params) |
| 2526 | 651 |
| 652 ret = 'repository `%s` does not exist' % 'no-such-repo' | |
| 653 expected = ret | |
| 654 self._compare_error(id_, expected, given=response.body) | |
| 655 | |
| 656 def test_api_get_repos(self): | |
| 657 id_, params = _build_data(self.apikey, 'get_repos') | |
| 2738 | 658 response = api_call(self, params) |
| 2526 | 659 |
| 660 result = [] | |
| 661 for repo in RepoModel().get_all(): | |
| 662 result.append(repo.get_api_data()) | |
| 663 ret = jsonify(result) | |
| 664 | |
| 665 expected = ret | |
| 666 self._compare_ok(id_, expected, given=response.body) | |
| 667 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
668 def test_api_get_repos_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
669 id_, params = _build_data(self.apikey_regular, 'get_repos') |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
670 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
671 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
672 result = [] |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
673 for repo in RepoModel().get_all_user_repos(self.TEST_USER_LOGIN): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
674 result.append(repo.get_api_data()) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
675 ret = jsonify(result) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
676 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
677 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
678 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
679 |
| 2526 | 680 @parameterized.expand([('all', 'all'), |
| 681 ('dirs', 'dirs'), | |
| 682 ('files', 'files'), ]) | |
| 683 def test_api_get_repo_nodes(self, name, ret_type): | |
| 684 rev = 'tip' | |
| 685 path = '/' | |
| 686 id_, params = _build_data(self.apikey, 'get_repo_nodes', | |
| 687 repoid=self.REPO, revision=rev, | |
| 688 root_path=path, | |
| 689 ret_type=ret_type) | |
| 2738 | 690 response = api_call(self, params) |
| 2526 | 691 |
| 692 # we don't the actual return types here since it's tested somewhere | |
| 693 # else | |
| 694 expected = json.loads(response.body)['result'] | |
| 695 self._compare_ok(id_, expected, given=response.body) | |
| 696 | |
| 697 def test_api_get_repo_nodes_bad_revisions(self): | |
| 698 rev = 'i-dont-exist' | |
| 699 path = '/' | |
| 700 id_, params = _build_data(self.apikey, 'get_repo_nodes', | |
| 701 repoid=self.REPO, revision=rev, | |
| 702 root_path=path,) | |
| 2738 | 703 response = api_call(self, params) |
| 2526 | 704 |
| 705 expected = 'failed to get repo: `%s` nodes' % self.REPO | |
| 706 self._compare_error(id_, expected, given=response.body) | |
| 707 | |
| 708 def test_api_get_repo_nodes_bad_path(self): | |
| 709 rev = 'tip' | |
| 710 path = '/idontexits' | |
| 711 id_, params = _build_data(self.apikey, 'get_repo_nodes', | |
| 712 repoid=self.REPO, revision=rev, | |
| 713 root_path=path,) | |
| 2738 | 714 response = api_call(self, params) |
| 2526 | 715 |
| 716 expected = 'failed to get repo: `%s` nodes' % self.REPO | |
| 717 self._compare_error(id_, expected, given=response.body) | |
| 718 | |
| 719 def test_api_get_repo_nodes_bad_ret_type(self): | |
| 720 rev = 'tip' | |
| 721 path = '/' | |
| 722 ret_type = 'error' | |
| 723 id_, params = _build_data(self.apikey, 'get_repo_nodes', | |
| 724 repoid=self.REPO, revision=rev, | |
| 725 root_path=path, | |
| 726 ret_type=ret_type) | |
| 2738 | 727 response = api_call(self, params) |
| 2526 | 728 |
| 729 expected = 'ret_type must be one of %s' % (['files', 'dirs', 'all']) | |
| 730 self._compare_error(id_, expected, given=response.body) | |
| 731 | |
| 732 def test_api_create_repo(self): | |
| 733 repo_name = 'api-repo' | |
| 734 id_, params = _build_data(self.apikey, 'create_repo', | |
| 735 repo_name=repo_name, | |
| 736 owner=TEST_USER_ADMIN_LOGIN, | |
| 737 repo_type='hg', | |
| 738 ) | |
| 2738 | 739 response = api_call(self, params) |
| 2526 | 740 |
| 741 repo = RepoModel().get_by_repo_name(repo_name) | |
| 742 ret = { | |
| 743 'msg': 'Created new repository `%s`' % repo_name, | |
| 744 'repo': jsonify(repo.get_api_data()) | |
| 745 } | |
| 746 expected = ret | |
| 747 self._compare_ok(id_, expected, given=response.body) | |
| 748 destroy_repo(repo_name) | |
| 749 | |
| 750 def test_api_create_repo_unknown_owner(self): | |
| 751 repo_name = 'api-repo' | |
| 752 owner = 'i-dont-exist' | |
| 753 id_, params = _build_data(self.apikey, 'create_repo', | |
| 754 repo_name=repo_name, | |
| 755 owner=owner, | |
| 756 repo_type='hg', | |
| 757 ) | |
| 2738 | 758 response = api_call(self, params) |
| 2526 | 759 expected = 'user `%s` does not exist' % owner |
| 760 self._compare_error(id_, expected, given=response.body) | |
| 761 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
762 def test_api_create_repo_dont_specify_owner(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
763 repo_name = 'api-repo' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
764 owner = 'i-dont-exist' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
765 id_, params = _build_data(self.apikey, 'create_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
766 repo_name=repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
767 repo_type='hg', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
768 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
769 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
770 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
771 repo = RepoModel().get_by_repo_name(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
772 ret = { |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
773 'msg': 'Created new repository `%s`' % repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
774 'repo': jsonify(repo.get_api_data()) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
775 } |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
776 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
777 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
778 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
779 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
780 def test_api_create_repo_by_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
781 repo_name = 'api-repo' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
782 owner = 'i-dont-exist' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
783 id_, params = _build_data(self.apikey_regular, 'create_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
784 repo_name=repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
785 repo_type='hg', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
786 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
787 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
788 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
789 repo = RepoModel().get_by_repo_name(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
790 ret = { |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
791 'msg': 'Created new repository `%s`' % repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
792 'repo': jsonify(repo.get_api_data()) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
793 } |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
794 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
795 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
796 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
797 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
798 def test_api_create_repo_by_non_admin_specify_owner(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
799 repo_name = 'api-repo' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
800 owner = 'i-dont-exist' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
801 id_, params = _build_data(self.apikey_regular, 'create_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
802 repo_name=repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
803 repo_type='hg', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
804 owner=owner |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
805 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
806 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
807 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
808 expected = 'Only RhodeCode admin can specify `owner` param' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
809 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
810 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
811 |
| 2526 | 812 def test_api_create_repo_exists(self): |
| 813 repo_name = self.REPO | |
| 814 id_, params = _build_data(self.apikey, 'create_repo', | |
| 815 repo_name=repo_name, | |
| 816 owner=TEST_USER_ADMIN_LOGIN, | |
| 817 repo_type='hg', | |
| 818 ) | |
| 2738 | 819 response = api_call(self, params) |
| 2526 | 820 expected = "repo `%s` already exist" % repo_name |
| 821 self._compare_error(id_, expected, given=response.body) | |
| 822 | |
| 823 @mock.patch.object(RepoModel, 'create_repo', crash) | |
| 824 def test_api_create_repo_exception_occurred(self): | |
| 825 repo_name = 'api-repo' | |
| 826 id_, params = _build_data(self.apikey, 'create_repo', | |
| 827 repo_name=repo_name, | |
| 828 owner=TEST_USER_ADMIN_LOGIN, | |
| 829 repo_type='hg', | |
| 830 ) | |
| 2738 | 831 response = api_call(self, params) |
| 2526 | 832 expected = 'failed to create repository `%s`' % repo_name |
| 833 self._compare_error(id_, expected, given=response.body) | |
| 834 | |
| 835 def test_api_delete_repo(self): | |
| 836 repo_name = 'api_delete_me' | |
| 837 create_repo(repo_name, self.REPO_TYPE) | |
| 838 | |
| 839 id_, params = _build_data(self.apikey, 'delete_repo', | |
| 840 repoid=repo_name,) | |
| 2738 | 841 response = api_call(self, params) |
| 2526 | 842 |
| 843 ret = { | |
| 844 'msg': 'Deleted repository `%s`' % repo_name, | |
| 845 'success': True | |
| 846 } | |
| 847 expected = ret | |
| 848 self._compare_ok(id_, expected, given=response.body) | |
| 849 | |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
850 def test_api_delete_repo_by_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
851 repo_name = 'api_delete_me' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
852 create_repo(repo_name, self.REPO_TYPE, owner=self.TEST_USER_LOGIN) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
853 try: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
854 id_, params = _build_data(self.apikey_regular, 'delete_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
855 repoid=repo_name,) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
856 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
857 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
858 ret = { |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
859 'msg': 'Deleted repository `%s`' % repo_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
860 'success': True |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
861 } |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
862 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
863 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
864 finally: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
865 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
866 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
867 def test_api_delete_repo_by_non_admin_no_permission(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
868 repo_name = 'api_delete_me' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
869 create_repo(repo_name, self.REPO_TYPE) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
870 try: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
871 id_, params = _build_data(self.apikey_regular, 'delete_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
872 repoid=repo_name,) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
873 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
874 expected = 'repository `%s` does not exist' % (repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
875 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
876 finally: |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
877 destroy_repo(repo_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
878 |
| 2526 | 879 def test_api_delete_repo_exception_occurred(self): |
| 880 repo_name = 'api_delete_me' | |
| 881 create_repo(repo_name, self.REPO_TYPE) | |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
882 try: |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
883 with mock.patch.object(RepoModel, 'delete', crash): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
884 id_, params = _build_data(self.apikey, 'delete_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
885 repoid=repo_name,) |
| 2738 | 886 response = api_call(self, params) |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
887 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
888 expected = 'failed to delete repository `%s`' % repo_name |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
889 self._compare_error(id_, expected, given=response.body) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
890 finally: |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
891 destroy_repo(repo_name) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
892 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
893 def test_api_fork_repo(self): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
894 fork_name = 'api-repo-fork' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
895 id_, params = _build_data(self.apikey, 'fork_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
896 repoid=self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
897 fork_name=fork_name, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
898 owner=TEST_USER_ADMIN_LOGIN, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
899 ) |
| 2738 | 900 response = api_call(self, params) |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
901 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
902 ret = { |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
903 'msg': 'Created fork of `%s` as `%s`' % (self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
904 fork_name), |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
905 'success': True |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
906 } |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
907 expected = ret |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
908 self._compare_ok(id_, expected, given=response.body) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
909 destroy_repo(fork_name) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
910 |
|
3163
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
911 def test_api_fork_repo_non_admin(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
912 fork_name = 'api-repo-fork' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
913 id_, params = _build_data(self.apikey_regular, 'fork_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
914 repoid=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
915 fork_name=fork_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
916 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
917 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
918 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
919 ret = { |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
920 'msg': 'Created fork of `%s` as `%s`' % (self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
921 fork_name), |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
922 'success': True |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
923 } |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
924 expected = ret |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
925 self._compare_ok(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
926 destroy_repo(fork_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
927 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
928 def test_api_fork_repo_non_admin_specify_owner(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
929 fork_name = 'api-repo-fork' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
930 id_, params = _build_data(self.apikey_regular, 'fork_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
931 repoid=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
932 fork_name=fork_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
933 owner=TEST_USER_ADMIN_LOGIN, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
934 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
935 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
936 expected = 'Only RhodeCode admin can specify `owner` param' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
937 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
938 destroy_repo(fork_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
939 |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
940 def test_api_fork_repo_non_admin_no_permission_to_fork(self): |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
941 RepoModel().grant_user_permission(repo=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
942 user=self.TEST_USER_LOGIN, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
943 perm='repository.none') |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
944 fork_name = 'api-repo-fork' |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
945 id_, params = _build_data(self.apikey_regular, 'fork_repo', |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
946 repoid=self.REPO, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
947 fork_name=fork_name, |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
948 ) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
949 response = api_call(self, params) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
950 expected = 'repository `%s` does not exist' % (self.REPO) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
951 self._compare_error(id_, expected, given=response.body) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
952 destroy_repo(fork_name) |
|
28571535dd61
API methods create_repo, fork_repo, delete_repo, get_repo, get_repos
Marcin Kuzminski <marcin@python-works.com>
parents:
3161
diff
changeset
|
953 |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
954 def test_api_fork_repo_unknown_owner(self): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
955 fork_name = 'api-repo-fork' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
956 owner = 'i-dont-exist' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
957 id_, params = _build_data(self.apikey, 'fork_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
958 repoid=self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
959 fork_name=fork_name, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
960 owner=owner, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
961 ) |
| 2738 | 962 response = api_call(self, params) |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
963 expected = 'user `%s` does not exist' % owner |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
964 self._compare_error(id_, expected, given=response.body) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
965 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
966 def test_api_fork_repo_fork_exists(self): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
967 fork_name = 'api-repo-fork' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
968 create_fork(fork_name, self.REPO_TYPE, self.REPO) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
969 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
970 try: |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
971 fork_name = 'api-repo-fork' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
972 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
973 id_, params = _build_data(self.apikey, 'fork_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
974 repoid=self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
975 fork_name=fork_name, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
976 owner=TEST_USER_ADMIN_LOGIN, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
977 ) |
| 2738 | 978 response = api_call(self, params) |
| 2526 | 979 |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
980 expected = "fork `%s` already exist" % fork_name |
| 2526 | 981 self._compare_error(id_, expected, given=response.body) |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
982 finally: |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
983 destroy_repo(fork_name) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
984 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
985 def test_api_fork_repo_repo_exists(self): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
986 fork_name = self.REPO |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
987 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
988 id_, params = _build_data(self.apikey, 'fork_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
989 repoid=self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
990 fork_name=fork_name, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
991 owner=TEST_USER_ADMIN_LOGIN, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
992 ) |
| 2738 | 993 response = api_call(self, params) |
| 2526 | 994 |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
995 expected = "repo `%s` already exist" % fork_name |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
996 self._compare_error(id_, expected, given=response.body) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
997 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
998 @mock.patch.object(RepoModel, 'create_fork', crash) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
999 def test_api_fork_repo_exception_occurred(self): |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1000 fork_name = 'api-repo-fork' |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1001 id_, params = _build_data(self.apikey, 'fork_repo', |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1002 repoid=self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1003 fork_name=fork_name, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1004 owner=TEST_USER_ADMIN_LOGIN, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1005 ) |
| 2738 | 1006 response = api_call(self, params) |
|
2653
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1007 |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1008 expected = 'failed to fork repository `%s` as `%s`' % (self.REPO, |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1009 fork_name) |
|
8be70a4d72c7
Implemented #354 added forking option to API
Marcin Kuzminski <marcin@python-works.com>
parents:
2529
diff
changeset
|
1010 self._compare_error(id_, expected, given=response.body) |
| 2526 | 1011 |
| 1012 def test_api_get_users_group(self): | |
| 1013 id_, params = _build_data(self.apikey, 'get_users_group', | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1014 usersgroupid=TEST_USER_GROUP) |
| 2738 | 1015 response = api_call(self, params) |
| 2526 | 1016 |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1017 users_group = UserGroupModel().get_group(TEST_USER_GROUP) |
| 2526 | 1018 members = [] |
| 1019 for user in users_group.members: | |
| 1020 user = user.user | |
| 1021 members.append(user.get_api_data()) | |
| 1022 | |
| 1023 ret = users_group.get_api_data() | |
| 1024 ret['members'] = members | |
| 1025 expected = ret | |
| 1026 self._compare_ok(id_, expected, given=response.body) | |
| 1027 | |
| 1028 def test_api_get_users_groups(self): | |
| 1029 | |
| 1030 make_users_group('test_users_group2') | |
| 1031 | |
| 1032 id_, params = _build_data(self.apikey, 'get_users_groups',) | |
| 2738 | 1033 response = api_call(self, params) |
| 2526 | 1034 |
| 1035 expected = [] | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1036 for gr_name in [TEST_USER_GROUP, 'test_users_group2']: |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1037 users_group = UserGroupModel().get_group(gr_name) |
| 2526 | 1038 ret = users_group.get_api_data() |
| 1039 expected.append(ret) | |
| 1040 self._compare_ok(id_, expected, given=response.body) | |
| 1041 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1042 UserGroupModel().delete(users_group='test_users_group2') |
| 2738 | 1043 Session().commit() |
| 2526 | 1044 |
| 1045 def test_api_create_users_group(self): | |
| 1046 group_name = 'some_new_group' | |
| 1047 id_, params = _build_data(self.apikey, 'create_users_group', | |
| 1048 group_name=group_name) | |
| 2738 | 1049 response = api_call(self, params) |
| 2526 | 1050 |
| 1051 ret = { | |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1052 'msg': 'created new user group `%s`' % group_name, |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1053 'users_group': jsonify(UserGroupModel()\ |
| 2526 | 1054 .get_by_name(group_name)\ |
| 1055 .get_api_data()) | |
| 1056 } | |
| 1057 expected = ret | |
| 1058 self._compare_ok(id_, expected, given=response.body) | |
| 1059 | |
| 1060 destroy_users_group(group_name) | |
| 1061 | |
| 1062 def test_api_get_users_group_that_exist(self): | |
| 1063 id_, params = _build_data(self.apikey, 'create_users_group', | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1064 group_name=TEST_USER_GROUP) |
| 2738 | 1065 response = api_call(self, params) |
| 2526 | 1066 |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1067 expected = "user group `%s` already exist" % TEST_USER_GROUP |
| 2526 | 1068 self._compare_error(id_, expected, given=response.body) |
| 1069 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1070 @mock.patch.object(UserGroupModel, 'create', crash) |
| 2526 | 1071 def test_api_get_users_group_exception_occurred(self): |
| 1072 group_name = 'exception_happens' | |
| 1073 id_, params = _build_data(self.apikey, 'create_users_group', | |
| 1074 group_name=group_name) | |
| 2738 | 1075 response = api_call(self, params) |
| 2526 | 1076 |
| 1077 expected = 'failed to create group `%s`' % group_name | |
| 1078 self._compare_error(id_, expected, given=response.body) | |
| 1079 | |
| 1080 def test_api_add_user_to_users_group(self): | |
| 1081 gr_name = 'test_group' | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1082 UserGroupModel().create(gr_name) |
| 2738 | 1083 Session().commit() |
| 2526 | 1084 id_, params = _build_data(self.apikey, 'add_user_to_users_group', |
| 1085 usersgroupid=gr_name, | |
| 1086 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 1087 response = api_call(self, params) |
| 2526 | 1088 |
| 1089 expected = { | |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1090 'msg': 'added member `%s` to user group `%s`' % ( |
| 2526 | 1091 TEST_USER_ADMIN_LOGIN, gr_name |
| 1092 ), | |
| 1093 'success': True} | |
| 1094 self._compare_ok(id_, expected, given=response.body) | |
| 1095 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1096 UserGroupModel().delete(users_group=gr_name) |
| 2738 | 1097 Session().commit() |
| 2526 | 1098 |
| 1099 def test_api_add_user_to_users_group_that_doesnt_exist(self): | |
| 1100 id_, params = _build_data(self.apikey, 'add_user_to_users_group', | |
| 1101 usersgroupid='false-group', | |
| 1102 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 1103 response = api_call(self, params) |
| 2526 | 1104 |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1105 expected = 'user group `%s` does not exist' % 'false-group' |
| 2526 | 1106 self._compare_error(id_, expected, given=response.body) |
| 1107 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1108 @mock.patch.object(UserGroupModel, 'add_user_to_group', crash) |
| 2526 | 1109 def test_api_add_user_to_users_group_exception_occurred(self): |
| 1110 gr_name = 'test_group' | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1111 UserGroupModel().create(gr_name) |
| 2738 | 1112 Session().commit() |
| 2526 | 1113 id_, params = _build_data(self.apikey, 'add_user_to_users_group', |
| 1114 usersgroupid=gr_name, | |
| 1115 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 1116 response = api_call(self, params) |
| 2526 | 1117 |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1118 expected = 'failed to add member to user group `%s`' % gr_name |
| 2526 | 1119 self._compare_error(id_, expected, given=response.body) |
| 1120 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1121 UserGroupModel().delete(users_group=gr_name) |
| 2738 | 1122 Session().commit() |
| 2526 | 1123 |
| 1124 def test_api_remove_user_from_users_group(self): | |
| 1125 gr_name = 'test_group_3' | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1126 gr = UserGroupModel().create(gr_name) |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1127 UserGroupModel().add_user_to_group(gr, user=TEST_USER_ADMIN_LOGIN) |
| 2738 | 1128 Session().commit() |
| 2526 | 1129 id_, params = _build_data(self.apikey, 'remove_user_from_users_group', |
| 1130 usersgroupid=gr_name, | |
| 1131 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 1132 response = api_call(self, params) |
| 2526 | 1133 |
| 1134 expected = { | |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1135 'msg': 'removed member `%s` from user group `%s`' % ( |
| 2526 | 1136 TEST_USER_ADMIN_LOGIN, gr_name |
| 1137 ), | |
| 1138 'success': True} | |
| 1139 self._compare_ok(id_, expected, given=response.body) | |
| 1140 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1141 UserGroupModel().delete(users_group=gr_name) |
| 2738 | 1142 Session().commit() |
| 2526 | 1143 |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1144 @mock.patch.object(UserGroupModel, 'remove_user_from_group', crash) |
| 2526 | 1145 def test_api_remove_user_from_users_group_exception_occurred(self): |
| 1146 gr_name = 'test_group_3' | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1147 gr = UserGroupModel().create(gr_name) |
|
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1148 UserGroupModel().add_user_to_group(gr, user=TEST_USER_ADMIN_LOGIN) |
| 2738 | 1149 Session().commit() |
| 2526 | 1150 id_, params = _build_data(self.apikey, 'remove_user_from_users_group', |
| 1151 usersgroupid=gr_name, | |
| 1152 userid=TEST_USER_ADMIN_LOGIN) | |
| 2738 | 1153 response = api_call(self, params) |
| 2526 | 1154 |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1155 expected = 'failed to remove member from user group `%s`' % gr_name |
| 2526 | 1156 self._compare_error(id_, expected, given=response.body) |
| 1157 | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1158 UserGroupModel().delete(users_group=gr_name) |
| 2738 | 1159 Session().commit() |
| 2526 | 1160 |
| 1161 @parameterized.expand([('none', 'repository.none'), | |
| 1162 ('read', 'repository.read'), | |
| 1163 ('write', 'repository.write'), | |
| 1164 ('admin', 'repository.admin')]) | |
| 1165 def test_api_grant_user_permission(self, name, perm): | |
| 1166 id_, params = _build_data(self.apikey, 'grant_user_permission', | |
| 1167 repoid=self.REPO, | |
| 1168 userid=TEST_USER_ADMIN_LOGIN, | |
| 1169 perm=perm) | |
| 2738 | 1170 response = api_call(self, params) |
| 2526 | 1171 |
| 1172 ret = { | |
| 1173 'msg': 'Granted perm: `%s` for user: `%s` in repo: `%s`' % ( | |
| 1174 perm, TEST_USER_ADMIN_LOGIN, self.REPO | |
| 1175 ), | |
| 1176 'success': True | |
| 1177 } | |
| 1178 expected = ret | |
| 1179 self._compare_ok(id_, expected, given=response.body) | |
| 1180 | |
| 1181 def test_api_grant_user_permission_wrong_permission(self): | |
| 1182 perm = 'haha.no.permission' | |
| 1183 id_, params = _build_data(self.apikey, 'grant_user_permission', | |
| 1184 repoid=self.REPO, | |
| 1185 userid=TEST_USER_ADMIN_LOGIN, | |
| 1186 perm=perm) | |
| 2738 | 1187 response = api_call(self, params) |
| 2526 | 1188 |
| 1189 expected = 'permission `%s` does not exist' % perm | |
| 1190 self._compare_error(id_, expected, given=response.body) | |
| 1191 | |
| 1192 @mock.patch.object(RepoModel, 'grant_user_permission', crash) | |
| 1193 def test_api_grant_user_permission_exception_when_adding(self): | |
| 1194 perm = 'repository.read' | |
| 1195 id_, params = _build_data(self.apikey, 'grant_user_permission', | |
| 1196 repoid=self.REPO, | |
| 1197 userid=TEST_USER_ADMIN_LOGIN, | |
| 1198 perm=perm) | |
| 2738 | 1199 response = api_call(self, params) |
| 2526 | 1200 |
| 1201 expected = 'failed to edit permission for user: `%s` in repo: `%s`' % ( | |
| 1202 TEST_USER_ADMIN_LOGIN, self.REPO | |
| 1203 ) | |
| 1204 self._compare_error(id_, expected, given=response.body) | |
| 1205 | |
| 1206 def test_api_revoke_user_permission(self): | |
| 1207 id_, params = _build_data(self.apikey, 'revoke_user_permission', | |
| 1208 repoid=self.REPO, | |
| 1209 userid=TEST_USER_ADMIN_LOGIN,) | |
| 2738 | 1210 response = api_call(self, params) |
| 2526 | 1211 |
| 1212 expected = { | |
| 1213 'msg': 'Revoked perm for user: `%s` in repo: `%s`' % ( | |
| 1214 TEST_USER_ADMIN_LOGIN, self.REPO | |
| 1215 ), | |
| 1216 'success': True | |
| 1217 } | |
| 1218 self._compare_ok(id_, expected, given=response.body) | |
| 1219 | |
| 1220 @mock.patch.object(RepoModel, 'revoke_user_permission', crash) | |
| 1221 def test_api_revoke_user_permission_exception_when_adding(self): | |
| 1222 id_, params = _build_data(self.apikey, 'revoke_user_permission', | |
| 1223 repoid=self.REPO, | |
| 1224 userid=TEST_USER_ADMIN_LOGIN,) | |
| 2738 | 1225 response = api_call(self, params) |
| 2526 | 1226 |
| 1227 expected = 'failed to edit permission for user: `%s` in repo: `%s`' % ( | |
| 1228 TEST_USER_ADMIN_LOGIN, self.REPO | |
| 1229 ) | |
| 1230 self._compare_error(id_, expected, given=response.body) | |
| 1231 | |
| 1232 @parameterized.expand([('none', 'repository.none'), | |
| 1233 ('read', 'repository.read'), | |
| 1234 ('write', 'repository.write'), | |
| 1235 ('admin', 'repository.admin')]) | |
| 1236 def test_api_grant_users_group_permission(self, name, perm): | |
| 1237 id_, params = _build_data(self.apikey, 'grant_users_group_permission', | |
| 1238 repoid=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1239 usersgroupid=TEST_USER_GROUP, |
| 2526 | 1240 perm=perm) |
| 2738 | 1241 response = api_call(self, params) |
| 2526 | 1242 |
| 1243 ret = { | |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1244 'msg': 'Granted perm: `%s` for user group: `%s` in repo: `%s`' % ( |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1245 perm, TEST_USER_GROUP, self.REPO |
| 2526 | 1246 ), |
| 1247 'success': True | |
| 1248 } | |
| 1249 expected = ret | |
| 1250 self._compare_ok(id_, expected, given=response.body) | |
| 1251 | |
| 1252 def test_api_grant_users_group_permission_wrong_permission(self): | |
| 1253 perm = 'haha.no.permission' | |
| 1254 id_, params = _build_data(self.apikey, 'grant_users_group_permission', | |
| 1255 repoid=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1256 usersgroupid=TEST_USER_GROUP, |
| 2526 | 1257 perm=perm) |
| 2738 | 1258 response = api_call(self, params) |
| 2526 | 1259 |
| 1260 expected = 'permission `%s` does not exist' % perm | |
| 1261 self._compare_error(id_, expected, given=response.body) | |
| 1262 | |
| 1263 @mock.patch.object(RepoModel, 'grant_users_group_permission', crash) | |
| 1264 def test_api_grant_users_group_permission_exception_when_adding(self): | |
| 1265 perm = 'repository.read' | |
| 1266 id_, params = _build_data(self.apikey, 'grant_users_group_permission', | |
| 1267 repoid=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1268 usersgroupid=TEST_USER_GROUP, |
| 2526 | 1269 perm=perm) |
| 2738 | 1270 response = api_call(self, params) |
| 2526 | 1271 |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1272 expected = 'failed to edit permission for user group: `%s` in repo: `%s`' % ( |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1273 TEST_USER_GROUP, self.REPO |
| 2526 | 1274 ) |
| 1275 self._compare_error(id_, expected, given=response.body) | |
| 1276 | |
| 1277 def test_api_revoke_users_group_permission(self): | |
| 1278 RepoModel().grant_users_group_permission(repo=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1279 group_name=TEST_USER_GROUP, |
| 2526 | 1280 perm='repository.read') |
| 2738 | 1281 Session().commit() |
| 2526 | 1282 id_, params = _build_data(self.apikey, 'revoke_users_group_permission', |
| 1283 repoid=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1284 usersgroupid=TEST_USER_GROUP,) |
| 2738 | 1285 response = api_call(self, params) |
| 2526 | 1286 |
| 1287 expected = { | |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1288 'msg': 'Revoked perm for user group: `%s` in repo: `%s`' % ( |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1289 TEST_USER_GROUP, self.REPO |
| 2526 | 1290 ), |
| 1291 'success': True | |
| 1292 } | |
| 1293 self._compare_ok(id_, expected, given=response.body) | |
| 1294 | |
| 1295 @mock.patch.object(RepoModel, 'revoke_users_group_permission', crash) | |
| 1296 def test_api_revoke_users_group_permission_exception_when_adding(self): | |
| 1297 | |
| 1298 id_, params = _build_data(self.apikey, 'revoke_users_group_permission', | |
| 1299 repoid=self.REPO, | |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1300 usersgroupid=TEST_USER_GROUP,) |
| 2738 | 1301 response = api_call(self, params) |
| 2526 | 1302 |
|
3415
b8f929bff7e3
fixed tests and missing replacements from 5f1850e4712a
Marcin Kuzminski <marcin@python-works.com>
parents:
3235
diff
changeset
|
1303 expected = 'failed to edit permission for user group: `%s` in repo: `%s`' % ( |
|
3417
fa6ba6727475
further cleanup of UsersGroup
Mads Kiilerich <madski@unity3d.com>
parents:
3415
diff
changeset
|
1304 TEST_USER_GROUP, self.REPO |
| 2526 | 1305 ) |
| 1306 self._compare_error(id_, expected, given=response.body) |