From 493421c1ec0f78f9a0793a6fd76b302445c4e064 Mon Sep 17 00:00:00 2001 From: Vasyl Nakvasiuk Date: Wed, 6 Mar 2013 19:52:18 +0200 Subject: [PATCH] fix `test_conditional`, add new conditional to "conditional_and_poll" test course --- .../xmodule/xmodule/tests/test_conditional.py | 18 +++++++++------ .../conditional/condone.xml | 3 +++ .../course/2013_Spring.xml | 6 +++++ .../conditional_and_poll/html/secret_page.xml | 4 ++++ .../problem/choiceprob.xml | 22 +++++++++++++++++++ 5 files changed, 46 insertions(+), 7 deletions(-) create mode 100644 common/test/data/conditional_and_poll/conditional/condone.xml create mode 100644 common/test/data/conditional_and_poll/html/secret_page.xml create mode 100644 common/test/data/conditional_and_poll/problem/choiceprob.xml diff --git a/common/lib/xmodule/xmodule/tests/test_conditional.py b/common/lib/xmodule/xmodule/tests/test_conditional.py index d4d5a1d2e1..1d63577073 100644 --- a/common/lib/xmodule/xmodule/tests/test_conditional.py +++ b/common/lib/xmodule/xmodule/tests/test_conditional.py @@ -70,7 +70,7 @@ class ConditionalModuleTest(unittest.TestCase): """Make sure that conditional module works""" print "Starting import" - course = self.get_course('conditional') + course = self.get_course('conditional_and_poll') print "Course: ", course print "id: ", course.id @@ -82,7 +82,9 @@ class ConditionalModuleTest(unittest.TestCase): location = descriptor.location return descriptor.xmodule(test_system) - location = Location(["i4x", "edX", "cond_test", "conditional", "condone"]) + # edx - HarvardX + # cond_test - ER22x + location = Location(["i4x", "HarvardX", "ER22x", "conditional", "condone"]) def replace_urls(text, staticfiles_prefix=None, replace_prefix='/static/', course_namespace=None): return text @@ -91,14 +93,14 @@ class ConditionalModuleTest(unittest.TestCase): module = inner_get_module(location) print "module: ", module - print "module.condition: ", module.condition + print "module.conditions_map: ", module.conditions_map print "module children: ", module.get_children() print "module display items (children): ", module.get_display_items() html = module.get_html() print "html type: ", type(html) print "html: ", html - html_expect = "{'ajax_url': 'courses/course_id/modx/a_location', 'element_id': 'i4x-edX-cond_test-conditional-condone', 'id': 'i4x://edX/cond_test/conditional/condone'}" + html_expect = "{'ajax_url': 'courses/course_id/modx/a_location', 'element_id': 'i4x-HarvardX-ER22x-conditional-condone', 'id': 'i4x://HarvardX/ER22x/conditional/condone', 'depends': 'i4x-HarvardX-ER22x-problem-choiceprob'}" self.assertEqual(html, html_expect) gdi = module.get_display_items() @@ -106,11 +108,13 @@ class ConditionalModuleTest(unittest.TestCase): ajax = json.loads(module.handle_ajax('', '')) print "ajax: ", ajax - self.assertTrue('ConditionalModule' in ajax['html']) + html = ajax['html'] + self.assertFalse(any(['This is a secret' in item for item in html])) # now change state of the capa problem to make it completed - inner_get_module(Location('i4x://edX/cond_test/problem/choiceprob')).attempts = 1 + inner_get_module(Location('i4x://HarvardX/ER22x/problem/choiceprob')).attempts = 1 ajax = json.loads(module.handle_ajax('', '')) print "post-attempt ajax: ", ajax - self.assertTrue('This is a secret' in ajax['html']) + html = ajax['html'] + self.assertTrue(any(['This is a secret' in item for item in html])) diff --git a/common/test/data/conditional_and_poll/conditional/condone.xml b/common/test/data/conditional_and_poll/conditional/condone.xml new file mode 100644 index 0000000000..80b061e244 --- /dev/null +++ b/common/test/data/conditional_and_poll/conditional/condone.xml @@ -0,0 +1,3 @@ + + + diff --git a/common/test/data/conditional_and_poll/course/2013_Spring.xml b/common/test/data/conditional_and_poll/course/2013_Spring.xml index cb6e7c1217..2eea422a2f 100644 --- a/common/test/data/conditional_and_poll/course/2013_Spring.xml +++ b/common/test/data/conditional_and_poll/course/2013_Spring.xml @@ -2,5 +2,11 @@ Take note of this name exactly, you'll need to use it everywhere. --> + + + + + + diff --git a/common/test/data/conditional_and_poll/html/secret_page.xml b/common/test/data/conditional_and_poll/html/secret_page.xml new file mode 100644 index 0000000000..63be3cfa8d --- /dev/null +++ b/common/test/data/conditional_and_poll/html/secret_page.xml @@ -0,0 +1,4 @@ + +

This is a secret!

+ + diff --git a/common/test/data/conditional_and_poll/problem/choiceprob.xml b/common/test/data/conditional_and_poll/problem/choiceprob.xml new file mode 100644 index 0000000000..fa91954977 --- /dev/null +++ b/common/test/data/conditional_and_poll/problem/choiceprob.xml @@ -0,0 +1,22 @@ + + + +

Consider a hypothetical magnetic field pointing out of your computer screen. Now imagine an electron traveling from right to left in the plane of your screen. A diagram of this situation is show below…

+
+ +

a. The magnitude of the force experienced by the electron is proportional the product of which of the following? (Select all that apply.)

+ + + + +Magnetic field strength… +Electric field strength… +Electric charge of the electron… +Radius of the electron… +Mass of the electron… +Velocity of the electron… + + + + +