feedback

mitodl · Jan 2, 2025 · 6865867 · 6865867
1 parent eee1694
commit 6865867
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/learning_resources/etl/utils.py b/learning_resources/etl/utils.py
@@ -823,12 +823,12 @@ def parse_resource_duration(duration_str: str) -> DurationConfig:
         DurationConfig: the standardized duration
     """
     if duration_str:
-        duration_regex = re.compile(r"(\d+)\D+(\d+)?\s*(\w+)?", re.IGNORECASE)
+        duration_regex = re.compile(r"(\d+)\s*(to|-)*\s*(\d+)?\s*(\w+)?", re.IGNORECASE)
         interval = transform_interval(duration_str)
         match = duration_regex.match(duration_str.lower().strip())
         if match and interval:
             dmin = match.group(1)
-            dmax = match.group(2)
+            dmax = match.group(3)
             return DurationConfig(
                 duration=duration_str,
                 min_weeks=calculate_weeks(int(dmin), interval.lower()),

diff --git a/learning_resources/etl/utils_test.py b/learning_resources/etl/utils_test.py
@@ -518,6 +518,8 @@ def test_parse_string_to_int(hour, expected):
         ("2 - 3 semanas", 2, 3),
         ("Unparseable duration", None, None),
         ("", None, None),
+        ("2 days in person+3 live webinars", 1, 1),
+        ("2 weeks in person+3 live webinars", 2, 2),
     ],
 )
 def test_parse_resource_duration(raw_value, min_weeks, max_weeks):