2526
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
1 import random
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
2 import mock
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
3
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
4 from rhodecode.tests import *
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
5 from rhodecode.model.meta import Session
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
6 from rhodecode.lib.compat import json
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
7 from rhodecode.lib.auth import AuthUser
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
8 from rhodecode.model.user import UserModel
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
9 from rhodecode.model.users_group import UsersGroupModel
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
10 from rhodecode.model.repo import RepoModel
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
11 from nose.tools import with_setup
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
12
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
13 API_URL = '/_admin/api'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
14
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
15
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
16 def _build_data(apikey, method, **kw):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
17 """
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
18 Builds API data with given random ID
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
19
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
20 :param random_id:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
21 :type random_id:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
22 """
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
23 random_id = random.randrange(1, 9999)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
24 return random_id, json.dumps({
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
25 "id": random_id,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
26 "api_key": apikey,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
27 "method": method,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
28 "args": kw
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
29 })
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
30
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
31 jsonify = lambda obj: json.loads(json.dumps(obj))
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
32
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
33
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
34 def crash(*args, **kwargs):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
35 raise Exception('Total Crash !')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
36
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
37
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
38 TEST_USERS_GROUP = 'test_users_group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
39
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
40
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
41 def make_users_group(name=TEST_USERS_GROUP):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
42 gr = UsersGroupModel().create(name=name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
43 UsersGroupModel().add_user_to_group(users_group=gr,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
44 user=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
45 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
46 return gr
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
47
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
48
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
49 def destroy_users_group(name=TEST_USERS_GROUP):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
50 UsersGroupModel().delete(users_group=name, force=True)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
51 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
52
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
53
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
54 def create_repo(repo_name, repo_type):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
55 # create new repo
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
56 form_data = dict(repo_name=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
57 repo_name_full=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
58 fork_name=None,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
59 description='description %s' % repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
60 repo_group=None,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
61 private=False,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
62 repo_type=repo_type,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
63 clone_uri=None,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
64 landing_rev='tip')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
65 cur_user = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
66 r = RepoModel().create(form_data, cur_user)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
67 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
68 return r
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
69
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
70
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
71 def destroy_repo(repo_name):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
72 RepoModel().delete(repo_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
73 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
74
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
75
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
76 class BaseTestApi(object):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
77 REPO = None
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
78 REPO_TYPE = None
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
79
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
80 @classmethod
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
81 def setUpClass(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
82 self.usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
83 self.apikey = self.usr.api_key
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
84 self.TEST_USER = UserModel().create_or_update(
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
85 username='test-api',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
86 password='test',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
87 email='test@api.rhodecode.org',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
88 firstname='first',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
89 lastname='last'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
90 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
91 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
92 self.TEST_USER_LOGIN = self.TEST_USER.username
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
93
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
94 @classmethod
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
95 def teardownClass(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
96 pass
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
97
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
98 def setUp(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
99 self.maxDiff = None
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
100 make_users_group()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
101
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
102 def tearDown(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
103 destroy_users_group()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
104
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
105 def _compare_ok(self, id_, expected, given):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
106 expected = jsonify({
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
107 'id': id_,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
108 'error': None,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
109 'result': expected
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
110 })
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
111 given = json.loads(given)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
112 self.assertEqual(expected, given)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
113
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
114 def _compare_error(self, id_, expected, given):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
115 expected = jsonify({
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
116 'id': id_,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
117 'error': expected,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
118 'result': None
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
119 })
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
120 given = json.loads(given)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
121 self.assertEqual(expected, given)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
122
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
123 # def test_Optional(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
124 # from rhodecode.controllers.api.api import Optional
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
125 # option1 = Optional(None)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
126 # self.assertEqual('<Optional:%s>' % None, repr(option1))
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
127 #
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
128 # self.assertEqual(1, Optional.extract(Optional(1)))
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
129 # self.assertEqual('trololo', Optional.extract('trololo'))
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
130
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
131 def test_api_wrong_key(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
132 id_, params = _build_data('trololo', 'get_user')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
133 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
134 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
135
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
136 expected = 'Invalid API KEY'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
137 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
138
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
139 def test_api_missing_non_optional_param(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
140 id_, params = _build_data(self.apikey, 'get_user')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
141 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
142 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
143
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
144 expected = 'Missing non optional `userid` arg in JSON DATA'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
145 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
146
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
147 def test_api_get_users(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
148 id_, params = _build_data(self.apikey, 'get_users',)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
149 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
150 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
151 ret_all = []
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
152 for usr in UserModel().get_all():
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
153 ret = usr.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
154 ret_all.append(jsonify(ret))
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
155 expected = ret_all
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
156 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
157
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
158 def test_api_get_user(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
159 id_, params = _build_data(self.apikey, 'get_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
160 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
161 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
162 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
163
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
164 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
165 ret = usr.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
166 ret['permissions'] = AuthUser(usr.user_id).permissions
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
167
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
168 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
169 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
170
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
171 def test_api_get_user_that_does_not_exist(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
172 id_, params = _build_data(self.apikey, 'get_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
173 userid='trololo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
174 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
175 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
176
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
177 expected = "user `%s` does not exist" % 'trololo'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
178 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
179
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
180 def test_api_pull(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
181 #TODO: issues with rhodecode_extras here.. not sure why !
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
182 pass
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
183
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
184 # repo_name = 'test_pull'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
185 # r = create_repo(repo_name, self.REPO_TYPE)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
186 # r.clone_uri = TEST_self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
187 # Session.add(r)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
188 # Session.commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
189 #
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
190 # id_, params = _build_data(self.apikey, 'pull',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
191 # repoid=repo_name,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
192 # response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
193 # params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
194 #
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
195 # expected = 'Pulled from `%s`' % repo_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
196 # self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
197 #
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
198 # destroy_repo(repo_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
199
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
200 def test_api_pull_error(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
201 id_, params = _build_data(self.apikey, 'pull',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
202 repoid=self.REPO,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
203 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
204 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
205
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
206 expected = 'Unable to pull changes from `%s`' % self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
207 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
208
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
209 def test_api_create_existing_user(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
210 id_, params = _build_data(self.apikey, 'create_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
211 username=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
212 email='test@foo.com',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
213 password='trololo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
214 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
215 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
216
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
217 expected = "user `%s` already exist" % TEST_USER_ADMIN_LOGIN
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
218 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
219
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
220 def test_api_create_user_with_existing_email(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
221 id_, params = _build_data(self.apikey, 'create_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
222 username=TEST_USER_ADMIN_LOGIN + 'new',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
223 email=TEST_USER_REGULAR_EMAIL,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
224 password='trololo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
225 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
226 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
227
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
228 expected = "email `%s` already exist" % TEST_USER_REGULAR_EMAIL
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
229 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
230
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
231 def test_api_create_user(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
232 username = 'test_new_api_user'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
233 email = username + "@foo.com"
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
234
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
235 id_, params = _build_data(self.apikey, 'create_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
236 username=username,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
237 email=email,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
238 password='trololo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
239 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
240 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
241
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
242 usr = UserModel().get_by_username(username)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
243 ret = dict(
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
244 msg='created new user `%s`' % username,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
245 user=jsonify(usr.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
246 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
247
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
248 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
249 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
250
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
251 UserModel().delete(usr.user_id)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
252 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
253
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
254 @mock.patch.object(UserModel, 'create_or_update', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
255 def test_api_create_user_when_exception_happened(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
256
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
257 username = 'test_new_api_user'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
258 email = username + "@foo.com"
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
259
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
260 id_, params = _build_data(self.apikey, 'create_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
261 username=username,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
262 email=email,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
263 password='trololo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
264 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
265 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
266 expected = 'failed to create user `%s`' % username
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
267 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
268
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
269 def test_api_delete_user(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
270 usr = UserModel().create_or_update(username=u'test_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
271 password=u'qweqwe',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
272 email=u'u232@rhodecode.org',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
273 firstname=u'u1', lastname=u'u1')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
274 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
275 username = usr.username
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
276 email = usr.email
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
277 usr_id = usr.user_id
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
278 ## DELETE THIS USER NOW
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
279
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
280 id_, params = _build_data(self.apikey, 'delete_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
281 userid=username,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
282 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
283 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
284
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
285 ret = {'msg': 'deleted user ID:%s %s' % (usr_id, username),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
286 'user': None}
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
287 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
288 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
289
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
290 @mock.patch.object(UserModel, 'delete', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
291 def test_api_delete_user_when_exception_happened(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
292 usr = UserModel().create_or_update(username=u'test_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
293 password=u'qweqwe',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
294 email=u'u232@rhodecode.org',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
295 firstname=u'u1', lastname=u'u1')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
296 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
297 username = usr.username
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
298
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
299 id_, params = _build_data(self.apikey, 'delete_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
300 userid=username,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
301 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
302 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
303 ret = 'failed to delete ID:%s %s' % (usr.user_id,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
304 usr.username)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
305 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
306 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
307
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
308 @parameterized.expand([('firstname', 'new_username'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
309 ('lastname', 'new_username'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
310 ('email', 'new_username'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
311 ('admin', True),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
312 ('admin', False),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
313 ('ldap_dn', 'test'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
314 ('ldap_dn', None),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
315 ('active', False),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
316 ('active', True),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
317 ('password', 'newpass')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
318 ])
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
319 def test_api_update_user(self, name, expected):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
320 usr = UserModel().get_by_username(self.TEST_USER_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
321 kw = {name: expected,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
322 'userid': usr.user_id}
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
323 id_, params = _build_data(self.apikey, 'update_user', **kw)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
324 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
325 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
326
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
327 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
328 'msg': 'updated user ID:%s %s' % (usr.user_id, self.TEST_USER_LOGIN),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
329 'user': jsonify(UserModel()\
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
330 .get_by_username(self.TEST_USER_LOGIN)\
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
331 .get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
332 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
333
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
334 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
335 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
336
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
337 def test_api_update_user_no_changed_params(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
338 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
339 ret = jsonify(usr.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
340 id_, params = _build_data(self.apikey, 'update_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
341 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
342
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
343 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
344 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
345 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
346 'msg': 'updated user ID:%s %s' % (usr.user_id, TEST_USER_ADMIN_LOGIN),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
347 'user': ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
348 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
349 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
350 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
351
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
352 def test_api_update_user_by_user_id(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
353 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
354 ret = jsonify(usr.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
355 id_, params = _build_data(self.apikey, 'update_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
356 userid=usr.user_id)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
357
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
358 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
359 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
360 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
361 'msg': 'updated user ID:%s %s' % (usr.user_id, TEST_USER_ADMIN_LOGIN),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
362 'user': ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
363 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
364 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
365 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
366
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
367 @mock.patch.object(UserModel, 'create_or_update', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
368 def test_api_update_user_when_exception_happens(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
369 usr = UserModel().get_by_username(TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
370 ret = jsonify(usr.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
371 id_, params = _build_data(self.apikey, 'update_user',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
372 userid=usr.user_id)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
373
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
374 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
375 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
376 ret = 'failed to update user `%s`' % usr.user_id
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
377
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
378 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
379 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
380
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
381 def test_api_get_repo(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
382 new_group = 'some_new_group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
383 make_users_group(new_group)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
384 RepoModel().grant_users_group_permission(repo=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
385 group_name=new_group,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
386 perm='repository.read')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
387 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
388 id_, params = _build_data(self.apikey, 'get_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
389 repoid=self.REPO)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
390 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
391 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
392
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
393 repo = RepoModel().get_by_repo_name(self.REPO)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
394 ret = repo.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
395
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
396 members = []
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
397 for user in repo.repo_to_perm:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
398 perm = user.permission.permission_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
399 user = user.user
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
400 user_data = user.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
401 user_data['type'] = "user"
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
402 user_data['permission'] = perm
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
403 members.append(user_data)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
404
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
405 for users_group in repo.users_group_to_perm:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
406 perm = users_group.permission.permission_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
407 users_group = users_group.users_group
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
408 users_group_data = users_group.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
409 users_group_data['type'] = "users_group"
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
410 users_group_data['permission'] = perm
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
411 members.append(users_group_data)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
412
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
413 ret['members'] = members
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
414
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
415 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
416 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
417 destroy_users_group(new_group)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
418
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
419 def test_api_get_repo_that_doesn_not_exist(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
420 id_, params = _build_data(self.apikey, 'get_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
421 repoid='no-such-repo')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
422 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
423 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
424
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
425 ret = 'repository `%s` does not exist' % 'no-such-repo'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
426 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
427 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
428
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
429 def test_api_get_repos(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
430 id_, params = _build_data(self.apikey, 'get_repos')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
431 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
432 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
433
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
434 result = []
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
435 for repo in RepoModel().get_all():
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
436 result.append(repo.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
437 ret = jsonify(result)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
438
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
439 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
440 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
441
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
442 @parameterized.expand([('all', 'all'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
443 ('dirs', 'dirs'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
444 ('files', 'files'), ])
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
445 def test_api_get_repo_nodes(self, name, ret_type):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
446 rev = 'tip'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
447 path = '/'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
448 id_, params = _build_data(self.apikey, 'get_repo_nodes',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
449 repoid=self.REPO, revision=rev,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
450 root_path=path,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
451 ret_type=ret_type)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
452 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
453 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
454
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
455 # we don't the actual return types here since it's tested somewhere
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
456 # else
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
457 expected = json.loads(response.body)['result']
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
458 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
459
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
460 def test_api_get_repo_nodes_bad_revisions(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
461 rev = 'i-dont-exist'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
462 path = '/'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
463 id_, params = _build_data(self.apikey, 'get_repo_nodes',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
464 repoid=self.REPO, revision=rev,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
465 root_path=path,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
466 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
467 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
468
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
469 expected = 'failed to get repo: `%s` nodes' % self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
470 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
471
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
472 def test_api_get_repo_nodes_bad_path(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
473 rev = 'tip'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
474 path = '/idontexits'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
475 id_, params = _build_data(self.apikey, 'get_repo_nodes',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
476 repoid=self.REPO, revision=rev,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
477 root_path=path,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
478 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
479 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
480
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
481 expected = 'failed to get repo: `%s` nodes' % self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
482 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
483
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
484 def test_api_get_repo_nodes_bad_ret_type(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
485 rev = 'tip'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
486 path = '/'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
487 ret_type = 'error'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
488 id_, params = _build_data(self.apikey, 'get_repo_nodes',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
489 repoid=self.REPO, revision=rev,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
490 root_path=path,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
491 ret_type=ret_type)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
492 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
493 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
494
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
495 expected = 'ret_type must be one of %s' % (['files', 'dirs', 'all'])
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
496 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
497
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
498 def test_api_create_repo(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
499 repo_name = 'api-repo'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
500 id_, params = _build_data(self.apikey, 'create_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
501 repo_name=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
502 owner=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
503 repo_type='hg',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
504 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
505 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
506 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
507
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
508 repo = RepoModel().get_by_repo_name(repo_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
509 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
510 'msg': 'Created new repository `%s`' % repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
511 'repo': jsonify(repo.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
512 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
513 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
514 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
515 destroy_repo(repo_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
516
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
517 def test_api_create_repo_unknown_owner(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
518 repo_name = 'api-repo'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
519 owner = 'i-dont-exist'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
520 id_, params = _build_data(self.apikey, 'create_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
521 repo_name=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
522 owner=owner,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
523 repo_type='hg',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
524 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
525 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
526 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
527 expected = 'user `%s` does not exist' % owner
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
528 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
529
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
530 def test_api_create_repo_exists(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
531 repo_name = self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
532 id_, params = _build_data(self.apikey, 'create_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
533 repo_name=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
534 owner=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
535 repo_type='hg',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
536 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
537 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
538 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
539 expected = "repo `%s` already exist" % repo_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
540 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
541
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
542 @mock.patch.object(RepoModel, 'create_repo', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
543 def test_api_create_repo_exception_occurred(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
544 repo_name = 'api-repo'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
545 id_, params = _build_data(self.apikey, 'create_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
546 repo_name=repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
547 owner=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
548 repo_type='hg',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
549 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
550 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
551 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
552 expected = 'failed to create repository `%s`' % repo_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
553 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
554
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
555 def test_api_delete_repo(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
556 repo_name = 'api_delete_me'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
557 create_repo(repo_name, self.REPO_TYPE)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
558
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
559 id_, params = _build_data(self.apikey, 'delete_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
560 repoid=repo_name,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
561 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
562 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
563
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
564 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
565 'msg': 'Deleted repository `%s`' % repo_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
566 'success': True
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
567 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
568 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
569 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
570
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
571 def test_api_delete_repo_exception_occurred(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
572 repo_name = 'api_delete_me'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
573 create_repo(repo_name, self.REPO_TYPE)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
574 with mock.patch.object(RepoModel, 'delete', crash):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
575 id_, params = _build_data(self.apikey, 'delete_repo',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
576 repoid=repo_name,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
577 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
578 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
579
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
580 expected = 'failed to delete repository `%s`' % repo_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
581 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
582 destroy_repo(repo_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
583
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
584 def test_api_fork_repo(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
585 self.failIf(False, 'TODO:')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
586
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
587 def test_api_get_users_group(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
588 id_, params = _build_data(self.apikey, 'get_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
589 usersgroupid=TEST_USERS_GROUP)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
590 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
591 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
592
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
593 users_group = UsersGroupModel().get_group(TEST_USERS_GROUP)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
594 members = []
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
595 for user in users_group.members:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
596 user = user.user
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
597 members.append(user.get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
598
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
599 ret = users_group.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
600 ret['members'] = members
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
601 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
602 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
603
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
604 def test_api_get_users_groups(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
605
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
606 make_users_group('test_users_group2')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
607
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
608 id_, params = _build_data(self.apikey, 'get_users_groups',)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
609 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
610 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
611
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
612 expected = []
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
613 for gr_name in [TEST_USERS_GROUP, 'test_users_group2']:
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
614 users_group = UsersGroupModel().get_group(gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
615 ret = users_group.get_api_data()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
616 expected.append(ret)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
617 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
618
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
619 UsersGroupModel().delete(users_group='test_users_group2')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
620 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
621
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
622 def test_api_create_users_group(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
623 group_name = 'some_new_group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
624 id_, params = _build_data(self.apikey, 'create_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
625 group_name=group_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
626 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
627 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
628
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
629 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
630 'msg': 'created new users group `%s`' % group_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
631 'users_group': jsonify(UsersGroupModel()\
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
632 .get_by_name(group_name)\
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
633 .get_api_data())
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
634 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
635 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
636 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
637
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
638 destroy_users_group(group_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
639
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
640 def test_api_get_users_group_that_exist(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
641 id_, params = _build_data(self.apikey, 'create_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
642 group_name=TEST_USERS_GROUP)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
643 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
644 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
645
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
646 expected = "users group `%s` already exist" % TEST_USERS_GROUP
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
647 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
648
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
649 @mock.patch.object(UsersGroupModel, 'create', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
650 def test_api_get_users_group_exception_occurred(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
651 group_name = 'exception_happens'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
652 id_, params = _build_data(self.apikey, 'create_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
653 group_name=group_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
654 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
655 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
656
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
657 expected = 'failed to create group `%s`' % group_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
658 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
659
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
660 def test_api_add_user_to_users_group(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
661 gr_name = 'test_group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
662 UsersGroupModel().create(gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
663 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
664 id_, params = _build_data(self.apikey, 'add_user_to_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
665 usersgroupid=gr_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
666 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
667 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
668 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
669
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
670 expected = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
671 'msg': 'added member `%s` to users group `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
672 TEST_USER_ADMIN_LOGIN, gr_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
673 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
674 'success': True}
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
675 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
676
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
677 UsersGroupModel().delete(users_group=gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
678 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
679
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
680 def test_api_add_user_to_users_group_that_doesnt_exist(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
681 id_, params = _build_data(self.apikey, 'add_user_to_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
682 usersgroupid='false-group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
683 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
684 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
685 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
686
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
687 expected = 'users group `%s` does not exist' % 'false-group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
688 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
689
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
690 @mock.patch.object(UsersGroupModel, 'add_user_to_group', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
691 def test_api_add_user_to_users_group_exception_occurred(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
692 gr_name = 'test_group'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
693 UsersGroupModel().create(gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
694 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
695 id_, params = _build_data(self.apikey, 'add_user_to_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
696 usersgroupid=gr_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
697 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
698 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
699 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
700
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
701 expected = 'failed to add member to users group `%s`' % gr_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
702 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
703
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
704 UsersGroupModel().delete(users_group=gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
705 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
706
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
707 def test_api_remove_user_from_users_group(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
708 gr_name = 'test_group_3'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
709 gr = UsersGroupModel().create(gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
710 UsersGroupModel().add_user_to_group(gr, user=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
711 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
712 id_, params = _build_data(self.apikey, 'remove_user_from_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
713 usersgroupid=gr_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
714 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
715 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
716 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
717
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
718 expected = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
719 'msg': 'removed member `%s` from users group `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
720 TEST_USER_ADMIN_LOGIN, gr_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
721 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
722 'success': True}
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
723 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
724
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
725 UsersGroupModel().delete(users_group=gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
726 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
727
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
728 @mock.patch.object(UsersGroupModel, 'remove_user_from_group', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
729 def test_api_remove_user_from_users_group_exception_occurred(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
730 gr_name = 'test_group_3'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
731 gr = UsersGroupModel().create(gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
732 UsersGroupModel().add_user_to_group(gr, user=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
733 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
734 id_, params = _build_data(self.apikey, 'remove_user_from_users_group',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
735 usersgroupid=gr_name,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
736 userid=TEST_USER_ADMIN_LOGIN)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
737 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
738 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
739
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
740 expected = 'failed to remove member from users group `%s`' % gr_name
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
741 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
742
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
743 UsersGroupModel().delete(users_group=gr_name)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
744 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
745
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
746 @parameterized.expand([('none', 'repository.none'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
747 ('read', 'repository.read'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
748 ('write', 'repository.write'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
749 ('admin', 'repository.admin')])
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
750 def test_api_grant_user_permission(self, name, perm):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
751 id_, params = _build_data(self.apikey, 'grant_user_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
752 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
753 userid=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
754 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
755 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
756 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
757
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
758 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
759 'msg': 'Granted perm: `%s` for user: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
760 perm, TEST_USER_ADMIN_LOGIN, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
761 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
762 'success': True
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
763 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
764 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
765 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
766
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
767 def test_api_grant_user_permission_wrong_permission(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
768 perm = 'haha.no.permission'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
769 id_, params = _build_data(self.apikey, 'grant_user_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
770 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
771 userid=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
772 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
773 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
774 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
775
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
776 expected = 'permission `%s` does not exist' % perm
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
777 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
778
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
779 @mock.patch.object(RepoModel, 'grant_user_permission', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
780 def test_api_grant_user_permission_exception_when_adding(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
781 perm = 'repository.read'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
782 id_, params = _build_data(self.apikey, 'grant_user_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
783 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
784 userid=TEST_USER_ADMIN_LOGIN,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
785 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
786 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
787 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
788
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
789 expected = 'failed to edit permission for user: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
790 TEST_USER_ADMIN_LOGIN, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
791 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
792 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
793
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
794 def test_api_revoke_user_permission(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
795 id_, params = _build_data(self.apikey, 'revoke_user_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
796 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
797 userid=TEST_USER_ADMIN_LOGIN,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
798 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
799 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
800
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
801 expected = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
802 'msg': 'Revoked perm for user: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
803 TEST_USER_ADMIN_LOGIN, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
804 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
805 'success': True
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
806 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
807 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
808
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
809 @mock.patch.object(RepoModel, 'revoke_user_permission', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
810 def test_api_revoke_user_permission_exception_when_adding(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
811 id_, params = _build_data(self.apikey, 'revoke_user_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
812 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
813 userid=TEST_USER_ADMIN_LOGIN,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
814 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
815 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
816
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
817 expected = 'failed to edit permission for user: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
818 TEST_USER_ADMIN_LOGIN, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
819 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
820 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
821
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
822 @parameterized.expand([('none', 'repository.none'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
823 ('read', 'repository.read'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
824 ('write', 'repository.write'),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
825 ('admin', 'repository.admin')])
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
826 def test_api_grant_users_group_permission(self, name, perm):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
827 id_, params = _build_data(self.apikey, 'grant_users_group_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
828 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
829 usersgroupid=TEST_USERS_GROUP,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
830 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
831 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
832 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
833
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
834 ret = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
835 'msg': 'Granted perm: `%s` for users group: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
836 perm, TEST_USERS_GROUP, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
837 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
838 'success': True
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
839 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
840 expected = ret
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
841 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
842
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
843 def test_api_grant_users_group_permission_wrong_permission(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
844 perm = 'haha.no.permission'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
845 id_, params = _build_data(self.apikey, 'grant_users_group_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
846 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
847 usersgroupid=TEST_USERS_GROUP,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
848 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
849 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
850 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
851
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
852 expected = 'permission `%s` does not exist' % perm
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
853 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
854
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
855 @mock.patch.object(RepoModel, 'grant_users_group_permission', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
856 def test_api_grant_users_group_permission_exception_when_adding(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
857 perm = 'repository.read'
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
858 id_, params = _build_data(self.apikey, 'grant_users_group_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
859 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
860 usersgroupid=TEST_USERS_GROUP,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
861 perm=perm)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
862 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
863 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
864
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
865 expected = 'failed to edit permission for users group: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
866 TEST_USERS_GROUP, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
867 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
868 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
869
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
870 def test_api_revoke_users_group_permission(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
871 RepoModel().grant_users_group_permission(repo=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
872 group_name=TEST_USERS_GROUP,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
873 perm='repository.read')
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
874 Session().commit()
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
875 id_, params = _build_data(self.apikey, 'revoke_users_group_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
876 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
877 usersgroupid=TEST_USERS_GROUP,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
878 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
879 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
880
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
881 expected = {
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
882 'msg': 'Revoked perm for users group: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
883 TEST_USERS_GROUP, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
884 ),
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
885 'success': True
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
886 }
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
887 self._compare_ok(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
888
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
889 @mock.patch.object(RepoModel, 'revoke_users_group_permission', crash)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
890 def test_api_revoke_users_group_permission_exception_when_adding(self):
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
891
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
892 id_, params = _build_data(self.apikey, 'revoke_users_group_permission',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
893 repoid=self.REPO,
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
894 usersgroupid=TEST_USERS_GROUP,)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
895 response = self.app.post(API_URL, content_type='application/json',
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
896 params=params)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
897
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
898 expected = 'failed to edit permission for users group: `%s` in repo: `%s`' % (
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
899 TEST_USERS_GROUP, self.REPO
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
900 )
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
901 self._compare_error(id_, expected, given=response.body)
|
Marcin Kuzminski <marcin@python-works.com>
parents:
diff
changeset
|
902
|