Fix tests to match the current desired functionality

murhum1 · Feb 19, 2024 · 504c722 · 504c722
1 parent bd5a40e
commit 504c722
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/exercise/tests_cache.py b/exercise/tests_cache.py
@@ -508,7 +508,9 @@ def test_accumulation(self):
         self.assertEqual(module.submission_count, 2)
         self.assertEqual(module.points, 50)
         self.assertEqual(module.points_by_difficulty.get('',0), 50)
-        self.assertFalse(module.passed)
+        # True for now; passing individual exercises should have no bearing
+        # on passing a module, to mimic previous functionality
+        self.assertTrue(module.passed)
         category = p.categories()[0]
         self.assertTrue(category.passed)
 

diff --git a/threshold/tests.py b/threshold/tests.py
@@ -147,7 +147,9 @@ def test_pass(self):
         self.assertFalse(t.is_passed(points))
         t.passed_exercises.clear()
         t.passed_modules.add(self.module)
-        self.assertFalse(t.is_passed(points))
+        # True for now to mimic previous functionality.
+        # The rest of the test is redundant.
+        self.assertTrue(t.is_passed(points))
         self.submission3.set_points(2,2)
         self.submission3.set_ready()
         self.submission3.save()