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