1
|
|
|
''' |
2
|
|
|
Test cases for the executor code. |
3
|
|
|
|
4
|
|
|
Note the neccessary python search path manipulation below. |
5
|
|
|
|
6
|
|
|
Missing tests: |
7
|
|
|
|
8
|
|
|
- Makefile in validator package should override student Makefile |
9
|
|
|
|
10
|
|
|
''' |
11
|
|
|
|
12
|
|
|
import os |
13
|
|
|
import os.path |
14
|
|
|
import sys |
15
|
|
|
import logging |
16
|
|
|
|
17
|
|
|
from django.core import mail |
18
|
|
|
from django.conf import settings |
19
|
|
|
from django.test import TestCase |
20
|
|
|
from django.test.utils import override_settings |
21
|
|
|
from opensubmit.tests.cases import SubmitStudentScenarioTestCase |
22
|
|
|
from django.core.urlresolvers import reverse |
23
|
|
|
|
24
|
|
|
from opensubmit.models import TestMachine, SubmissionTestResult, Submission |
25
|
|
|
from opensubmit.tests import utils |
26
|
|
|
|
27
|
|
|
from .helpers.submission import create_validatable_submission |
28
|
|
|
from .helpers.submission import create_validated_submission |
29
|
|
|
from .helpers.assignment import create_validated_assignment |
30
|
|
|
from .helpers.assignment import create_pass_fail_grading |
31
|
|
|
from .helpers.djangofiles import create_submission_file |
32
|
|
|
from .helpers.user import create_user, get_student_dict |
33
|
|
|
from . import uccrap, rootdir |
34
|
|
|
|
35
|
|
|
sys.path.insert(0, os.path.dirname(__file__) + '/../../../executor/') |
36
|
|
|
from opensubmitexec import config, cmdline, server, locking, compiler, exceptions # NOQA |
37
|
|
|
|
38
|
|
|
logger = logging.getLogger('opensubmitexec') |
39
|
|
|
|
40
|
|
|
|
41
|
|
|
|
42
|
|
|
class CmdLine(TestCase): |
43
|
|
|
''' |
44
|
|
|
Test cases for the executor command-line script. |
45
|
|
|
''' |
46
|
|
|
def setUp(self): |
47
|
|
|
self.config = config.read_config( |
48
|
|
|
os.path.dirname(__file__) + "/executor.cfg") |
49
|
|
|
|
50
|
|
|
def test_help_call_without_config(self): |
51
|
|
|
# simulate 'help' call |
52
|
|
|
sys.argv = ['opensubmit-exec', 'help'] |
53
|
|
|
cmdline.console_script() |
54
|
|
|
|
55
|
|
|
def test_unlock_call_without_lock(self): |
56
|
|
|
# simulate 'unlock' call without existing lock |
57
|
|
|
sys.argv = ['opensubmit-exec', 'unlock'] |
58
|
|
|
cmdline.console_script() |
59
|
|
|
|
60
|
|
|
def test_unlock_call_with_lock(self): |
61
|
|
|
# simulate 'unlock' call with existing lock |
62
|
|
|
sys.argv = ['opensubmit-exec', 'unlock'] |
63
|
|
|
with locking.ScriptLock(self.config): |
64
|
|
|
cmdline.console_script() |
65
|
|
|
|
66
|
|
|
|
67
|
|
|
class Library(SubmitStudentScenarioTestCase): |
68
|
|
|
''' |
69
|
|
|
Tests for the executor library functions used by the validator script. |
70
|
|
|
''' |
71
|
|
|
def setUp(self): |
72
|
|
|
settings.MAIN_URL = self.live_server_url |
73
|
|
|
super(Library, self).setUp() |
74
|
|
|
self.config = config.read_config( |
75
|
|
|
os.path.dirname(__file__) + "/executor.cfg", |
76
|
|
|
override_url=self.live_server_url) |
77
|
|
|
|
78
|
|
|
def _register_executor(self): |
79
|
|
|
server.send_hostinfo(self.config) |
80
|
|
|
return TestMachine.objects.order_by('-last_contact')[0] |
81
|
|
|
|
82
|
|
|
def _run_executor(self): |
83
|
|
|
return cmdline.download_and_run(self.config) |
84
|
|
|
|
85
|
|
|
def test_send_result(self): |
86
|
|
|
sf = create_submission_file() |
87
|
|
|
sub = create_validatable_submission( |
88
|
|
|
self.user, self.validated_assignment, sf) |
89
|
|
|
test_machine = self._register_executor() |
90
|
|
|
sub.assignment.test_machines.add(test_machine) |
91
|
|
|
|
92
|
|
|
job = server.fetch_job(self.config) |
93
|
|
|
self.assertNotEquals(None, job) |
94
|
|
|
|
95
|
|
|
msg = uccrap + "Message from validation script." |
96
|
|
|
|
97
|
|
|
job.send_pass_result(info_student=msg) |
98
|
|
|
|
99
|
|
|
db_entries = SubmissionTestResult.objects.filter( |
100
|
|
|
kind=SubmissionTestResult.VALIDITY_TEST) |
101
|
|
|
|
102
|
|
|
self.assertEqual(db_entries[0].result, msg) |
103
|
|
|
|
104
|
|
|
def test_fetch_job_renaming(self): |
105
|
|
|
test_machine = self._register_executor() |
106
|
|
|
self.validated_assignment.test_machines.add(test_machine) |
107
|
|
|
with open(rootdir + "submfiles/validation/1000fff/helloworld.c", 'rb') as f: |
108
|
|
|
self.c.post('/assignments/%s/new/' % self.validated_assignment.pk, { |
109
|
|
|
'notes': 'This is a test submission.', |
110
|
|
|
'authors': str(self.user.pk), |
111
|
|
|
'attachment': f |
112
|
|
|
}) |
113
|
|
|
job = server.fetch_job(self.config) |
114
|
|
|
self.assertIn('helloworld.c', os.listdir(job.working_dir)) |
115
|
|
|
|
116
|
|
|
def test_job_attributes(self): |
117
|
|
|
sf = create_submission_file() |
118
|
|
|
sub = create_validatable_submission( |
119
|
|
|
self.user, self.validated_assignment, sf) |
120
|
|
|
test_machine = self._register_executor() |
121
|
|
|
sub.assignment.test_machines.add(test_machine) |
122
|
|
|
job = server.fetch_job(self.config) |
123
|
|
|
self.assertNotEquals(None, job) |
124
|
|
|
self.assertEquals(job.timeout, self.validated_assignment.attachment_test_timeout) |
125
|
|
|
self.assertEquals(job.sub_id, str(sub.pk)) |
126
|
|
|
self.assertEquals(job.file_id, str(sf.pk)) |
127
|
|
|
self.assertEquals(job.submitter_name, sub.submitter.get_full_name()) |
128
|
|
|
self.assertEquals(job.submitter_student_id, str(sub.submitter.profile.student_id)) |
129
|
|
|
self.assertEquals(job.submitter_studyprogram, str(sub.submitter.profile.study_program)) |
130
|
|
|
self.assertEquals(job.course, str(self.course)) |
131
|
|
|
self.assertEquals(job.assignment, str(self.validated_assignment)) |
132
|
|
|
|
133
|
|
|
def test_wrong_compile_call(self): |
134
|
|
|
sf = create_submission_file() |
135
|
|
|
sub = create_validatable_submission( |
136
|
|
|
self.user, self.validated_assignment, sf) |
137
|
|
|
test_machine = self._register_executor() |
138
|
|
|
sub.assignment.test_machines.add(test_machine) |
139
|
|
|
job = server.fetch_job(self.config) |
140
|
|
|
# Missing output |
141
|
|
|
self.assertRaises(exceptions.ValidatorBrokenException, job.run_compiler, inputs=['input.c']) |
142
|
|
|
# Missing input |
143
|
|
|
self.assertRaises(exceptions.ValidatorBrokenException, job.run_compiler, output='output') |
144
|
|
|
# Wrong file, determined by compiler |
145
|
|
|
self.assertRaises(exceptions.WrongExitStatusException, job.run_compiler, inputs=['foo.c'], output='output') |
146
|
|
|
|
147
|
|
|
|
148
|
|
|
class Validation(TestCase): |
149
|
|
|
''' |
150
|
|
|
Tests for the execution of validation scripts by the executor. |
151
|
|
|
''' |
152
|
|
|
|
153
|
|
|
def setUp(self): |
154
|
|
|
super(Validation, self).setUp() |
155
|
|
|
self.config = config.read_config( |
156
|
|
|
os.path.dirname(__file__) + "/executor.cfg") |
157
|
|
|
|
158
|
|
|
def _test_validation_case(self, directory): |
159
|
|
|
''' |
160
|
|
|
Each of the validator.py files uses the Python assert() |
161
|
|
|
statement to check by itself if the result is the expected |
162
|
|
|
one for its case. |
163
|
|
|
''' |
164
|
|
|
base_dir = os.path.dirname(__file__) + '/submfiles/validation/' |
165
|
|
|
cmdline.copy_and_run(self.config, base_dir + directory) |
166
|
|
|
|
167
|
|
|
def test_0100fff(self): |
168
|
|
|
self._test_validation_case('0100fff') |
169
|
|
|
|
170
|
|
|
def test_0100tff(self): |
171
|
|
|
self._test_validation_case('0100tff') |
172
|
|
|
|
173
|
|
|
def test_0100ttf(self): |
174
|
|
|
self._test_validation_case('0100ttf') |
175
|
|
|
|
176
|
|
|
def test_1000fff(self): |
177
|
|
|
self._test_validation_case('1000fff') |
178
|
|
|
|
179
|
|
|
def test_1000fft(self): |
180
|
|
|
self._test_validation_case('1000fft') |
181
|
|
|
|
182
|
|
|
def test_1000tff(self): |
183
|
|
|
self._test_validation_case('1000tff') |
184
|
|
|
|
185
|
|
|
def test_1000tft(self): |
186
|
|
|
self._test_validation_case('1000tft') |
187
|
|
|
|
188
|
|
|
def test_1000ttf(self): |
189
|
|
|
self._test_validation_case('1000ttf') |
190
|
|
|
|
191
|
|
|
def test_1000ttt(self): |
192
|
|
|
self._test_validation_case('1000ttt') |
193
|
|
|
|
194
|
|
|
def test_1010tff(self): |
195
|
|
|
self._test_validation_case('1010tff') |
196
|
|
|
|
197
|
|
|
def test_1010ttf(self): |
198
|
|
|
self._test_validation_case('1010ttf') |
199
|
|
|
|
200
|
|
|
def test_1100tff(self): |
201
|
|
|
self._test_validation_case('1100tff') |
202
|
|
|
|
203
|
|
|
def test_1100ttf(self): |
204
|
|
|
self._test_validation_case('1100ttf') |
205
|
|
|
|
206
|
|
|
def test_3000tff(self): |
207
|
|
|
self._test_validation_case('3000tff') |
208
|
|
|
|
209
|
|
|
def test_3000ttf(self): |
210
|
|
|
self._test_validation_case('3000ttf') |
211
|
|
|
|
212
|
|
|
def test_3010tff(self): |
213
|
|
|
self._test_validation_case('3010tff') |
214
|
|
|
|
215
|
|
|
def test_3010ttf(self): |
216
|
|
|
self._test_validation_case('3010ttf') |
217
|
|
|
|
218
|
|
|
def test_b000tff(self): |
219
|
|
|
self._test_validation_case('b000tff') |
220
|
|
|
|
221
|
|
|
def test_b010tff(self): |
222
|
|
|
self._test_validation_case('b010tff') |
223
|
|
|
|
224
|
|
|
|
225
|
|
|
class Communication(SubmitStudentScenarioTestCase): |
226
|
|
|
''' |
227
|
|
|
Tests for the communication of the executor with the OpenSubmit server. |
228
|
|
|
''' |
229
|
|
|
|
230
|
|
|
def setUp(self): |
231
|
|
|
settings.MAIN_URL = self.live_server_url |
232
|
|
|
super(Communication, self).setUp() |
233
|
|
|
self.config = config.read_config( |
234
|
|
|
os.path.dirname(__file__) + "/executor.cfg", |
235
|
|
|
override_url=self.live_server_url) |
236
|
|
|
|
237
|
|
|
def _register_executor(self): |
238
|
|
|
server.send_hostinfo(self.config) |
239
|
|
|
return TestMachine.objects.order_by('-last_contact')[0] |
240
|
|
|
|
241
|
|
|
def _run_executor(self): |
242
|
|
|
return cmdline.download_and_run(self.config) |
243
|
|
|
|
244
|
|
|
def _register_test_machine(self): |
245
|
|
|
''' |
246
|
|
|
Utility step for a common test case preparation: |
247
|
|
|
- Create validatable submission |
248
|
|
|
- Register a test machine for it |
249
|
|
|
''' |
250
|
|
|
sf = create_submission_file() |
251
|
|
|
sub = create_validatable_submission( |
252
|
|
|
self.user, self.validated_assignment, sf) |
253
|
|
|
test_machine = self._register_executor() |
254
|
|
|
sub.assignment.test_machines.add(test_machine) |
255
|
|
|
return sub |
256
|
|
|
|
257
|
|
|
def test_register_executor_explicit(self): |
258
|
|
|
machine_count = TestMachine.objects.all().count() |
259
|
|
View Code Duplication |
assert(self._register_executor().pk) |
|
|
|
|
260
|
|
|
self.assertEqual(machine_count + 1, TestMachine.objects.all().count()) |
261
|
|
|
|
262
|
|
|
@override_settings(JOB_EXECUTOR_SECRET='foo') |
263
|
|
|
def test_invalid_secret(self): |
264
|
|
|
self.assertNotEqual(True, self._run_executor()) |
265
|
|
|
|
266
|
|
|
def test_everything_already_tested(self): |
267
|
|
|
create_validated_submission(self.user, self.validated_assignment) |
268
|
|
|
assert(self._register_executor().pk) |
269
|
|
|
self.assertEqual(False, self._run_executor()) |
270
|
|
|
|
271
|
|
|
def test_parallel_executors_test(self): |
272
|
|
|
NUM_PARALLEL = 3 |
273
|
|
|
self.validated_assignment.test_machines.add(self._register_executor()) |
274
|
|
|
subs = [] |
275
|
|
|
for i in range(1, NUM_PARALLEL + 1): |
276
|
|
|
stud = create_user(get_student_dict(i)) |
277
|
|
|
self.course.participants.add(stud.profile) |
278
|
|
|
self.course.save() |
279
|
|
View Code Duplication |
sf = create_submission_file() |
|
|
|
|
280
|
|
|
subs.append(create_validatable_submission( |
281
|
|
|
stud, self.validated_assignment, sf)) |
282
|
|
|
|
283
|
|
|
# Span a number of threads, each triggering the executor |
284
|
|
|
# This only creates a real test case if executor serialization |
285
|
|
|
# is off (see tests/executor.cfg) |
286
|
|
|
return_codes = utils.run_parallel(len(subs), self._run_executor) |
287
|
|
|
self.assertEqual( |
288
|
|
|
len(list(filter((lambda x: x is True), return_codes))), |
289
|
|
|
len(subs)) |
290
|
|
|
|
291
|
|
|
for sub in subs: |
292
|
|
|
results = SubmissionTestResult.objects.filter( |
293
|
|
|
kind=SubmissionTestResult.VALIDITY_TEST |
294
|
|
|
) |
295
|
|
|
self.assertEqual(NUM_PARALLEL, len(results)) |
296
|
|
|
self.assertNotEqual(0, len(results[0].result)) |
297
|
|
|
|
298
|
|
|
def test_too_long_validation(self): |
299
|
|
|
grading = create_pass_fail_grading() |
300
|
|
|
assignment = create_validated_assignment( |
301
|
|
|
self.course, grading, "/submfiles/validation/d000fff/", "validator_run.py") |
302
|
|
|
assignment.attachment_test_timeout = 1 |
303
|
|
|
assignment.save() |
304
|
|
|
sf = create_submission_file("/submfiles/validation/d000fff/helloworld.c") |
305
|
|
|
sub = create_validatable_submission( |
306
|
|
|
self.user, assignment, sf) |
307
|
|
|
test_machine = self._register_executor() |
308
|
|
|
sub.assignment.test_machines.add(test_machine) |
309
|
|
|
|
310
|
|
|
# Fire up the executor, should mark the submission as timed out |
311
|
|
|
self.assertEqual(True, self._run_executor()) |
312
|
|
|
# Check if timeout marking took place |
313
|
|
|
sub.refresh_from_db() |
314
|
|
|
self.assertEqual(sub.state, Submission.TEST_VALIDITY_FAILED) |
315
|
|
|
text = sub.get_validation_result().result |
316
|
|
|
self.assertIn("took too long", text) |
317
|
|
|
|
318
|
|
|
def test_too_long_full_test(self): |
319
|
|
|
grading = create_pass_fail_grading() |
320
|
|
|
assignment = create_validated_assignment( |
321
|
|
|
self.course, |
322
|
|
|
grading, |
323
|
|
|
"/submfiles/validation/d000fff/", |
324
|
|
|
"validator_build.py", |
325
|
|
|
"validator_run.py") |
326
|
|
|
assignment.attachment_test_timeout = 1 |
327
|
|
|
assignment.save() |
328
|
|
|
sf = create_submission_file("/submfiles/validation/d000fff/helloworld.c") |
329
|
|
|
sub = create_validatable_submission( |
330
|
|
|
self.user, assignment, sf) |
331
|
|
|
test_machine = self._register_executor() |
332
|
|
|
sub.assignment.test_machines.add(test_machine) |
333
|
|
|
|
334
|
|
|
# Fire up the executor for validation |
335
|
|
|
self.assertEqual(True, self._run_executor()) |
336
|
|
|
# Fire up the executor for full test |
337
|
|
|
self.assertEqual(True, self._run_executor()) |
338
|
|
|
# Check if timeout marking took place |
339
|
|
|
sub.refresh_from_db() |
340
|
|
|
self.assertEqual(sub.state, Submission.TEST_FULL_FAILED) |
341
|
|
|
assert("took too long" in sub.get_fulltest_result().result) |
342
|
|
|
|
343
|
|
|
def test_single_file_validator_test(self): |
344
|
|
|
sub = self._register_test_machine() |
345
|
|
|
|
346
|
|
|
self.assertEqual(True, self._run_executor()) |
347
|
|
|
results = SubmissionTestResult.objects.filter( |
348
|
|
|
submission_file=sub.file_upload, |
349
|
|
|
kind=SubmissionTestResult.VALIDITY_TEST |
350
|
|
|
) |
351
|
|
|
self.assertEqual(1, len(results)) |
352
|
|
|
|
353
|
|
|
self.assertEqual(True, self._run_executor()) |
354
|
|
|
results = SubmissionTestResult.objects.filter( |
355
|
|
|
submission_file=sub.file_upload, |
356
|
|
|
kind=SubmissionTestResult.VALIDITY_TEST |
357
|
|
|
) |
358
|
|
|
self.assertEqual(1, len(results)) |
359
|
|
|
self.assertNotEqual(0, len(results[0].result)) |
360
|
|
|
|
361
|
|
|
def test_full_test(self): |
362
|
|
|
sub = self._register_test_machine() |
363
|
|
|
# validation test |
364
|
|
|
self.assertEqual(True, self._run_executor()) |
365
|
|
|
# full test |
366
|
|
|
self.assertEqual(True, self._run_executor()) |
367
|
|
|
results = SubmissionTestResult.objects.filter( |
368
|
|
|
submission_file=sub.file_upload, |
369
|
|
|
kind=SubmissionTestResult.FULL_TEST |
370
|
|
|
) |
371
|
|
|
self.assertEqual(1, len(results)) |
372
|
|
|
self.assertNotEqual(0, len(results[0].result)) |
373
|
|
|
|
374
|
|
|
def test_inconsistent_state_email(self): |
375
|
|
|
''' |
376
|
|
|
Test operator email on inconsistent state. |
377
|
|
|
Since executor execution and submission sending is one step, |
378
|
|
|
we need to mock the incoming invalid executor request. |
379
|
|
|
''' |
380
|
|
|
sf = create_submission_file() |
381
|
|
|
self.sub = create_validatable_submission( |
382
|
|
|
self.user, self.validated_assignment, sf) |
383
|
|
|
test_machine = self._register_executor() |
384
|
|
|
self.sub.assignment.test_machines.add(test_machine) |
385
|
|
|
self.sub.state = Submission.TEST_FULL_PENDING |
386
|
|
|
self.sub.save() |
387
|
|
|
post_data = {'Secret': settings.JOB_EXECUTOR_SECRET, |
388
|
|
|
'UUID': test_machine.host, |
389
|
|
|
'Action': 'test_compile', |
390
|
|
|
'SubmissionFileId': self.sub.file_upload.pk, |
391
|
|
|
'PerfData': '', |
392
|
|
|
'ErrorCode': 0, |
393
|
|
|
'Message': 'In A Bottle'} |
394
|
|
|
self.c.post(reverse('jobs'), post_data) |
395
|
|
|
self.assertNotEqual(len(mail.outbox), 0) |
396
|
|
|
self.assertIn('Action reported by the executor: test_compile', |
397
|
|
|
mail.outbox[0].message().as_string()) |
398
|
|
|
|
399
|
|
|
def test_assignment_specific_test_machine(self): |
400
|
|
|
# Register two test machines T1 and T2 |
401
|
|
|
real_machine = self._register_executor() |
402
|
|
|
fake_machine = TestMachine(host="127.0.0.2") |
403
|
|
|
fake_machine.save() |
404
|
|
|
# Assign each of them to a different assignment |
405
|
|
|
self.open_assignment.test_machines.add(real_machine) |
406
|
|
|
self.validated_assignment.test_machines.add(fake_machine) |
407
|
|
|
# Produce submission for the assignment linked to fake_machine |
408
|
|
|
sub1 = Submission( |
409
|
|
|
assignment=self.validated_assignment, |
410
|
|
|
submitter=self.user, |
411
|
|
|
state=Submission.TEST_VALIDITY_PENDING, |
412
|
|
|
file_upload=create_submission_file() |
413
|
|
|
) |
414
|
|
|
sub1.save() |
415
|
|
|
# Run real_machine executor, should not react on this submission |
416
|
|
|
old_sub1_state = sub1.state |
417
|
|
|
self.assertEqual(False, self._run_executor()) |
418
|
|
|
# Make sure that submission object was not touched, |
419
|
|
|
# whatever the executor says |
420
|
|
|
sub1 = Submission.objects.get(pk=sub1.pk) |
421
|
|
|
self.assertEqual(old_sub1_state, sub1.state) |
422
|
|
|
|