1
|
|
|
# -*- coding: utf-8 -*- |
2
|
|
|
# pylint: disable=invalid-name, too-many-ancestors |
3
|
|
|
|
4
|
|
|
import html |
5
|
|
|
from http import HTTPStatus |
6
|
|
|
|
7
|
|
|
from django.utils import formats |
8
|
|
|
from django.urls import reverse |
9
|
|
|
from django.conf import settings |
10
|
|
|
from django.contrib.auth.models import Permission |
11
|
|
|
from django.utils.translation import ugettext_lazy as _ |
12
|
|
|
|
13
|
|
|
from tcms.testruns.models import TestExecutionStatus |
14
|
|
|
from tcms.testruns.models import TestRun |
15
|
|
|
from tcms.utils.permissions import initiate_user_with_default_setups |
16
|
|
|
|
17
|
|
|
from tcms.tests import BaseCaseRun |
18
|
|
|
from tcms.tests import BasePlanCase |
19
|
|
|
from tcms.tests import remove_perm_from_user |
20
|
|
|
from tcms.tests import user_should_have_perm |
21
|
|
|
from tcms.tests.factories import BuildFactory |
22
|
|
|
from tcms.tests.factories import TagFactory |
23
|
|
|
from tcms.tests.factories import TestCaseFactory |
24
|
|
|
from tcms.tests.factories import UserFactory |
25
|
|
|
|
26
|
|
|
|
27
|
|
|
class TestGetRun(BaseCaseRun): |
28
|
|
|
"""Test get view method""" |
29
|
|
|
|
30
|
|
|
@classmethod |
31
|
|
|
def setUpTestData(cls): |
32
|
|
|
super().setUpTestData() |
33
|
|
|
initiate_user_with_default_setups(cls.tester) |
34
|
|
|
|
35
|
|
|
for _i in range(3): |
36
|
|
|
cls.test_run.add_tag(TagFactory()) |
37
|
|
|
|
38
|
|
|
cls.unauthorized = UserFactory() |
39
|
|
|
cls.unauthorized.set_password('password') |
40
|
|
|
cls.unauthorized.save() |
41
|
|
|
|
42
|
|
|
cls.unauthorized.user_permissions.add(*Permission.objects.all()) |
43
|
|
|
remove_perm_from_user(cls.unauthorized, 'testruns.add_testruntag') |
44
|
|
|
remove_perm_from_user(cls.unauthorized, 'testruns.delete_testruntag') |
45
|
|
|
|
46
|
|
|
def test_404_if_non_existing_pk(self): |
47
|
|
|
url = reverse('testruns-get', args=[99999999]) |
48
|
|
|
response = self.client.get(url) |
49
|
|
|
self.assertEqual(HTTPStatus.NOT_FOUND, response.status_code) |
50
|
|
|
|
51
|
|
|
def test_get_a_run(self): |
52
|
|
|
url = reverse('testruns-get', args=[self.test_run.pk]) |
53
|
|
|
response = self.client.get(url) |
54
|
|
|
|
55
|
|
|
self.assertEqual(HTTPStatus.OK, response.status_code) |
56
|
|
|
self.assertContains(response, 'Add Tag') |
57
|
|
|
self.assertContains(response, 'js-remove-tag') |
58
|
|
|
|
59
|
|
|
for i, case_run in enumerate( |
60
|
|
|
(self.execution_1, self.execution_2, self.execution_3), 1): |
61
|
|
|
self.assertContains( |
62
|
|
|
response, |
63
|
|
|
'<a href="#caserun_{0}">#{0}</a>'.format(case_run.pk), |
64
|
|
|
html=True) |
65
|
|
|
self.assertContains( |
66
|
|
|
response, |
67
|
|
|
'<a id="link_{0}" href="#caserun_{1}" title="Expand test case">' |
68
|
|
|
'{2}</a>'.format(i, case_run.pk, case_run.case.summary), |
69
|
|
|
html=True) |
70
|
|
|
|
71
|
|
|
def test_get_run_without_permissions_to_add_or_remove_tags(self): |
72
|
|
|
self.client.logout() |
73
|
|
|
|
74
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
75
|
|
|
username=self.unauthorized.username, |
76
|
|
|
password='password') |
77
|
|
|
|
78
|
|
|
url = reverse('testruns-get', args=[self.test_run.pk]) |
79
|
|
|
response = self.client.get(url) |
80
|
|
|
|
81
|
|
|
self.assertEqual(HTTPStatus.OK, response.status_code) |
82
|
|
|
self.assertNotContains(response, 'Add Tag') |
83
|
|
|
self.assertNotContains(response, 'js-remove-tag') |
84
|
|
|
|
85
|
|
|
|
86
|
|
|
class TestCreateNewRun(BasePlanCase): |
87
|
|
|
"""Test creating new run""" |
88
|
|
|
|
89
|
|
|
@classmethod |
90
|
|
|
def setUpTestData(cls): |
91
|
|
|
super(TestCreateNewRun, cls).setUpTestData() |
92
|
|
|
|
93
|
|
|
cls.permission = 'testruns.add_testrun' |
94
|
|
|
user_should_have_perm(cls.tester, cls.permission) |
95
|
|
|
cls.url = reverse('testruns-new') |
96
|
|
|
cls.build_fast = BuildFactory(name='fast', product=cls.product) |
97
|
|
|
|
98
|
|
|
def test_refuse_if_missing_plan_pk(self): |
99
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
100
|
|
|
username=self.tester.username, |
101
|
|
|
password='password') |
102
|
|
|
response = self.client.post(self.url, {}) |
103
|
|
|
self.assertRedirects(response, reverse('plans-search')) |
104
|
|
|
|
105
|
|
|
def test_refuse_if_missing_cases_pks(self): |
106
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
107
|
|
|
username=self.tester.username, |
108
|
|
|
password='password') |
109
|
|
|
response = self.client.post(self.url, {'from_plan': self.plan.pk}, follow=True) |
110
|
|
|
self.assertContains(response, _('Creating a TestRun requires at least one TestCase')) |
111
|
|
|
|
112
|
|
|
def test_show_create_new_run_page(self): |
113
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
114
|
|
|
username=self.tester.username, |
115
|
|
|
password='password') |
116
|
|
|
|
117
|
|
|
response = self.client.post(self.url, { |
118
|
|
|
'from_plan': self.plan.pk, |
119
|
|
|
'case': [self.case_1.pk, self.case_2.pk, self.case_3.pk] |
120
|
|
|
}) |
121
|
|
|
|
122
|
|
|
# Assert listed cases |
123
|
|
|
for _i, case in enumerate((self.case_1, self.case_2, self.case_3), 1): |
124
|
|
|
case_url = reverse('testcases-get', args=[case.pk]) |
125
|
|
|
self.assertContains( |
126
|
|
|
response, |
127
|
|
|
'<a href="%s">TC-%d: %s</a>' % (case_url, case.pk, case.summary), |
128
|
|
|
html=True) |
129
|
|
|
|
130
|
|
|
def test_create_a_new_run(self): |
131
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
132
|
|
|
username=self.tester.username, |
133
|
|
|
password='password') |
134
|
|
|
|
135
|
|
|
clone_data = { |
136
|
|
|
'summary': self.plan.name, |
137
|
|
|
'from_plan': self.plan.pk, |
138
|
|
|
'build': self.build_fast.pk, |
139
|
|
|
'manager': self.tester.email, |
140
|
|
|
'default_tester': self.tester.email, |
141
|
|
|
'notes': 'Clone new run', |
142
|
|
|
'case': [self.case_1.pk, self.case_2.pk], |
143
|
|
|
'POSTING_TO_CREATE': 'YES', |
144
|
|
|
} |
145
|
|
|
|
146
|
|
|
url = reverse('testruns-new') |
147
|
|
|
response = self.client.post(url, clone_data) |
148
|
|
|
|
149
|
|
|
new_run = TestRun.objects.last() |
150
|
|
|
|
151
|
|
|
self.assertRedirects( |
152
|
|
|
response, |
153
|
|
|
reverse('testruns-get', args=[new_run.pk])) |
154
|
|
|
|
155
|
|
|
self.assertEqual(self.plan.name, new_run.summary) |
156
|
|
|
self.assertEqual(self.plan, new_run.plan) |
157
|
|
|
self.assertEqual(self.version, new_run.product_version) |
158
|
|
|
self.assertEqual(None, new_run.stop_date) |
159
|
|
|
self.assertEqual('Clone new run', new_run.notes) |
160
|
|
|
self.assertEqual(self.build_fast, new_run.build) |
161
|
|
|
self.assertEqual(self.tester, new_run.manager) |
162
|
|
|
self.assertEqual(self.tester, new_run.default_tester) |
163
|
|
|
|
164
|
|
|
for case, case_run in zip((self.case_1, self.case_2), |
165
|
|
|
new_run.case_run.order_by('case')): |
166
|
|
|
self.assertEqual(case, case_run.case) |
167
|
|
|
self.assertEqual(None, case_run.tested_by) |
168
|
|
|
self.assertEqual(self.tester, case_run.assignee) |
169
|
|
|
self.assertEqual(TestExecutionStatus.objects.get(name='IDLE'), |
170
|
|
|
case_run.status) |
171
|
|
|
self.assertEqual(case.history.latest().history_id, case_run.case_text_version) |
172
|
|
|
self.assertEqual(new_run.build, case_run.build) |
173
|
|
|
self.assertEqual(None, case_run.close_date) |
174
|
|
|
|
175
|
|
|
def test_create_a_new_run_without_permissions_should_fail(self): |
176
|
|
|
remove_perm_from_user(self.tester, 'testruns.add_testrun') |
177
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
178
|
|
|
username=self.tester.username, |
179
|
|
|
password='password') |
180
|
|
|
|
181
|
|
|
clone_data = { |
182
|
|
|
'summary': self.plan.name, |
183
|
|
|
'from_plan': self.plan.pk, |
184
|
|
|
'build': self.build_fast.pk, |
185
|
|
|
'manager': self.tester.email, |
186
|
|
|
'default_tester': self.tester.email, |
187
|
|
|
'notes': 'Clone new run', |
188
|
|
|
'case': [self.case_1.pk, self.case_2.pk], |
189
|
|
|
'POSTING_TO_CREATE': 'YES', |
190
|
|
|
} |
191
|
|
|
|
192
|
|
|
url = reverse('testruns-new') |
193
|
|
|
|
194
|
|
|
self.assertRedirects( |
195
|
|
|
self.client.post(url, clone_data), |
196
|
|
|
reverse('tcms-login') + '?next=' + url) |
197
|
|
|
|
198
|
|
|
|
199
|
|
|
class CloneRunBaseTest(BaseCaseRun): |
200
|
|
|
|
201
|
|
|
def assert_one_run_clone_page(self, response): |
202
|
|
|
"""Verify clone page for cloning one test run""" |
203
|
|
|
|
204
|
|
|
self.assertContains( |
205
|
|
|
response, |
206
|
|
|
'<input id="id_summary" class="form-control" name="summary" ' |
207
|
|
|
'type="text" value="%s%s" required>' % (_('Clone of '), self.test_run.summary), |
208
|
|
|
html=True) |
209
|
|
|
|
210
|
|
|
for case_run in (self.execution_1, self.execution_2): |
211
|
|
|
case_url = reverse('testcases-get', args=[case_run.case.pk]) |
212
|
|
|
|
213
|
|
|
self.assertContains( |
214
|
|
|
response, |
215
|
|
|
'<a href="%s">TC-%d: %s</a>' % (case_url, case_run.case.pk, case_run.case.summary), |
216
|
|
|
html=True) |
217
|
|
|
|
218
|
|
|
|
219
|
|
|
class TestStartCloneRunFromRunPage(CloneRunBaseTest): |
220
|
|
|
"""Test case for cloning run from a run page""" |
221
|
|
|
|
222
|
|
|
@classmethod |
223
|
|
|
def setUpTestData(cls): |
224
|
|
|
super(TestStartCloneRunFromRunPage, cls).setUpTestData() |
225
|
|
|
|
226
|
|
|
cls.permission = 'testruns.add_testrun' |
227
|
|
|
user_should_have_perm(cls.tester, cls.permission) |
228
|
|
|
|
229
|
|
|
def test_refuse_without_selecting_case_runs(self): |
230
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
231
|
|
|
username=self.tester.username, |
232
|
|
|
password='password') |
233
|
|
|
url = reverse('testruns-clone', args=[self.test_run.pk]) |
234
|
|
|
|
235
|
|
|
response = self.client.post(url, {}, follow=True) |
236
|
|
|
|
237
|
|
|
self.assertContains(response, _('At least one TestCase is required')) |
238
|
|
|
|
239
|
|
|
def test_open_clone_page_by_selecting_case_runs(self): |
240
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
241
|
|
|
username=self.tester.username, |
242
|
|
|
password='password') |
243
|
|
|
url = reverse('testruns-clone', args=[self.test_run.pk]) |
244
|
|
|
|
245
|
|
|
response = self.client.post(url, {'case_run': [self.execution_1.pk, self.execution_2.pk]}) |
246
|
|
|
|
247
|
|
|
self.assert_one_run_clone_page(response) |
248
|
|
|
|
249
|
|
|
def test_clone_a_run(self): |
250
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
251
|
|
|
username=self.tester.username, |
252
|
|
|
password='password') |
253
|
|
|
|
254
|
|
|
new_summary = 'Clone {} - {}'.format(self.test_run.pk, self.test_run.summary) |
255
|
|
|
|
256
|
|
|
clone_data = { |
257
|
|
|
'summary': new_summary, |
258
|
|
|
'from_plan': self.plan.pk, |
259
|
|
|
'product_id': self.test_run.plan.product_id, |
260
|
|
|
'do': 'clone_run', |
261
|
|
|
'orig_run_id': self.test_run.pk, |
262
|
|
|
'POSTING_TO_CREATE': 'YES', |
263
|
|
|
'product': self.test_run.plan.product_id, |
264
|
|
|
'product_version': self.test_run.product_version.pk, |
265
|
|
|
'build': self.test_run.build.pk, |
266
|
|
|
'errata_id': '', |
267
|
|
|
'manager': self.test_run.manager.email, |
268
|
|
|
'default_tester': self.test_run.default_tester.email, |
269
|
|
|
'notes': '', |
270
|
|
|
'case': [self.execution_1.case.pk, self.execution_2.case.pk], |
271
|
|
|
'case_run_id': [self.execution_1.pk, self.execution_2.pk], |
272
|
|
|
} |
273
|
|
|
|
274
|
|
|
url = reverse('testruns-new') |
275
|
|
|
response = self.client.post(url, clone_data) |
276
|
|
|
|
277
|
|
|
cloned_run = TestRun.objects.get(summary=new_summary) |
278
|
|
|
|
279
|
|
|
self.assertRedirects( |
280
|
|
|
response, |
281
|
|
|
reverse('testruns-get', args=[cloned_run.pk])) |
282
|
|
|
|
283
|
|
|
self.assert_cloned_run(cloned_run) |
284
|
|
|
|
285
|
|
|
def test_clone_a_run_without_permissions(self): |
286
|
|
|
remove_perm_from_user(self.tester, 'testruns.add_testrun') |
287
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
288
|
|
|
username=self.tester.username, |
289
|
|
|
password='password') |
290
|
|
|
|
291
|
|
|
new_summary = 'Clone {} - {}'.format(self.test_run.pk, self.test_run.summary) |
292
|
|
|
|
293
|
|
|
clone_data = { |
294
|
|
|
'summary': new_summary, |
295
|
|
|
'from_plan': self.plan.pk, |
296
|
|
|
'product_id': self.test_run.plan.product_id, |
297
|
|
|
'do': 'clone_run', |
298
|
|
|
'orig_run_id': self.test_run.pk, |
299
|
|
|
'POSTING_TO_CREATE': 'YES', |
300
|
|
|
'product': self.test_run.plan.product_id, |
301
|
|
|
'product_version': self.test_run.product_version.pk, |
302
|
|
|
'build': self.test_run.build.pk, |
303
|
|
|
'errata_id': '', |
304
|
|
|
'manager': self.test_run.manager.email, |
305
|
|
|
'default_tester': self.test_run.default_tester.email, |
306
|
|
|
'notes': '', |
307
|
|
|
'case': [self.execution_1.case.pk, self.execution_2.case.pk], |
308
|
|
|
'case_run_id': [self.execution_1.pk, self.execution_2.pk], |
309
|
|
|
} |
310
|
|
|
|
311
|
|
|
url = reverse('testruns-new') |
312
|
|
|
response = self.client.post(url, clone_data) |
313
|
|
|
|
314
|
|
|
self.assertRedirects( |
315
|
|
|
response, |
316
|
|
|
reverse('tcms-login') + '?next=' + url) |
317
|
|
|
|
318
|
|
|
def assert_cloned_run(self, cloned_run): |
319
|
|
|
# Assert clone settings result |
320
|
|
|
for origin_case_run, cloned_case_run in zip((self.execution_1, self.execution_2), |
321
|
|
|
cloned_run.case_run.order_by('pk')): |
322
|
|
|
self.assertEqual(TestExecutionStatus.objects.get(name='IDLE'), |
323
|
|
|
cloned_case_run.status) |
324
|
|
|
self.assertEqual(origin_case_run.assignee, cloned_case_run.assignee) |
325
|
|
|
|
326
|
|
|
|
327
|
|
|
class TestSearchRuns(BaseCaseRun): |
328
|
|
|
|
329
|
|
|
@classmethod |
330
|
|
|
def setUpTestData(cls): |
331
|
|
|
super(TestSearchRuns, cls).setUpTestData() |
332
|
|
|
|
333
|
|
|
cls.search_runs_url = reverse('testruns-search') |
334
|
|
|
|
335
|
|
|
def test_search_page_is_shown(self): |
336
|
|
|
response = self.client.get(self.search_runs_url) |
337
|
|
|
self.assertContains(response, '<input id="id_summary" type="text"') |
338
|
|
|
|
339
|
|
|
def test_search_page_is_shown_with_get_parameter_used(self): |
340
|
|
|
response = self.client.get(self.search_runs_url, {'product': self.product.pk}) |
341
|
|
|
self.assertContains(response, |
342
|
|
|
'<option value="%d" selected>%s</option>' % (self.product.pk, |
343
|
|
|
self.product.name), |
344
|
|
|
html=True) |
345
|
|
|
|
346
|
|
|
|
347
|
|
|
class TestAddRemoveRunCC(BaseCaseRun): |
348
|
|
|
"""Test view tcms.testruns.views.cc""" |
349
|
|
|
|
350
|
|
|
@classmethod |
351
|
|
|
def setUpTestData(cls): |
352
|
|
|
super(TestAddRemoveRunCC, cls).setUpTestData() |
353
|
|
|
|
354
|
|
|
cls.cc_url = reverse('testruns-cc', args=[cls.test_run.pk]) |
355
|
|
|
|
356
|
|
|
cls.cc_user_1 = UserFactory(username='cc-user-1', |
357
|
|
|
email='[email protected]') |
358
|
|
|
cls.cc_user_2 = UserFactory(username='cc-user-2', |
359
|
|
|
email='[email protected]') |
360
|
|
|
cls.cc_user_3 = UserFactory(username='cc-user-3', |
361
|
|
|
email='[email protected]') |
362
|
|
|
|
363
|
|
|
cls.test_run.add_cc(cls.cc_user_2) |
364
|
|
|
cls.test_run.add_cc(cls.cc_user_3) |
365
|
|
|
|
366
|
|
|
def test_404_if_run_not_exist(self): |
367
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
368
|
|
|
cc_url = reverse('testruns-cc', args=[999999]) |
369
|
|
|
response = self.client.get(cc_url) |
370
|
|
|
self.assert404(response) |
371
|
|
|
|
372
|
|
|
def assert_cc(self, response, expected_cc): |
373
|
|
|
self.assertEqual(len(expected_cc), self.test_run.cc.count()) # pylint: disable=no-member |
374
|
|
|
|
375
|
|
|
for cc in expected_cc: |
376
|
|
|
href = reverse('tcms-profile', args=[cc.username]) |
377
|
|
|
self.assertContains( |
378
|
|
|
response, |
379
|
|
|
'<a href="%s">%s</a>' % (href, cc.username), |
380
|
|
|
html=True) |
381
|
|
|
|
382
|
|
|
def test_refuse_if_missing_action(self): |
383
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
384
|
|
|
response = self.client.get(self.cc_url, |
385
|
|
|
{'user': self.cc_user_1.username}) |
386
|
|
|
self.assert_cc(response, [self.cc_user_2, self.cc_user_3]) |
387
|
|
|
|
388
|
|
|
def test_add_cc(self): |
389
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
390
|
|
|
response = self.client.get( |
391
|
|
|
self.cc_url, |
392
|
|
|
{'do': 'add', 'user': self.cc_user_1.username}) |
393
|
|
|
|
394
|
|
|
self.assert_cc(response, |
395
|
|
|
[self.cc_user_2, self.cc_user_3, self.cc_user_1]) |
396
|
|
|
|
397
|
|
|
def test_remove_cc(self): |
398
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
399
|
|
|
response = self.client.get( |
400
|
|
|
self.cc_url, |
401
|
|
|
{'do': 'remove', 'user': self.cc_user_2.username}) |
402
|
|
|
|
403
|
|
|
self.assert_cc(response, [self.cc_user_3]) |
404
|
|
|
|
405
|
|
|
def test_refuse_to_remove_if_missing_user(self): |
406
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
407
|
|
|
response = self.client.get(self.cc_url, {'do': 'remove'}) |
408
|
|
|
|
409
|
|
|
response_text = html.unescape(str(response.content, encoding=settings.DEFAULT_CHARSET)) |
410
|
|
|
self.assertIn(str(_('The user you typed does not exist in database')), |
411
|
|
|
response_text) |
412
|
|
|
|
413
|
|
|
self.assert_cc(response, [self.cc_user_2, self.cc_user_3]) |
414
|
|
|
|
415
|
|
|
def test_refuse_to_add_if_missing_user(self): |
416
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
417
|
|
|
response = self.client.get(self.cc_url, {'do': 'add'}) |
418
|
|
|
|
419
|
|
|
response_text = html.unescape(str(response.content, encoding=settings.DEFAULT_CHARSET)) |
420
|
|
|
self.assertIn(str(_('The user you typed does not exist in database')), |
421
|
|
|
response_text) |
422
|
|
|
|
423
|
|
|
self.assert_cc(response, [self.cc_user_2, self.cc_user_3]) |
424
|
|
|
|
425
|
|
|
def test_refuse_if_user_not_exist(self): |
426
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
427
|
|
|
response = self.client.get(self.cc_url, |
428
|
|
|
{'do': 'add', 'user': 'not exist'}) |
429
|
|
|
|
430
|
|
|
response_text = html.unescape(str(response.content, encoding=settings.DEFAULT_CHARSET)) |
431
|
|
|
self.assertIn(str(_('The user you typed does not exist in database')), |
432
|
|
|
response_text) |
433
|
|
|
|
434
|
|
|
self.assert_cc(response, [self.cc_user_2, self.cc_user_3]) |
435
|
|
|
|
436
|
|
|
def test_should_not_be_able_use_cc_when_user_has_no_pemissions(self): |
437
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testrun') |
438
|
|
|
|
439
|
|
|
self.assertRedirects( |
440
|
|
|
self.client.get(self.cc_url), |
441
|
|
|
reverse('tcms-login') + '?next=%s' % self.cc_url |
442
|
|
|
) |
443
|
|
|
|
444
|
|
|
|
445
|
|
|
class TestUpdateCaseRunText(BaseCaseRun): |
446
|
|
|
"""Test update_case_run_text view method""" |
447
|
|
|
|
448
|
|
|
@classmethod |
449
|
|
|
def setUpTestData(cls): |
450
|
|
|
super().setUpTestData() |
451
|
|
|
|
452
|
|
|
cls.testruns_url = reverse('testruns-get', args=[cls.test_run.pk]) |
453
|
|
|
cls.update_url = reverse('testruns-update_case_run_text', |
454
|
|
|
args=[cls.test_run.pk]) |
455
|
|
|
|
456
|
|
|
# To increase case text version |
457
|
|
|
cls.execution_1.case.text = "Scenario Version 1" |
458
|
|
|
cls.execution_1.case.save() |
459
|
|
|
|
460
|
|
|
cls.execution_1.case.text = "Scenario Version 2" |
461
|
|
|
cls.execution_1.case.save() |
462
|
|
|
|
463
|
|
|
def test_get_update_caserun_text_with_permissions(self): |
464
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testexecution') |
465
|
|
|
response = self.client.get(self.testruns_url) |
466
|
|
|
self.assertContains(response, 'id="update_case_run_text"') |
467
|
|
|
|
468
|
|
|
def test_update_selected_case_runs_with_permissions(self): |
469
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testexecution') |
470
|
|
|
|
471
|
|
|
self.assertNotEqual(self.execution_1.case.history.latest().history_id, |
472
|
|
|
self.execution_1.case_text_version) |
473
|
|
|
|
474
|
|
|
expected_text = "%s: %s -> %s" % ( |
475
|
|
|
self.execution_1.case.summary, |
476
|
|
|
self.execution_1.case_text_version, |
477
|
|
|
self.execution_1.case.history.latest().history_id |
478
|
|
|
) |
479
|
|
|
|
480
|
|
|
response = self.client.post(self.update_url, |
481
|
|
|
{'case_run': [self.execution_1.pk]}, |
482
|
|
|
follow=True) |
483
|
|
|
|
484
|
|
|
self.assertContains(response, expected_text) |
485
|
|
|
|
486
|
|
|
self.execution_1.refresh_from_db() |
487
|
|
|
|
488
|
|
|
self.assertEqual( |
489
|
|
|
self.execution_1.case.get_text_with_version( |
490
|
|
|
self.execution_1.case_text_version |
491
|
|
|
), |
492
|
|
|
"Scenario Version 2" |
493
|
|
|
) |
494
|
|
|
self.assertEqual( |
495
|
|
|
self.execution_1.case.history.latest().history_id, |
496
|
|
|
self.execution_1.case_text_version |
497
|
|
|
) |
498
|
|
|
|
499
|
|
|
def test_get_update_caserun_text_without_permissions(self): |
500
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testexecution') |
501
|
|
|
response = self.client.get(self.testruns_url) |
502
|
|
|
self.assertNotContains(response, 'id="update_case_run_text"') |
503
|
|
|
|
504
|
|
|
def test_update_selected_case_runs_without_permissions(self): |
505
|
|
|
self.execution_1.case.text = "Scenario Version 3" |
506
|
|
|
self.execution_1.case.save() |
507
|
|
|
|
508
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testexecution') |
509
|
|
|
|
510
|
|
|
self.client.login( # nosec:B106:hardcoded_password_funcarg |
511
|
|
|
username=self.tester.username, |
512
|
|
|
password='password') |
513
|
|
|
|
514
|
|
|
self.assertNotEqual( |
515
|
|
|
self.execution_1.case.history.latest().history_id, |
516
|
|
|
self.execution_1.case_text_version |
517
|
|
|
) |
518
|
|
|
|
519
|
|
|
response = self.client.post(self.update_url, |
520
|
|
|
{'case_run': [self.execution_1.pk]}, |
521
|
|
|
follow=True) |
522
|
|
|
|
523
|
|
|
self.assertRedirects( |
524
|
|
|
response, |
525
|
|
|
reverse('tcms-login') + '?next=' + self.update_url |
526
|
|
|
) |
527
|
|
|
|
528
|
|
|
self.execution_1.refresh_from_db() |
529
|
|
|
|
530
|
|
|
self.assertNotEqual( |
531
|
|
|
self.execution_1.case.get_text_with_version( |
532
|
|
|
self.execution_1.case_text_version |
533
|
|
|
), |
534
|
|
|
"Scenario Version 3" |
535
|
|
|
) |
536
|
|
|
|
537
|
|
|
self.assertNotEqual( |
538
|
|
|
self.execution_1.case.history.latest().history_id, |
539
|
|
|
self.execution_1.case_text_version |
540
|
|
|
) |
541
|
|
|
|
542
|
|
|
|
543
|
|
|
class TestEditRun(BaseCaseRun): |
544
|
|
|
"""Test edit view method""" |
545
|
|
|
|
546
|
|
|
@classmethod |
547
|
|
|
def setUpTestData(cls): |
548
|
|
|
super(TestEditRun, cls).setUpTestData() |
549
|
|
|
|
550
|
|
|
user_should_have_perm(cls.tester, 'testruns.change_testrun') |
551
|
|
|
cls.edit_url = reverse('testruns-edit', args=[cls.test_run.pk]) |
552
|
|
|
|
553
|
|
|
cls.new_build = BuildFactory(name='FastTest', |
554
|
|
|
product=cls.test_run.plan.product) |
555
|
|
|
cls.intern = UserFactory(username='intern', |
556
|
|
|
email='[email protected]') |
557
|
|
|
|
558
|
|
|
def test_404_if_edit_non_existing_run(self): |
559
|
|
|
url = reverse('testruns-edit', args=[9999]) |
560
|
|
|
response = self.client.get(url) |
561
|
|
|
|
562
|
|
|
self.assert404(response) |
563
|
|
|
|
564
|
|
|
def test_edit_run(self): |
565
|
|
|
|
566
|
|
|
post_data = { |
567
|
|
|
'summary': 'New run summary', |
568
|
|
|
'build': self.new_build.pk, |
569
|
|
|
'manager': self.test_run.manager.email, |
570
|
|
|
'default_tester': self.intern.email, |
571
|
|
|
'notes': 'easytest', |
572
|
|
|
} |
573
|
|
|
|
574
|
|
|
response = self.client.post(self.edit_url, post_data) |
575
|
|
|
|
576
|
|
|
run = TestRun.objects.get(pk=self.test_run.pk) |
577
|
|
|
self.assertEqual('New run summary', run.summary) |
578
|
|
|
self.assertEqual(self.new_build, run.build) |
579
|
|
|
|
580
|
|
|
self.assertRedirects(response, reverse('testruns-get', args=[run.pk])) |
581
|
|
|
|
582
|
|
|
|
583
|
|
|
class TestAddCasesToRun(BaseCaseRun): |
584
|
|
|
"""Test AddCasesToRunView""" |
585
|
|
|
|
586
|
|
|
@classmethod |
587
|
|
|
def setUpTestData(cls): |
588
|
|
|
super(TestAddCasesToRun, cls).setUpTestData() |
589
|
|
|
|
590
|
|
|
cls.proposed_case = TestCaseFactory( |
591
|
|
|
author=cls.tester, |
592
|
|
|
default_tester=None, |
593
|
|
|
reviewer=cls.tester, |
594
|
|
|
case_status=cls.case_status_proposed, |
595
|
|
|
plan=[cls.plan]) |
596
|
|
|
|
597
|
|
|
user_should_have_perm(cls.tester, 'testruns.add_testexecution') |
598
|
|
|
|
599
|
|
|
def test_show_add_cases_to_run(self): |
600
|
|
|
url = reverse('add-cases-to-run', args=[self.test_run.pk]) |
601
|
|
|
response = self.client.get(url) |
602
|
|
|
|
603
|
|
|
self.assertNotContains( |
604
|
|
|
response, |
605
|
|
|
'<a href="{0}">{1}</a>'.format( |
606
|
|
|
reverse('testcases-get', |
607
|
|
|
args=[self.proposed_case.pk]), |
608
|
|
|
self.proposed_case.pk), |
609
|
|
|
html=True |
610
|
|
|
) |
611
|
|
|
|
612
|
|
|
confirmed_cases = [self.case, self.case_1, self.case_2, self.case_3] |
613
|
|
|
|
614
|
|
|
# Check selected and unselected case id checkboxes |
615
|
|
|
# cls.case is not added to cls.test_run, so it should not be checked. |
616
|
|
|
self.assertContains( |
617
|
|
|
response, |
618
|
|
|
'<td align="left">' |
619
|
|
|
'<input type="checkbox" name="case" value="{0}">' |
620
|
|
|
'</td>'.format(self.case.pk), |
621
|
|
|
html=True) |
622
|
|
|
|
623
|
|
|
# other cases are added to cls.test_run, so must be checked. |
624
|
|
|
for case in confirmed_cases[1:]: |
625
|
|
|
self.assertContains( |
626
|
|
|
response, |
627
|
|
|
'<td align="left">' |
628
|
|
|
'<input type="checkbox" name="case" value="{0}" ' |
629
|
|
|
'disabled="true" checked="true">' |
630
|
|
|
'</td>'.format(case.pk), |
631
|
|
|
html=True) |
632
|
|
|
|
633
|
|
|
# Check listed case properties |
634
|
|
|
# note: the response is ordered by 'case' |
635
|
|
|
for loop_counter, case in enumerate(confirmed_cases, 1): |
636
|
|
|
html_pieces = [ |
637
|
|
|
'<a href="{0}">{1}</a>'.format( |
638
|
|
|
reverse('testcases-get', args=[case.pk]), |
639
|
|
|
case.pk), |
640
|
|
|
|
641
|
|
|
'<td class="js-case-summary" data-param="{0}">' |
642
|
|
|
'<a id="link_{0}" class="blind_title_link" ' |
643
|
|
|
'href="javascript:void(0);">{1}</a></td>'.format(loop_counter, |
644
|
|
|
case.summary), |
645
|
|
|
|
646
|
|
|
'<td>{0}</td>'.format(case.author.username), |
647
|
|
|
'<td>{0}</td>'.format( |
648
|
|
|
formats.date_format(case.create_date, 'DATETIME_FORMAT')), |
649
|
|
|
'<td>{0}</td>'.format(case.category.name), |
650
|
|
|
'<td>{0}</td>'.format(case.priority.value), |
651
|
|
|
] |
652
|
|
|
for piece in html_pieces: |
653
|
|
|
self.assertContains(response, piece, html=True) |
654
|
|
|
|
655
|
|
|
|
656
|
|
|
class TestRunCasesMenu(BaseCaseRun): |
657
|
|
|
@classmethod |
658
|
|
|
def setUpTestData(cls): |
659
|
|
|
super().setUpTestData() |
660
|
|
|
|
661
|
|
|
cls.url = reverse('testruns-get', args=[cls.test_run.pk]) |
662
|
|
|
|
663
|
|
|
cls.add_cases_html = \ |
664
|
|
|
'<a href="{0}" class="addBlue9">{1}</a>' \ |
665
|
|
|
.format( |
666
|
|
|
reverse('add-cases-to-run', args=[cls.test_run.pk]), |
667
|
|
|
_('Add') |
668
|
|
|
) |
669
|
|
|
|
670
|
|
|
cls.remove_cases_html = \ |
671
|
|
|
'<a href="#" title="{0}" data-param="{1}" \ |
672
|
|
|
class="removeBlue9 js-del-case">{2}</a>' \ |
673
|
|
|
.format( |
674
|
|
|
_('Remove selected cases form this test run'), |
675
|
|
|
cls.test_run.pk, |
676
|
|
|
_('Remove') |
677
|
|
|
) |
678
|
|
|
|
679
|
|
|
cls.update_case_run_text_html = \ |
680
|
|
|
'<a href="#" title="{0}" \ |
681
|
|
|
href="javascript:void(0)" data-param="{1}" \ |
682
|
|
|
class="updateBlue9 js-update-case" id="update_case_run_text">{2}</a>' \ |
683
|
|
|
.format( |
684
|
|
|
_('Update the IDLE case runs to newest case text'), |
685
|
|
|
reverse('testruns-update_case_run_text', args=[cls.test_run.pk]), |
686
|
|
|
_('Update') |
687
|
|
|
) |
688
|
|
|
|
689
|
|
|
cls.change_assignee_html = \ |
690
|
|
|
'<a href="#" title="{0}" \ |
691
|
|
|
class="assigneeBlue9 js-change-assignee">{1}</a>' \ |
692
|
|
|
.format( |
693
|
|
|
_('Assign this case(s) to other people'), |
694
|
|
|
_('Assignee') |
695
|
|
|
) |
696
|
|
|
|
697
|
|
|
def test_add_cases_to_run_with_permission(self): |
698
|
|
|
user_should_have_perm(self.tester, 'testruns.add_testexecution') |
699
|
|
|
response = self.client.get(self.url) |
700
|
|
|
self.assertContains(response, self.add_cases_html, html=True) |
701
|
|
|
|
702
|
|
|
def test_remove_cases_from_run_with_permission(self): |
703
|
|
|
user_should_have_perm(self.tester, 'testruns.delete_testexecution') |
704
|
|
|
response = self.client.get(self.url) |
705
|
|
|
self.assertContains(response, self.remove_cases_html, html=True) |
706
|
|
|
|
707
|
|
|
def test_update_caserun_text_with_permission(self): |
708
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testexecution') |
709
|
|
|
response = self.client.get(self.url) |
710
|
|
|
self.assertContains(response, self.update_case_run_text_html, html=True) |
711
|
|
|
|
712
|
|
|
def test_change_assignee_with_permission(self): |
713
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testexecution') |
714
|
|
|
response = self.client.get(self.url) |
715
|
|
|
self.assertContains(response, self.change_assignee_html, html=True) |
716
|
|
|
|
717
|
|
|
def test_add_cases_to_run_without_permission(self): |
718
|
|
|
remove_perm_from_user(self.tester, 'testruns.add_testexecution') |
719
|
|
|
response = self.client.get(self.url) |
720
|
|
|
self.assertNotContains(response, self.add_cases_html, html=True) |
721
|
|
|
|
722
|
|
|
def test_remove_cases_from_run_without_permission(self): |
723
|
|
|
remove_perm_from_user(self.tester, 'testruns.delete_testexecution') |
724
|
|
|
response = self.client.get(self.url) |
725
|
|
|
self.assertNotContains(response, self.remove_cases_html, html=True) |
726
|
|
|
|
727
|
|
|
def test_update_caserun_text_without_permission(self): |
728
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testexecution') |
729
|
|
|
response = self.client.get(self.url) |
730
|
|
|
self.assertNotContains(response, self.update_case_run_text_html, html=True) |
731
|
|
|
|
732
|
|
|
def test_change_assignee_without_permission(self): |
733
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testexecution') |
734
|
|
|
response = self.client.get(self.url) |
735
|
|
|
self.assertNotContains(response, self.change_assignee_html, html=True) |
736
|
|
|
|
737
|
|
|
|
738
|
|
|
class TestRunStatusMenu(BaseCaseRun): |
739
|
|
|
@classmethod |
740
|
|
|
def setUpTestData(cls): |
741
|
|
|
super().setUpTestData() |
742
|
|
|
cls.url = reverse('testruns-get', args=[cls.test_run.pk]) |
743
|
|
|
cls.status_menu_html = [] |
744
|
|
|
|
745
|
|
|
for tcrs in TestExecutionStatus.objects.all(): |
746
|
|
|
cls.status_menu_html.append( |
747
|
|
|
'<a value="{0}" href="#" class="{1}Blue9">{2}</a>' |
748
|
|
|
.format(tcrs.pk, tcrs.name.lower(), tcrs.name) |
749
|
|
|
) |
750
|
|
|
|
751
|
|
|
def test_get_status_options_with_permission(self): |
752
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testexecution') |
753
|
|
|
response = self.client.get(self.url) |
754
|
|
|
self.assertEqual(HTTPStatus.OK, response.status_code) |
755
|
|
|
|
756
|
|
|
for html_code in self.status_menu_html: |
757
|
|
|
self.assertContains(response, html_code, html=True) |
758
|
|
|
|
759
|
|
|
def test_get_status_options_without_permission(self): |
760
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testexecution') |
761
|
|
|
response = self.client.get(self.url) |
762
|
|
|
self.assertEqual(HTTPStatus.OK, response.status_code) |
763
|
|
|
|
764
|
|
|
for tcrs in TestExecutionStatus.objects.all(): |
765
|
|
|
self.assertNotContains(response, self.status_menu_html, html=True) |
766
|
|
|
|
767
|
|
|
|
768
|
|
|
class TestExecutionComments(BaseCaseRun): |
769
|
|
|
@classmethod |
770
|
|
|
def setUpTestData(cls): |
771
|
|
|
super().setUpTestData() |
772
|
|
|
cls.url = reverse('testruns-get', args=[cls.test_run.pk]) |
773
|
|
|
|
774
|
|
|
cls.add_comment_html = \ |
775
|
|
|
'<a href="#" class="addBlue9 js-show-commentdialog">{0}</a>' \ |
776
|
|
|
.format(_('Add')) |
777
|
|
|
|
778
|
|
|
def test_get_add_comment_with_permission(self): |
779
|
|
|
user_should_have_perm(self.tester, 'django_comments.add_comment') |
780
|
|
|
response = self.client.get(self.url) |
781
|
|
|
self.assertContains(response, self.add_comment_html, html=True) |
782
|
|
|
|
783
|
|
|
def test_get_add_comment_without_permission(self): |
784
|
|
|
remove_perm_from_user(self.tester, 'django_comments.add_comment') |
785
|
|
|
response = self.client.get(self.url) |
786
|
|
|
self.assertNotContains(response, self.add_comment_html, html=True) |
787
|
|
|
|
788
|
|
|
|
789
|
|
|
class TestChangeTestRunStatus(BaseCaseRun): |
790
|
|
|
|
791
|
|
|
@classmethod |
792
|
|
|
def setUpTestData(cls): |
793
|
|
|
super().setUpTestData() |
794
|
|
|
cls.url = reverse('testruns-change_status', args=[cls.test_run.pk]) |
795
|
|
|
|
796
|
|
|
def test_change_status_to_finished(self): |
797
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
798
|
|
|
response = self.client.get(self.url, {'finished': 1}) |
799
|
|
|
self.assertRedirects( |
800
|
|
|
response, |
801
|
|
|
reverse('testruns-get', args=[self.test_run.pk])) |
802
|
|
|
|
803
|
|
|
self.test_run.refresh_from_db() |
804
|
|
|
self.assertIsNotNone(self.test_run.stop_date) |
805
|
|
|
|
806
|
|
|
def test_change_status_to_running(self): |
807
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
808
|
|
|
response = self.client.get(self.url, {'finished': 0}) |
809
|
|
|
|
810
|
|
|
self.assertRedirects( |
811
|
|
|
response, |
812
|
|
|
reverse('testruns-get', args=[self.test_run.pk])) |
813
|
|
|
|
814
|
|
|
self.test_run.refresh_from_db() |
815
|
|
|
self.assertIsNone(self.test_run.stop_date) |
816
|
|
|
|
817
|
|
|
def test_should_throw_404_on_non_existing_testrun(self): |
818
|
|
|
user_should_have_perm(self.tester, 'testruns.change_testrun') |
819
|
|
|
response = self.client.get(reverse('testruns-change_status', args=[99999]), {'finished': 0}) |
820
|
|
|
self.assertEqual(HTTPStatus.NOT_FOUND, response.status_code) |
821
|
|
|
|
822
|
|
|
def test_should_fail_when_try_to_change_status_without_permissions(self): |
823
|
|
|
remove_perm_from_user(self.tester, 'testruns.change_testrun') |
824
|
|
|
self.assertRedirects( |
825
|
|
|
self.client.get(self.url, {'finished': 1}), |
826
|
|
|
reverse('tcms-login') + '?next=%s?finished=1' % self.url) |
827
|
|
|
|