Commit 9da3a18b by Braden MacDonald

Fix: Don't import all the problem builder blocks when the django app is loaded.

parent 859df415
from .mentoring import MentoringBlock
from .answer import AnswerBlock, AnswerRecapBlock
from .choice import ChoiceBlock
from .dashboard import DashboardBlock
from .mcq import MCQBlock, RatingBlock
from .mrq import MRQBlock
from .message import MentoringMessageBlock
from .table import MentoringTableBlock, MentoringTableColumn
from .tip import TipBlock
...@@ -24,7 +24,7 @@ from xblockutils.base_test import SeleniumBaseTest ...@@ -24,7 +24,7 @@ from xblockutils.base_test import SeleniumBaseTest
# Studio adds a url_name property to each XBlock but Workbench doesn't. # Studio adds a url_name property to each XBlock but Workbench doesn't.
# Since we rely on it, we need to mock url_name support so it can be set via XML and # Since we rely on it, we need to mock url_name support so it can be set via XML and
# accessed like a normal field. # accessed like a normal field.
from problem_builder import MentoringBlock from problem_builder.mentoring import MentoringBlock
MentoringBlock.url_name = String() MentoringBlock.url_name = String()
......
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
import ddt import ddt
from mock import patch, Mock from mock import patch, Mock
from problem_builder import MentoringBlock from problem_builder.mentoring import MentoringBlock
from .base_test import MentoringBaseTest from .base_test import MentoringBaseTest
......
...@@ -69,6 +69,6 @@ class MentoringThemeTest(MentoringAssessmentBaseTest): ...@@ -69,6 +69,6 @@ class MentoringThemeTest(MentoringAssessmentBaseTest):
('apros', "#ff0000") ('apros', "#ff0000")
) )
def test_lms_theme_applied(self, theme, expected_color): def test_lms_theme_applied(self, theme, expected_color):
with mock.patch("problem_builder.MentoringBlock.get_theme") as patched_theme: with mock.patch("problem_builder.mentoring.MentoringBlock.get_theme") as patched_theme:
patched_theme.return_value = _get_mentoring_theme_settings(theme) patched_theme.return_value = _get_mentoring_theme_settings(theme)
self.assert_status_icon_color(expected_color) self.assert_status_icon_color(expected_color)
...@@ -2,8 +2,7 @@ import unittest ...@@ -2,8 +2,7 @@ import unittest
import ddt import ddt
from mock import MagicMock, Mock, patch from mock import MagicMock, Mock, patch
from xblock.field_data import DictFieldData from xblock.field_data import DictFieldData
from problem_builder import MentoringBlock from problem_builder.mentoring import MentoringBlock, _default_theme_config
from problem_builder.mentoring import _default_theme_config
class TestMentoringBlock(unittest.TestCase): class TestMentoringBlock(unittest.TestCase):
......
import copy import copy
from problem_builder import MentoringBlock from problem_builder.mentoring import MentoringBlock
from mock import MagicMock, Mock from mock import MagicMock, Mock
import unittest import unittest
from xblock.field_data import DictFieldData from xblock.field_data import DictFieldData
......
...@@ -22,7 +22,7 @@ Test that we can upgrade from mentoring v1 to problem builder (v2). ...@@ -22,7 +22,7 @@ Test that we can upgrade from mentoring v1 to problem builder (v2).
""" """
import ddt import ddt
from lxml import etree from lxml import etree
from problem_builder import MentoringBlock from problem_builder.mentoring import MentoringBlock
from problem_builder.v1.xml_changes import convert_xml_v1_to_v2 from problem_builder.v1.xml_changes import convert_xml_v1_to_v2
import os.path import os.path
from StringIO import StringIO from StringIO import StringIO
......
...@@ -40,20 +40,20 @@ def package_data(pkg, root_list): ...@@ -40,20 +40,20 @@ def package_data(pkg, root_list):
# Main ############################################################## # Main ##############################################################
BLOCKS = [ BLOCKS = [
'problem-builder = problem_builder:MentoringBlock', 'problem-builder = problem_builder.mentoring:MentoringBlock',
'pb-table = problem_builder:MentoringTableBlock', 'pb-table = problem_builder.table:MentoringTableBlock',
'pb-column = problem_builder:MentoringTableColumn', 'pb-column = problem_builder.table:MentoringTableColumn',
'pb-answer = problem_builder:AnswerBlock', 'pb-answer = problem_builder.answer:AnswerBlock',
'pb-answer-recap = problem_builder:AnswerRecapBlock', 'pb-answer-recap = problem_builder.answer:AnswerRecapBlock',
'pb-mcq = problem_builder:MCQBlock', 'pb-mcq = problem_builder.mcq:MCQBlock',
'pb-rating = problem_builder:RatingBlock', 'pb-rating = problem_builder.mcq:RatingBlock',
'pb-mrq = problem_builder:MRQBlock', 'pb-mrq = problem_builder.mrq:MRQBlock',
'pb-message = problem_builder:MentoringMessageBlock', 'pb-message = problem_builder.message:MentoringMessageBlock',
'pb-tip = problem_builder:TipBlock', 'pb-tip = problem_builder.tip:TipBlock',
'pb-choice = problem_builder:ChoiceBlock', 'pb-choice = problem_builder.choice:ChoiceBlock',
'pb-dashboard = problem_builder:DashboardBlock', 'pb-dashboard = problem_builder.dashboard:DashboardBlock',
# Deprecated. You can temporarily uncomment and run 'python setup.py develop' if you have these blocks # Deprecated. You can temporarily uncomment and run 'python setup.py develop' if you have these blocks
# installed from testing mentoring v2 and need to get past an error message. # installed from testing mentoring v2 and need to get past an error message.
#'mentoring = problem_builder:MentoringBlock', # Deprecated alias for problem-builder #'mentoring = problem_builder:MentoringBlock', # Deprecated alias for problem-builder
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment