Commit a5f405c0 by McKenzie Welter Committed by McKenzie Welter

default to populate max_effort with only 1 effort value present

parent 4ac90b9f
......@@ -542,7 +542,7 @@ class CourseMarketingSiteDataLoader(AbstractMarketingSiteDataLoader):
effort_values = [int(keyword) for keyword in re.split(r'\s|-|–|,|\+|~', effort_per_week) if keyword.isdigit()]
if len(effort_values) == 1:
min_effort = effort_values[0]
max_effort = effort_values[0]
if len(effort_values) == 2:
min_effort = effort_values[0]
max_effort = effort_values[1]
......
......@@ -375,12 +375,12 @@ class CourseMarketingSiteDataLoaderTests(AbstractMarketingSiteDataLoaderTestMixi
(None, None, None),
('Browse at your own pace.', None, None),
('1.5 - 3.5 hours/week', None, None),
('8 hours/week', 8, None),
('2.5-5 hours.', 5, None),
('5+ hours per week', 5, None),
('3 horas por semana', 3, None),
('1 - 1.5 hours per week', 1, None),
('6 hours of video/300 multiple choice questions', 6, None),
('8 hours/week', None, 8),
('2.5-5 hours.', None, 5),
('5+ hours per week', None, 5),
('3 horas por semana', None, 3),
('1 - 1.5 hours per week', None, 1),
('6 hours of video/300 multiple choice questions', None, 6),
('6 to 9 hours/week', 6, 9),
('4-6 hours per week', 4, 6),
('About 5-12 hrs/week.', 5, 12),
......
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