Newer
Older
participation=self.ta_participation, points=3)))
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(self.student_participation, self.gopp)
self.assertResponseContextEqual(resp, "avg_grade_percentage", 6)
self.assertResponseContextEqual(resp, "avg_grade_population", 1)
temp_ptcp = factories.ParticipationFactory.create(
course=self.course)
factories.GradeChangeFactory.create(
**(self.gc(participation=temp_ptcp, points=3)))
with self.temporarily_switch_to_user(temp_ptcp.user):
resp = self.get_view_single_grade(temp_ptcp, self.gopp)
self.assertResponseContextEqual(resp, "avg_grade_percentage", 4.5)
self.assertResponseContextEqual(resp, "avg_grade_population", 2)
def test_append_gc(self):
self.use_default_setup()
self.append_gc(self.gc(points=8, flow_session=self.session2))
self.assertGradeChangeMachineReadableStateEqual(8)
self.assertGradeChangeStateEqual("8.0% (/3)")
self.append_gc(self.gc(points=0, flow_session=self.session2))
self.assertGradeChangeMachineReadableStateEqual(0)
self.assertGradeChangeStateEqual("0.0% (/3)")
def test_update_latest_gc_of_latest_finished_session(self):
self.use_default_setup()
self.assertGradeChangeMachineReadableStateEqual(6)
self.update_gc(self.gc_session2, points=10)
self.assertGradeChangeMachineReadableStateEqual(10)
self.assertGradeChangeStateEqual("10.0% (/3)")
def test_update_ealiest_gc_of_ealier_finished_session(self):
self.use_default_setup()
self.assertGradeChangeMachineReadableStateEqual(6)
self.update_gc(self.gc_main_2, update_time=False, points=15)
self.assertGradeChangeMachineReadableStateEqual(6)
self.assertGradeChangeStateEqual("6.0% (/3)")
1045
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
1066
1067
1068
1069
# TODO: Is it a bug? percentage of GradeChanges without attempt_id are
# put at the begining of the valid_percentages list.
# Uncomment the following to see the failure
# self.use_default_setup()
# self.assertGradeChangeMachineReadableStateEqual(6)
# print(self.gc_main_1.grade_time)
#
# self.time_increment()
# create a gc without attempt_id
gc = factories.GradeChangeFactory.create( # noqa
**(self.gc(points=8.5, null_attempt_id=True))) # noqa
# print(gc.grade_time)
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual(8.5)
self.assertEqual(machine.valid_percentages, [8.5])
def test_gc_unavailable(self):
factories.GradeChangeFactory.create(**(self.gc(points=9.1)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.unavailable)))
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual("OTHER_STATE")
self.assertGradeChangeStateEqual("(other state)")
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(self.student_participation, self.gopp)
self.assertResponseContextEqual(resp, "avg_grade_percentage", None)
self.assertResponseContextEqual(resp, "avg_grade_population", 0)
# failure when unavailable gc follows another grade change
factories.GradeChangeFactory.create(**(self.gc(points=5)))
with self.assertRaises(ValueError) as e:
self.get_gc_stringify_machine_readable_state()
self.assertIn("cannot accept grade once opportunity has been "
"marked 'unavailable'", e.exception)
def test_gc_exempt(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.exempt)))
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual("EXEMPT")
self.assertGradeChangeStateEqual("(exempt)")
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(self.student_participation, self.gopp)
self.assertResponseContextEqual(resp, "avg_grade_percentage", None)
self.assertResponseContextEqual(resp, "avg_grade_population", 0)
# failure when exempt gc follows another grade change
factories.GradeChangeFactory.create(**(self.gc(points=5)))
with self.assertRaises(ValueError) as e:
self.get_gc_stringify_machine_readable_state()
self.assertIn("cannot accept grade once opportunity has been "
"marked 'exempt'", e.exception)
def test_gc_do_over(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
# This creates a GradeChange object with no attempt_id
factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.do_over,
null_attempt_id=True)))
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual("NONE")
self.assertGradeChangeStateEqual("- ∅ -")
# This make sure new grade change objects following do_over gc is
# consumed without problem
factories.GradeChangeFactory.create(**(self.gc(points=5)))
self.assertGradeChangeMachineReadableStateEqual("5")
machine = self.get_gc_machine()
self.assertEqual(machine.valid_percentages, [5])
self.assertGradeChangeStateEqual("5.0%")
def test_gc_do_over_average_grade_value(self):
self.use_default_setup()
factories.GradeChangeFactory.create(
**(self.gc(points=None, state=g_state.do_over,
flow_session=self.session2)))
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(self.student_participation, self.gopp)
self.assertResponseContextEqual(resp, "avg_grade_percentage", None)
self.assertResponseContextEqual(resp, "avg_grade_population", 0)
def test_gc_report_sent(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
gc2 = factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.report_sent)))
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual("6")
self.assertGradeChangeStateEqual("6.0%")
self.assertEqual(machine.last_report_time, gc2.grade_time)
def test_gc_extension(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
gc2 = factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.extension,
due_time=self.time + timedelta(days=1))))
machine = self.get_gc_machine()
self.assertGradeChangeMachineReadableStateEqual("6")
self.assertGradeChangeStateEqual("6.0%")
self.assertEqual(machine.due_time, gc2.due_time)
def test_gc_grading_started(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.grading_started)))
self.assertGradeChangeMachineReadableStateEqual("6")
self.assertGradeChangeStateEqual("6.0%")
def test_gc_retrieved(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, state=g_state.retrieved)))
self.assertGradeChangeMachineReadableStateEqual("6")
self.assertGradeChangeStateEqual("6.0%")
def test_gc_non_exist_state(self):
factories.GradeChangeFactory.create(**(self.gc(points=6)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, state="some_state")))
self.get_gc_stringify_machine_readable_state()
def test_gc_non_point(self):
factories.GradeChangeFactory.create(**(self.gc(points=None)))
self.assertGradeChangeMachineReadableStateEqual("NONE")
self.assertGradeChangeStateEqual("- ∅ -")
1187
1188
1189
1190
1191
1192
1193
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
1220
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232
1233
1234
1235
1236
1237
1238
1239
1240
1241
1242
1243
1244
1245
1246
1247
1248
1249
1250
1251
1252
1253
1254
1255
1256
1257
1258
1259
1260
1261
1262
1263
1264
1265
1266
1267
1268
1269
1270
1271
1272
1273
1274
1275
1276
1277
1278
1279
1280
1281
1282
1283
1284
1285
1286
1287
1288
1289
1290
1291
1292
1293
1294
1295
1296
1297
1298
1299
1300
1301
1302
1303
1304
1305
1306
1307
1308
1309
1310
1311
1312
1313
1314
1315
1316
1317
1318
1319
1320
1321
1322
1323
1324
1325
1326
1327
1328
1329
1330
1331
1332
1333
1334
1335
1336
1337
1338
1339
1340
1341
1342
1343
1344
1345
1346
1347
1348
1349
1350
1351
1352
1353
1354
1355
1356
1357
1358
1359
1360
1361
1362
1363
1364
1365
1366
1367
1368
1369
1370
1371
1372
1373
1374
1375
1376
1377
1378
1379
1380
1381
1382
1383
1384
1385
1386
1387
1388
1389
1390
1391
1392
1393
1394
1395
1396
1397
1398
1399
1400
1401
1402
1403
1404
1405
1406
1407
1408
1409
1410
1411
1412
1413
1414
1415
1416
1417
1418
1419
1420
1421
1422
1423
1424
1425
1426
1427
1428
1429
1430
1431
1432
1433
1434
1435
1436
1437
1438
1439
1440
1441
1442
1443
1444
1445
1446
1447
1448
1449
1450
1451
1452
1453
1454
1455
1456
1457
1458
1459
1460
1461
1462
1463
1464
1465
1466
1467
1468
1469
1470
1471
1472
1473
1474
1475
1476
1477
1478
1479
1480
1481
1482
1483
1484
1485
1486
1487
1488
1489
1490
1491
1492
1493
1494
1495
1496
1497
1498
1499
1500
1501
1502
1503
1504
1505
1506
1507
1508
1509
1510
1511
1512
1513
1514
1515
1516
1517
1518
1519
1520
1521
1522
1523
1524
1525
1526
1527
1528
1529
1530
1531
1532
1533
1534
1535
1536
1537
1538
1539
1540
1541
1542
1543
1544
1545
1546
1547
1548
1549
1550
1551
1552
1553
1554
1555
1556
1557
1558
1559
1560
1561
1562
1563
1564
1565
1566
1567
1568
1569
1570
1571
1572
1573
1574
1575
1576
1577
1578
1579
1580
1581
1582
1583
1584
1585
1586
1587
1588
1589
1590
1591
1592
1593
1594
1595
1596
1597
1598
1599
1600
1601
1602
1603
1604
1605
1606
1607
1608
1609
1610
1611
1612
1613
1614
1615
1616
1617
1618
1619
1620
1621
1622
1623
1624
1625
1626
1627
1628
1629
1630
1631
1632
1633
1634
1635
1636
1637
1638
1639
1640
1641
1642
1643
1644
1645
1646
1647
1648
1649
1650
1651
1652
1653
1654
1655
1656
1657
1658
1659
1660
1661
1662
1663
1664
1665
1666
1667
1668
1669
1670
1671
1672
1673
1674
1675
1676
1677
1678
1679
1680
1681
1682
1683
1684
1685
1686
1687
1688
1689
1690
1691
1692
1693
1694
1695
1696
1697
1698
1699
1700
1701
1702
1703
1704
1705
1706
1707
1708
1709
1710
1711
1712
1713
1714
1715
1716
1717
1718
1719
1720
1721
1722
1723
1724
1725
1726
1727
1728
1729
1730
1731
1732
1733
1734
1735
1736
1737
1738
1739
1740
1741
1742
1743
1744
1745
1746
1747
1748
1749
1750
1751
1752
1753
1754
1755
1756
1757
1758
1759
1760
1761
class ViewParticipantGradesTest2(GradesTestMixin, TestCase):
def setUp(self):
super(ViewParticipantGradesTest2, self).setUp()
self.use_default_setup()
self.gopp_hidden_in_gradebook = factories.GradingOpportunityFactory(
course=self.course, aggregation_strategy=g_stragety.use_latest,
flow_id=None, shown_in_grade_book=False,
identifier="hidden_in_instructor_grade_book")
self.gopp_hidden_in_gradebook = factories.GradingOpportunityFactory(
course=self.course, aggregation_strategy=g_stragety.use_latest,
flow_id=None, shown_in_grade_book=False,
identifier="only_hidden_in_grade_book")
self.gopp_hidden_in_participation_gradebook = (
factories.GradingOpportunityFactory(
course=self.course,
shown_in_participant_grade_book=False,
aggregation_strategy=g_stragety.use_latest,
flow_id=None, identifier="all_hidden_in_ptcp_gradebook"))
self.gopp_result_hidden_in_participation_gradebook = (
factories.GradingOpportunityFactory(
course=self.course, result_shown_in_participant_grade_book=False,
aggregation_strategy=g_stragety.use_latest,
flow_id=None, identifier="result_hidden_in_ptcp_gradebook"))
self.gc_gopp_result_hidden = factories.GradeChangeFactory(
**self.gc(points=66.67,
opportunity=self.gopp_result_hidden_in_participation_gradebook,
state=g_state.graded))
def test_view_my_grade(self):
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_my_grades()
self.assertEqual(resp.status_code, 200)
grade_table = self.get_response_context_value_by_name(
resp, "grade_table")
self.assertEqual((len(grade_table)), 2)
self.assertEqual([g_info.opportunity.identifier
for g_info in grade_table],
[factories.DEFAULT_GRADE_IDENTIFIER,
"result_hidden_in_ptcp_gradebook"])
# the grade is hidden
self.assertNotContains(resp, 66.67)
grade_participation = self.get_response_context_value_by_name(
resp, "grade_participation")
self.assertEqual(grade_participation.pk, self.student_participation.pk)
# shown
self.assertContains(resp, factories.DEFAULT_GRADE_IDENTIFIER)
self.assertContains(resp, "result_hidden_in_ptcp_gradebook")
# hidden
self.assertNotContains(resp, "hidden_in_instructor_grade_book")
self.assertNotContains(resp, "all_hidden_in_ptcp_gradebook")
def test_view_participant_grades(self):
with self.temporarily_switch_to_user(self.instructor_participation.user):
resp = self.get_view_participant_grades(self.student_participation.id)
self.assertEqual(resp.status_code, 200)
grade_table = self.get_response_context_value_by_name(
resp, "grade_table")
self.assertEqual((len(grade_table)), 3)
self.assertEqual([g_info.opportunity.identifier
for g_info in grade_table],
['all_hidden_in_ptcp_gradebook',
factories.DEFAULT_GRADE_IDENTIFIER,
"result_hidden_in_ptcp_gradebook"])
# the grade hidden to participation is show to instructor
# self.assertContains(resp, "66.67%(not released)")
grade_participation = self.get_response_context_value_by_name(
resp, "grade_participation")
self.assertEqual(grade_participation.pk, self.student_participation.pk)
# shown
self.assertContains(resp, factories.DEFAULT_GRADE_IDENTIFIER)
self.assertContains(resp, "result_hidden_in_ptcp_gradebook")
self.assertContains(resp, "all_hidden_in_ptcp_gradebook")
# hidden
self.assertNotContains(resp, "hidden_in_instructor_grade_book")
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_participant_grades(
participation_id=self.instructor_participation.id)
self.assertEqual(resp.status_code, 403)
class ViewReopenSessionTest(GradesTestMixin, TestCase):
# grades.view_reopen_session (currently for cases not covered by other tests)
gopp_id = "la_quiz"
def setUp(self):
super(ViewReopenSessionTest, self).setUp()
self.fs1 = factories.FlowSessionFactory(
participation=self.student_participation, in_progress=False)
self.fs2 = factories.FlowSessionFactory(
participation=self.student_participation, in_progress=True)
def test_flow_desc_not_exist(self):
with mock.patch("course.content.get_flow_desc") as mock_get_flow_desc:
from django.core.exceptions import ObjectDoesNotExist
mock_get_flow_desc.side_effect = ObjectDoesNotExist
resp = self.get_reopen_session_view(
self.gopp_id, flow_session_id=self.fs1.pk)
self.assertEqual(resp.status_code, 404)
def test_already_in_progress(self):
# not unsubmit, because we don't have previoius grade visit (which will
# result in error)
data = {'set_access_rules_tag': ['<<<NONE>>>'],
'comment': ['test reopen'],
'reopen': ''}
resp = self.post_reopen_session_view(
self.gopp_id, flow_session_id=self.fs2.pk, data=data)
self.assertEqual(resp.status_code, 200)
self.assertEqual(self.mock_add_message.call_count, 1)
self.assertIn(
"Cannot reopen a session that's already in progress.",
self.mock_add_message.call_args[0])
self.assertTrue(self.fs2.in_progress)
def test_reopen_success(self):
resp = self.get_reopen_session_view(
self.gopp_id, flow_session_id=self.fs1.pk)
self.assertEqual(resp.status_code, 200)
# not unsubmit, because we don't have previoius grade visit (which will
# result in error)
data = {'set_access_rules_tag': ['<<<NONE>>>'],
'comment': ['test reopen'],
'reopen': ''}
resp = self.post_reopen_session_view(
self.gopp_id, flow_session_id=self.fs1.pk, data=data)
self.assertEqual(resp.status_code, 302)
self.fs1.refresh_from_db()
self.assertTrue(self.fs1.in_progress)
def test_set_access_rule_tag(self):
from relate.utils import dict_to_struct, struct_to_dict
hacked_flow_desc_dict = self.get_hacked_flow_desc(as_dict=True)
rules = hacked_flow_desc_dict["rules"]
rules_dict = struct_to_dict(rules)
rules_dict["tags"] = ["blahblah"]
rules = dict_to_struct(rules_dict)
hacked_flow_desc_dict["rules"] = rules
hacked_flow_desc = dict_to_struct(hacked_flow_desc_dict)
assert hacked_flow_desc.rules.tags == ["blahblah"]
with mock.patch("course.content.get_flow_desc") as mock_get_flow_desc:
mock_get_flow_desc.return_value = hacked_flow_desc
# not unsubmit, because we don't have previoius grade visit (which will
# result in error)
data = {'set_access_rules_tag': ['blahblah'],
'comment': ['test reopen'],
'reopen': ''}
resp = self.post_reopen_session_view(
self.gopp_id, flow_session_id=self.fs1.pk, data=data)
self.assertEqual(resp.status_code, 302)
self.fs1.refresh_from_db()
self.assertTrue(self.fs1.in_progress)
self.assertEqual(self.fs1.access_rules_tag, 'blahblah')
class ViewSingleGradeTest(GradesTestMixin, TestCase):
# grades.view_single_grade (currently for cases not covered by other tests)
def setUp(self):
super(ViewSingleGradeTest, self).setUp()
fake_regrade_session = mock.patch("course.flow.regrade_session")
self.mock_regrade_session = fake_regrade_session.start()
self.addCleanup(fake_regrade_session.stop)
fake_recalculate_session_grade = mock.patch(
"course.flow.recalculate_session_grade")
self.mock_recalculate_session_grade = fake_recalculate_session_grade.start()
self.addCleanup(fake_recalculate_session_grade.stop)
fake_expire_flow_session_standalone = mock.patch(
"course.flow.expire_flow_session_standalone")
self.mock_expire_flow_session_standalone = (
fake_expire_flow_session_standalone.start())
self.addCleanup(fake_expire_flow_session_standalone.stop)
fake_finish_flow_session_standalone = mock.patch(
"course.flow.finish_flow_session_standalone")
self.mock_finish_flow_session_standalone = (
fake_finish_flow_session_standalone.start())
self.addCleanup(fake_finish_flow_session_standalone.stop)
def test_participation_course_not_match(self):
another_course_participation = factories.ParticipationFactory(
course=factories.CourseFactory(identifier="another-course"))
resp = self.get_view_single_grade(another_course_participation, self.gopp)
self.assertEqual(resp.status_code, 400)
def test_gopp_course_not_match(self):
another_course_gopp = factories.GradingOpportunityFactory(
course=factories.CourseFactory(identifier="another-course"),
identifier=QUIZ_FLOW_ID)
with self.temporarily_switch_to_user(self.instructor_participation.user):
resp = self.c.get(self.get_single_grade_url(
self.student_participation.pk, another_course_gopp.pk))
self.assertEqual(resp.status_code, 400)
def test_view_other_single_grade_no_pperm(self):
another_participation = factories.ParticipationFactory(
course=self.course)
with self.temporarily_switch_to_user(another_participation.user):
resp = self.get_view_single_grade(
self.student_participation, self.gopp, force_login_instructor=False)
self.assertEqual(resp.status_code, 403)
resp = self.post_view_single_grade(
self.student_participation, self.gopp, data={},
force_login_instructor=False)
self.assertEqual(resp.status_code, 403)
def test_view_success(self):
resp = self.get_view_single_grade(
self.student_participation, self.gopp)
self.assertEqual(resp.status_code, 200)
def test_view_not_shown_in_grade_book(self):
hidden_gopp = factories.GradingOpportunityFactory(
course=self.course, identifier="hidden",
shown_in_grade_book=False)
resp = self.get_view_single_grade(
self.student_participation, hidden_gopp)
self.assertEqual(resp.status_code, 200)
self.assertIn(
"This grade is not shown in the grade book.",
self.mock_add_message.call_args[0])
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(
self.student_participation, hidden_gopp,
force_login_instructor=False)
self.assertEqual(resp.status_code, 403)
def test_view_not_shown_in_participant_grade_book(self):
hidden_gopp = factories.GradingOpportunityFactory(
course=self.course, identifier="hidden",
shown_in_participant_grade_book=False)
resp = self.get_view_single_grade(
self.student_participation, hidden_gopp)
self.assertEqual(resp.status_code, 200)
self.assertIn(
"This grade is not shown in the student grade book.",
self.mock_add_message.call_args[0])
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(
self.student_participation, hidden_gopp,
force_login_instructor=False)
self.assertEqual(resp.status_code, 403)
@unittest.skipIf(six.PY2, "PY2 doesn't support subTest")
def test_post_no_pperm(self):
another_participation = factories.ParticipationFactory(
course=self.course)
# only view_gradebook pperm
pp = models.ParticipationPermission(
participation=another_participation,
permission=pperm.view_gradebook)
pp.save()
fs = factories.FlowSessionFactory(
participation=self.student_participation, flow_id=self.flow_id)
for op in ["imposedl", "end", "regrade", "recalculate"]:
with self.subTest(op=op):
resp = self.post_view_single_grade(
self.student_participation, self.gopp,
data={"%s_%d" % (op, fs.pk): ''},
force_login_instructor=False)
self.assertEqual(resp.status_code, 403)
def test_post_no_action_match(self):
resp = self.post_view_single_grade(
self.student_participation, self.gopp,
data={"blablabal": ''})
self.assertEqual(resp.status_code, 400)
@unittest.skipIf(six.PY2, "PY2 doesn't support subTest")
def test_post(self):
fs = factories.FlowSessionFactory(
participation=self.student_participation, flow_id=self.flow_id)
tup = (
("imposedl", self.mock_expire_flow_session_standalone,
"Session deadline imposed."),
("end", self.mock_finish_flow_session_standalone, "Session ended."),
("regrade", self.mock_regrade_session, "Session regraded."),
("recalculate", self.mock_recalculate_session_grade,
"Session grade recalculated."))
for op, mock_func, msg in tup:
with self.subTest(op=op):
resp = self.post_view_single_grade(
self.student_participation, self.gopp,
data={"%s_%d" % (op, fs.pk): ''})
self.assertEqual(resp.status_code, 200)
self.assertEqual(mock_func.call_count, 1)
self.assertIn(msg, self.mock_add_message.call_args[0])
mock_func.reset_mock()
self.mock_add_message.reset_mock()
def test_post_invalid_session_op(self):
fs = factories.FlowSessionFactory(
participation=self.student_participation, flow_id=self.flow_id)
resp = self.post_view_single_grade(
self.student_participation, self.gopp,
data={"blablabal_%d" % fs.pk: ''})
self.assertEqual(resp.status_code, 400)
@unittest.skipIf(six.PY2, "PY2 doesn't support subTest")
def test_post_keyboard_interrupt(self):
fs = factories.FlowSessionFactory(
participation=self.student_participation, flow_id=self.flow_id)
tup = (
("imposedl", self.mock_expire_flow_session_standalone,
"Session deadline imposed."),
("end", self.mock_finish_flow_session_standalone, "Session ended."),
("regrade", self.mock_regrade_session, "Session regraded."),
("recalculate", self.mock_recalculate_session_grade,
"Session grade recalculated."))
err = "foo"
self.mock_regrade_session.side_effect = KeyboardInterrupt(err)
self.mock_recalculate_session_grade.side_effect = KeyboardInterrupt(err)
self.mock_expire_flow_session_standalone.side_effect = KeyboardInterrupt(err)
self.mock_finish_flow_session_standalone.side_effect = KeyboardInterrupt(err)
for op, mock_func, msg in tup:
with self.subTest(op=op):
resp = self.post_view_single_grade(
self.student_participation, self.gopp,
data={"%s_%d" % (op, fs.pk): ''})
self.assertEqual(resp.status_code, 200)
self.assertNotIn(msg, self.mock_add_message.call_args[0])
self.assertIn(
"Error: KeyboardInterrupt %s" % err,
self.mock_add_message.call_args[0])
mock_func.reset_mock()
self.mock_add_message.reset_mock()
def test_view_gopp_flow_desc_not_exist(self):
with mock.patch("course.content.get_flow_desc") as mock_get_flow_desc:
from django.core.exceptions import ObjectDoesNotExist
mock_get_flow_desc.side_effect = ObjectDoesNotExist()
resp = self.get_view_single_grade(
self.student_participation, self.gopp)
self.assertEqual(resp.status_code, 200)
self.assertResponseContextIsNone(
resp, "flow_sessions_and_session_properties")
def test_view_gopp_no_flow_id(self):
gopp = factories.GradingOpportunityFactory(
course=self.course,
identifier="no_flow_id",
flow_id=None)
factories.GradeChangeFactory(
**self.gc(
opportunity=gopp))
resp = self.get_view_single_grade(
self.student_participation, gopp)
self.assertEqual(resp.status_code, 200)
self.assertResponseContextIsNone(
resp, "flow_sessions_and_session_properties")
def test_filter_out_pre_public_grade_changes(self):
gopp = factories.GradingOpportunityFactory(
course=self.course,
identifier="no_flow_id",
flow_id=None)
# 5 gchanges
factories.GradeChangeFactory(**self.gc(
opportunity=gopp))
factories.GradeChangeFactory(**self.gc(
opportunity=gopp))
factories.GradeChangeFactory(**self.gc(
opportunity=gopp))
fourth_gc = factories.GradeChangeFactory(**self.gc(
opportunity=gopp))
factories.GradeChangeFactory(**self.gc(
opportunity=gopp))
resp = self.get_view_single_grade(
self.student_participation, gopp)
self.assertEqual(resp.status_code, 200)
resp_gchanges = resp.context["grade_changes"]
self.assertEqual(len(resp_gchanges), 5)
# update_gopp
gopp.hide_superseded_grade_history_before = (
fourth_gc.grade_time - timedelta(minutes=1))
gopp.save()
# view by instructor
resp = self.get_view_single_grade(
self.student_participation, gopp)
self.assertEqual(resp.status_code, 200)
resp_gchanges = resp.context["grade_changes"]
self.assertEqual(len(resp_gchanges), 5)
# view by student
with self.temporarily_switch_to_user(self.student_participation.user):
resp = self.get_view_single_grade(
self.student_participation, gopp, force_login_instructor=False)
self.assertEqual(resp.status_code, 200)
resp_gchanges = resp.context["grade_changes"]
self.assertEqual(len(resp_gchanges), 2)
@unittest.SkipTest
class FixingTest(GradesTestMixin, TestCase):
# currently skipped
def reopen_session1(self):
existing_gc_count = models.GradeChange.objects.count()
reopen_session(now_datetime=local_now(), session=self.session1,
generate_grade_change=True,
suppress_log=True)
self.assertEqual(models.GradeChange.objects.count(), existing_gc_count+1)
self.session1.refresh_from_db()
def reopen_session2(self):
existing_gc_count = models.GradeChange.objects.count()
reopen_session(now_datetime=local_now(), session=self.session2,
generate_grade_change=True,
suppress_log=True)
self.assertEqual(models.GradeChange.objects.count(), existing_gc_count+1)
self.session2.refresh_from_db()
def test_append_gc_with_session_after_reopen_session2(self):
self.use_default_setup()
self.reopen_session2()
# append a grade change for session2
# grade_time need to be specified, because the faked gc
# is using fake time, while reopen a session will create
# an actual gc using the actual time.
latest_gc = models.GradeChange.objects.all().order_by("-grade_time")[0]
self.append_gc(self.gc(points=12, flow_session=self.session2,
grade_time=now(),
effective_time=latest_gc.effective_time))
self.assertGradeChangeMachineReadableStateEqual(12)
self.assertGradeChangeStateEqual("12.00% (/3)")
def test_append_nonsession_gc_after_reopen_session2(self):
self.use_default_setup()
self.reopen_session2()
# Append a grade change without session
# grade_time need to be specified, because the faked gc
# is using fake time, while reopen a session will create
# an actual gc using the actual time.
self.append_gc(self.gc(points=11, grade_time=now()))
self.assertGradeChangeMachineReadableStateEqual(11)
self.assertGradeChangeStateEqual("11.00% (/3)")
def test_new_gchange_created_when_finish_flow_use_last_has_activity(self):
# With use_last_activity_as_completion_time = True, if a flow session HAS
# last_activity, the expected effective_time of the new gchange should be
# the last_activity() of the related flow_session.
with self.temporarily_switch_to_user(self.instructor_participation.user):
self.start_flow(QUIZ_FLOW_ID)
# create a flow page visit, then there should be last_activity() for
# the session.
self.post_answer_by_ordinal(1, {"answer": ['0.5']})
self.assertEqual(
models.FlowPageVisit.objects.filter(answer__isnull=False).count(),
1)
last_answered_visit = (
models.FlowPageVisit.objects.filter(answer__isnull=False).first())
last_answered_visit.visit_time = now() - timedelta(hours=1)
last_answered_visit.save()
self.assertEqual(models.GradeChange.objects.count(), 0)
with mock.patch("course.flow.get_session_grading_rule") as \
mock_get_grading_rule:
mock_get_grading_rule.side_effect = (
get_session_grading_rule_use_last_activity_as_cmplt_time_side_effect) # noqa
resp = self.end_flow()
self.assertEqual(resp.status_code, 200)
self.assertEqual(models.GradeChange.objects.count(), 1)
latest_gchange = models.GradeChange.objects.last()
latest_flow_session = models.FlowSession.objects.last()
self.assertIsNotNone(latest_flow_session.last_activity())
self.assertEqual(latest_flow_session.completion_time,
latest_flow_session.last_activity())
self.assertEqual(latest_gchange.effective_time,
latest_flow_session.last_activity())
# {{{ Fixed issue #263 and #417
def test_update_latest_gc_of_ealier_finished_session(self):
self.use_default_setup()
self.assertGradeChangeMachineReadableStateEqual(6)
# Issue #263 and #417
# gc_session1 is the GradeChange object of session 1, update it's
# value won't change the consumed state.
self.update_gc(self.gc_session1, points=10)
self.assertGradeChangeMachineReadableStateEqual(6)
self.assertGradeChangeStateEqual("6.00% (/3)")
def test_special_case(self):
# https://github.com/inducer/relate/pull/423#discussion_r162121467
gc2015 = factories.GradeChangeFactory.create(**(self.gc(points=5)))
session1 = factories.FlowSessionFactory.create(
participation=self.student_participation,
start_time=self.time-timedelta(days=17),
completion_time=self.time-timedelta(days=14))
self.time_increment()
gc2016 = factories.GradeChangeFactory.create(
**(self.gc(points=0, flow_session=session1, grade_time=self.time)))
gc2017 = factories.GradeChangeFactory.create(**(self.gc(points=7)))
session2 = factories.FlowSessionFactory.create(
participation=self.student_participation,
start_time=self.time-timedelta(days=17),
completion_time=self.time-timedelta(days=15))
self.time_increment()
gc2018 = factories.GradeChangeFactory.create(
**(self.gc(points=6, flow_session=session2)))
assert models.GradingOpportunity.objects.count() == 1
assert models.GradeChange.objects.count() == 4
assert models.FlowSession.objects.count() == 2
self.assertTrue(session2.completion_time < session1.completion_time)
self.assertTrue(
gc2015.grade_time < gc2016.grade_time < gc2017.grade_time
< gc2018.grade_time)
self.assertGradeChangeMachineReadableStateEqual(gc2017.percentage())
# {{{ When two grade changes have the same grade_time
# The expected behavior is GradeChange object with the larger pk
1763
1764
1765
1766
1767
1768
1769
1770
1771
1772
1773
1774
1775
1776
1777
1778
1779
1780
1781
1782
1783
1784
1785
1786
1787
1788
1789
1790
1791
1792
1793
1794
1795
1796
1797
1798
1799
1800
1801
1802
1803
1804
1805
1806
1807
1808
1809
1810
1811
1812
1813
1814
1815
1816
1817
1818
1819
1820
1821
1822
1823
1824
1825
1826
1827
1828
1829
1830
1831
1832
1833
1834
1835
1836
1837
1838
1839
1840
1841
1842
1843
1844
1845
1846
1847
1848
1849
1850
1851
1852
1853
1854
1855
1856
1857
1858
1859
1860
1861
1862
1863
1864
1865
1866
1867
1868
1869
1870
1871
1872
1873
1874
1875
1876
1877
1878
1879
1880
1881
1882
1883
1884
1885
1886
1887
1888
1889
1890
1891
# dominate. Fixed with #263 and #417
def test_gcs_have_same_grade_time1(self):
gc1 = factories.GradeChangeFactory.create(**(self.gc(points=0)))
session = factories.FlowSessionFactory.create(
participation=self.student_participation,
completion_time=gc1.grade_time-timedelta(days=1))
factories.GradeChangeFactory.create(
**(self.gc(points=5, flow_session=session,
grade_time=gc1.grade_time)))
self.assertGradeChangeMachineReadableStateEqual(5)
self.assertGradeChangeStateEqual("5.0% (/2)")
def test_gc_have_same_grade_time2(self):
session = factories.FlowSessionFactory.create(
participation=self.student_participation,
start_time=self.time-timedelta(days=1),
completion_time=self.time)
self.time_increment()
gc1 = factories.GradeChangeFactory.create(
**(self.gc(points=5, flow_session=session)))
factories.GradeChangeFactory.create(
**(self.gc(points=0, grade_time=gc1.grade_time)))
self.assertGradeChangeMachineReadableStateEqual(0)
self.assertGradeChangeStateEqual("0.00% (/2)")
# }}}
# {{{ Fix #430
def test_reopen_session2(self):
self.use_default_setup()
# original state
self.assertGradeChangeMachineReadableStateEqual("6")
n_gc = models.GradeChange.objects.count()
self.reopen_session2()
# A new GradeChange object is created, with state "do_over"
expected_n_gc = models.GradeChange.objects.count()
self.assertEqual(expected_n_gc, n_gc + 1)
self.assertEqual(
models.GradeChange.objects.order_by("grade_time").last().state,
g_state.do_over)
self.assertGradeChangeMachineReadableStateEqual("NONE")
self.assertGradeChangeStateEqual("- ∅ - (/3)")
def test_reopen_session_without_existing_gc(self):
# This is rare, because a completed_session should had created
# a GradeChange object.
session_temp = factories.FlowSessionFactory.create(
participation=self.student_participation, completion_time=self.time)
existing_gc_count = models.GradeChange.objects.count()
reopen_session(now_datetime=local_now(), session=session_temp,
generate_grade_change=True,
suppress_log=True)
self.assertEqual(models.GradeChange.objects.count(), existing_gc_count)
def test_reopen_session1(self):
self.use_default_setup()
self.assertGradeChangeMachineReadableStateEqual("6")
n_gc = models.GradeChange.objects.count()
self.reopen_session1()
# A new GradeChange object is created, with state "do_over"
expected_n_gc = models.GradeChange.objects.count()
self.assertEqual(expected_n_gc, n_gc + 1)
self.assertEqual(
models.GradeChange.objects.order_by("grade_time").last().state,
g_state.do_over)
# session 1 is not the latest session
self.assertGradeChangeMachineReadableStateEqual("6")
self.assertGradeChangeStateEqual("6.00% (/3)")
def _get_admin_flow_session_delete_url(self, args):
return reverse("admin:course_flowsession_delete", args=args)
def _delete_flow_session_admin(self, flow_session):
exist_flow_session_count = models.FlowSession.objects.count()
flow_session_delete_url = self._get_admin_flow_session_delete_url(
args=(flow_session.id,))
delete_dict = {'post': 'yes'}
with self.temporarily_switch_to_user(self.superuser):
resp = self.c.get(flow_session_delete_url)
self.assertEqual(resp.status_code, 200)
resp = self.c.post(flow_session_delete_url, data=delete_dict)
self.assertEqual(resp.status_code, 302)
self.assertEqual(exist_flow_session_count,
models.FlowSession.objects.count() + 1)
def test_delete_flow_session_admin_new_exempt_gradechange_created(self):
self.use_default_setup()
exist_grade_change_count = models.GradeChange.objects.count()
# session1 has related grade changes, so a new grade change with 'exempt' is
# created
self._delete_flow_session_admin(self.session1)
self.assertEqual(exist_grade_change_count + 1,
models.GradeChange.objects.count())
last_gchange = (
models.GradeChange.objects
.order_by("-grade_time").first())
self.assertIsNone(last_gchange.flow_session)
self.assertEqual(last_gchange.state, g_state.exempt)
def test_delete_flow_session_admin_no_new_gradechange_created(self):
session_temp = factories.FlowSessionFactory.create(
participation=self.student_participation, completion_time=self.time)
exist_grade_change_count = models.GradeChange.objects.count()
last_gchange_of_session_temp = (
models.GradeChange.objects
.filter(flow_session=session_temp)
.order_by("-grade_time")[:1])
self.assertEqual(last_gchange_of_session_temp.count(), 0)
# session_temp has no related grade changes, so no new grade change
# is created after deleted
self._delete_flow_session_admin(session_temp)
self.assertEqual(exist_grade_change_count,
models.GradeChange.objects.count())
1894
1895
1896
1897
1898
1899
1900
1901
1902
1903
1904
1905
1906
1907
1908
1909
1910
1911
1912
1913
1914
1915
1916
1917
1918
1919
1920
1921
1922
1923
1924
1925
1926
1927
1928
1929
1930
1931
1932
1933
1934
1935
1936
1937
1938
1939
1940
1941
1942
1943
1944
1945
1946
1947
1948
1949
1950
1951
1952
1953
1954
1955
1956
1957
1958
1959
1960
1961
1962
1963
1964
1965
1966
1967
1968
1969
1970
1971
1972
1973
1974
1975
1976
1977
1978
1979
1980
1981
1982
1983
1984
1985
1986
def test_backward_compatibility_merging_466(self):
# this make sure after merging https://github.com/inducer/relate/pull/466
# gchanges are consumed without issue
self.use_default_setup()
self.gc_session2.effective_time = None
self.gc_session2.save()
self.gc_session2.refresh_from_db()
# We are not using reopen_session(), because that will create new
# gchange, which only happen after #466 was merged.
self.session2.in_progress = True
self.session2.save()
self.session2.refresh_from_db()
machine = self.get_gc_machine()
# session2's gchange is excluded
self.assertGradeChangeMachineReadableStateEqual(7)
self.assertEqual(machine.valid_percentages, [0, 7])
self.assertGradeChangeStateEqual("7.00% (/2)")
# {{{ test new gchange created when finishing flow
def test_new_gchange_created_when_finish_flow_use_last_no_activity(self):
# With use_last_activity_as_completion_time = True, if a flow session has
# no last_activity, the expected effective_time of the new gchange should
# be the completion time of the related flow_session.
with self.temporarily_switch_to_user(self.student_participation.user):
self.start_flow(QUIZ_FLOW_ID)
self.assertEqual(models.GradeChange.objects.count(), 0)
with mock.patch("course.flow.get_session_grading_rule") as \
mock_get_grading_rule:
mock_get_grading_rule.side_effect = (
get_session_grading_rule_use_last_activity_as_cmplt_time_side_effect) # noqa
resp = self.end_flow()
self.assertEqual(resp.status_code, 200)
self.assertEqual(models.GradeChange.objects.count(), 1)
latest_gchange = models.GradeChange.objects.last()
latest_flow_session = models.FlowSession.objects.last()
self.assertIsNone(latest_flow_session.last_activity())
self.assertEqual(latest_gchange.effective_time,
latest_flow_session.completion_time)
def test_new_gchange_created_when_finish_flow_not_use_last_no_activity(self):
# With use_last_activity_as_completion_time = False, if a flow session has
# no last_activity, the expected effective_time of the new gchange should
# be the completion time of the related flow_session.
with self.temporarily_switch_to_user(self.student_participation.user):
self.start_flow(QUIZ_FLOW_ID)
self.assertEqual(models.GradeChange.objects.count(), 0)
resp = self.end_flow()
self.assertEqual(resp.status_code, 200)
self.assertEqual(models.GradeChange.objects.count(), 1)
latest_gchange = models.GradeChange.objects.last()
latest_flow_session = models.FlowSession.objects.last()
self.assertIsNone(latest_flow_session.last_activity())
self.assertEqual(latest_gchange.effective_time,
latest_flow_session.completion_time)
def test_new_gchange_created_when_finish_flow_not_use_last_has_activity(self):
# With use_last_activity_as_completion_time = False, even if a flow session
# HAS last_activity, the expected effective_time of the new gchange should
# be the completion_time of the related flow_session.
with self.temporarily_switch_to_user(self.instructor_participation.user):
self.start_flow(QUIZ_FLOW_ID)
# create a flow page visit, then there should be last_activity() for
# the session.
self.post_answer_by_ordinal(1, {"answer": ['0.5']})
self.assertEqual(
models.FlowPageVisit.objects.filter(answer__isnull=False).count(),
1)
last_answered_visit = (
models.FlowPageVisit.objects.filter(answer__isnull=False).first())
last_answered_visit.visit_time = now() - timedelta(hours=1)
last_answered_visit.save()
self.assertEqual(models.GradeChange.objects.count(), 0)
resp = self.end_flow()
self.assertEqual(resp.status_code, 200)
self.assertEqual(models.GradeChange.objects.count(), 1)
latest_gchange = models.GradeChange.objects.last()
latest_flow_session = models.FlowSession.objects.last()
self.assertIsNotNone(latest_flow_session.last_activity())
self.assertNotEqual(latest_flow_session.completion_time,
latest_flow_session.last_activity())
self.assertEqual(latest_gchange.effective_time,
latest_flow_session.completion_time)