From dbf15fc2ac5322660446b979a7f3e7b31087d7ec Mon Sep 17 00:00:00 2001 From: Chris Dodge Date: Wed, 30 Jan 2013 09:46:38 -0500 Subject: [PATCH] add comment on draft store test which explains test success is just for get_items() not to throw an exception when passed in a course_id --- lms/djangoapps/courseware/tests/tests.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lms/djangoapps/courseware/tests/tests.py b/lms/djangoapps/courseware/tests/tests.py index fad8ac6fef..6c41cbac14 100644 --- a/lms/djangoapps/courseware/tests/tests.py +++ b/lms/djangoapps/courseware/tests/tests.py @@ -445,6 +445,8 @@ class TestDraftModuleStore(TestCase): store = modulestore() # fix was to allow get_items() to take the course_id parameter store.get_items(Location(None, None, 'vertical', None, None), course_id='abc', depth=0) + # test success is just getting through the above statement. The bug was that 'course_id' argument was + # not allowed to be passed in (i.e. was throwing exception) @override_settings(MODULESTORE=TEST_DATA_XML_MODULESTORE)