Update master with fixes from release

problem_builder/__init__.py

@@ -1,11 +0,0 @@
-from .mentoring import MentoringBlock, MentoringWithExplicitStepsBlock
-from .step import MentoringStepBlock, ReviewStepBlock
-from .answer import AnswerBlock, AnswerRecapBlock
-from .choice import ChoiceBlock
-from .dashboard import DashboardBlock
-from .instructor_tool import InstructorToolBlock
-from .mcq import MCQBlock, RatingBlock
-from .mrq import MRQBlock
-from .message import MentoringMessageBlock
-from .table import MentoringTableBlock, MentoringTableColumn
-from .tip import TipBlock

problem_builder/tasks.py

@@ -12,7 +12,7 @@
 from xmodule.modulestore.exceptions import ItemNotFoundError
 
 from .mcq import MCQBlock, RatingBlock
-from problem_builder import AnswerBlock
+from problem_builder.answer import AnswerBlock
 from .questionnaire import QuestionnaireAbstractBlock
 from .sub_api import sub_api
 

problem_builder/tests/integration/base_test.py

@@ -25,7 +25,7 @@
 # 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
 # accessed like a normal field.
-from problem_builder import MentoringBlock
+from problem_builder.mentoring import MentoringBlock
 MentoringBlock.url_name = String()
 
 loader = ResourceLoader(__name__)

problem_builder/tests/integration/test_mentoring.py

@@ -70,7 +70,7 @@ def assert_status_icon_color(self, color, question_text):
         ('apros', "#ff0000", "QUESTION")
     )
     def test_lms_theme_applied(self, theme, expected_color, question_text):
-        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)
             self.assert_status_icon_color(expected_color, question_text)
 

problem_builder/tests/integration/test_questionnaire.py

@@ -23,7 +23,7 @@
 import ddt
 from mock import patch, Mock
 
-from problem_builder import MentoringBlock
+from problem_builder.mentoring import MentoringBlock
 from .base_test import MentoringBaseTest
 
 

problem_builder/tests/unit/test_instructor_tool.py

@@ -5,7 +5,7 @@
 import unittest
 from mock import Mock, patch
 from xblock.field_data import DictFieldData
-from problem_builder import InstructorToolBlock
+from problem_builder.instructor_tool import InstructorToolBlock
 
 
 @ddt.ddt

problem_builder/tests/unit/test_mentoring.py

@@ -2,8 +2,8 @@
 import ddt
 from mock import MagicMock, Mock, patch
 from xblock.field_data import DictFieldData
-from problem_builder import MentoringBlock, MentoringMessageBlock, MCQBlock
-from problem_builder.mentoring import _default_theme_config
+from problem_builder.mcq import MCQBlock
+from problem_builder.mentoring import MentoringBlock, MentoringMessageBlock, _default_theme_config
 
 
 @ddt.ddt

problem_builder/tests/unit/test_migration.py

@@ -1,5 +1,5 @@
 import copy
-from problem_builder import MentoringBlock
+from problem_builder.mentoring import MentoringBlock
 from mock import MagicMock, Mock
 import unittest
 from xblock.field_data import DictFieldData

problem_builder/v1/tests/test_upgrade.py

@@ -22,7 +22,7 @@
 """
 import ddt
 from lxml import etree
-from problem_builder import MentoringBlock
+from problem_builder.mentoring import MentoringBlock
 from problem_builder.v1.xml_changes import convert_xml_to_v2
 import os.path
 from StringIO import StringIO

problem_builder/v1/upgrade.py

@@ -32,7 +32,7 @@
 import logging
 from lxml import etree
 from mentoring import MentoringBlock
-from problem_builder import MentoringBlock as NewMentoringBlock
+from problem_builder.mentoring import MentoringBlock as NewMentoringBlock
 from StringIO import StringIO
 import sys
 import warnings

setup.py

@@ -40,25 +40,25 @@ def package_data(pkg, root_list):
 # Main ##############################################################
 
 BLOCKS = [
-    'problem-builder = problem_builder:MentoringBlock',
-    'step-builder = problem_builder:MentoringWithExplicitStepsBlock',
-    'sb-step = problem_builder:MentoringStepBlock',
-    'sb-review-step = problem_builder:ReviewStepBlock',
+    'problem-builder = problem_builder.mentoring:MentoringBlock',
+    'step-builder = problem_builder.mentoring:MentoringWithExplicitStepsBlock',
+    'sb-step = problem_builder.step:MentoringStepBlock',
+    'sb-review-step = problem_builder.step:ReviewStepBlock',
 
-    'pb-table = problem_builder:MentoringTableBlock',
-    'pb-column = problem_builder:MentoringTableColumn',
-    'pb-answer = problem_builder:AnswerBlock',
-    'pb-answer-recap = problem_builder:AnswerRecapBlock',
-    'pb-mcq = problem_builder:MCQBlock',
-    'pb-rating = problem_builder:RatingBlock',
-    'pb-mrq = problem_builder:MRQBlock',
-    'pb-message = problem_builder:MentoringMessageBlock',
-    'pb-tip = problem_builder:TipBlock',
-    'pb-choice = problem_builder:ChoiceBlock',
+    'pb-table = problem_builder.table:MentoringTableBlock',
+    'pb-column = problem_builder.table:MentoringTableColumn',
+    'pb-answer = problem_builder.answer:AnswerBlock',
+    'pb-answer-recap = problem_builder.answer:AnswerRecapBlock',
+    'pb-mcq = problem_builder.mcq:MCQBlock',
+    'pb-rating = problem_builder.mcq:RatingBlock',
+    'pb-mrq = problem_builder.mrq:MRQBlock',
+    'pb-message = problem_builder.message:MentoringMessageBlock',
+    'pb-tip = problem_builder.tip:TipBlock',
+    'pb-choice = problem_builder.choice:ChoiceBlock',
 
-    'pb-dashboard = problem_builder:DashboardBlock',
-    'pb-data-export = problem_builder:InstructorToolBlock',  # Deprecated; use pb-instructor-tool instead
-    'pb-instructor-tool = problem_builder:InstructorToolBlock',
+    'pb-dashboard = problem_builder.dashboard:DashboardBlock',
+    'pb-data-export = problem_builder.instructor_tool:InstructorToolBlock',  # Deprecated; use 'pb-instructor-tool' now
+    'pb-instructor-tool = problem_builder.instructor_tool:InstructorToolBlock',
 ]
 
 setup(