diff --git a/common/lib/xmodule/xmodule/tests/test_peer_grading.py b/common/lib/xmodule/xmodule/tests/test_peer_grading.py index 036ab4a85b..a0877eab81 100644 --- a/common/lib/xmodule/xmodule/tests/test_peer_grading.py +++ b/common/lib/xmodule/xmodule/tests/test_peer_grading.py @@ -2,6 +2,7 @@ import unittest from xmodule.modulestore import Location from .import test_system from test_util_open_ended import MockQueryDict, DummyModulestore +import json from xmodule.peer_grading_module import PeerGradingModule, PeerGradingDescriptor from xmodule.open_ended_grading_classes.grading_service_module import GradingServiceError @@ -63,10 +64,8 @@ class PeerGradingModuleTest(unittest.TestCase, DummyModulestore): Try getting data from the external grading service @return: """ - try: - success, data = self.peer_grading.query_data_for_location() - except GradingServiceError: - pass + success, data = self.peer_grading.query_data_for_location() + self.assertEqual(success, True) def test_get_score(self): """ @@ -74,6 +73,7 @@ class PeerGradingModuleTest(unittest.TestCase, DummyModulestore): @return: """ score = self.peer_grading.get_score() + self.assertEquals(score['score'], None) def test_get_max_score(self): """ @@ -81,6 +81,7 @@ class PeerGradingModuleTest(unittest.TestCase, DummyModulestore): @return: """ max_score = self.peer_grading.max_score() + self.assertEquals(max_score, None) def get_next_submission(self): """ @@ -88,13 +89,15 @@ class PeerGradingModuleTest(unittest.TestCase, DummyModulestore): @return: """ success, next_submission = self.peer_grading.get_next_submission({'location': 'blah'}) + self.assertEqual(success, True) def test_save_grade(self): """ Test if we can save the grade @return: """ - self.peer_grading.save_grade(self.save_dict) + response = self.peer_grading.save_grade(self.save_dict) + self.assertEqual(response['success'], True) def test_is_student_calibrated(self): """ @@ -102,28 +105,32 @@ class PeerGradingModuleTest(unittest.TestCase, DummyModulestore): @return: """ calibrated_dict = {'location': "blah"} - self.peer_grading.is_student_calibrated(self.calibrated_dict) + response = self.peer_grading.is_student_calibrated(self.calibrated_dict) + self.assertEqual(response['success'], True) def test_show_calibration_essay(self): """ Test showing the calibration essay @return: """ - self.peer_grading.show_calibration_essay(self.calibrated_dict) + response = self.peer_grading.show_calibration_essay(self.calibrated_dict) + self.assertEqual(response['success'], True) def test_save_calibration_essay(self): """ Test saving the calibration essay @return: """ - self.peer_grading.save_calibration_essay(self.save_dict) + response = self.peer_grading.save_calibration_essay(self.save_dict) + self.assertEqual(response['success'], True) def test_peer_grading_problem(self): """ See if we can render a single problem @return: """ - self.peer_grading.peer_grading_problem(self.calibrated_dict) + response = self.peer_grading.peer_grading_problem(self.calibrated_dict) + self.assertEqual(response['success'], True) def test_get_instance_state(self): """