ECOM-1592: code refactoring
This commit is contained in:
@@ -136,13 +136,13 @@ def _is_credit_requirement(xblock):
|
|||||||
if not callable(getattr(xblock, "get_credit_requirement_namespace", None)):
|
if not callable(getattr(xblock, "get_credit_requirement_namespace", None)):
|
||||||
is_credit_requirement = False
|
is_credit_requirement = False
|
||||||
LOGGER.error(
|
LOGGER.error(
|
||||||
"XBlock %v is marked as a credit requirement but does not "
|
"XBlock %s is marked as a credit requirement but does not "
|
||||||
"implement get_credit_requirement_namespace()", xblock
|
"implement get_credit_requirement_namespace()", unicode(xblock)
|
||||||
)
|
)
|
||||||
if not callable(getattr(xblock, "get_credit_requirement_name", None)):
|
if not callable(getattr(xblock, "get_credit_requirement_name", None)):
|
||||||
is_credit_requirement = False
|
is_credit_requirement = False
|
||||||
LOGGER.error(
|
LOGGER.error(
|
||||||
"XBlock %v is marked as a credit requirement but does not "
|
"XBlock %s is marked as a credit requirement but does not "
|
||||||
"implement get_credit_requirement_name()", xblock
|
"implement get_credit_requirement_name()", unicode(xblock)
|
||||||
)
|
)
|
||||||
return is_credit_requirement
|
return is_credit_requirement
|
||||||
|
|||||||
@@ -32,7 +32,7 @@ class TestTaskExecution(ModuleStoreTestCase):
|
|||||||
section = ItemFactory.create(parent=self.course, category='chapter', display_name='Test Section')
|
section = ItemFactory.create(parent=self.course, category='chapter', display_name='Test Section')
|
||||||
subsection = ItemFactory.create(parent=section, category='sequential', display_name='Test Subsection')
|
subsection = ItemFactory.create(parent=section, category='sequential', display_name='Test Subsection')
|
||||||
vertical = ItemFactory.create(parent=subsection, category='vertical', display_name='Test Unit')
|
vertical = ItemFactory.create(parent=subsection, category='vertical', display_name='Test Unit')
|
||||||
reverification = ItemFactory.create(
|
ItemFactory.create(
|
||||||
parent=vertical,
|
parent=vertical,
|
||||||
category='edx-reverification-block',
|
category='edx-reverification-block',
|
||||||
display_name='Test Verification Block'
|
display_name='Test Verification Block'
|
||||||
|
|||||||
Reference in New Issue
Block a user