Get rid of some debug logging

Author: mbertrand

learning_resources/etl/utils.py

@@ -467,16 +467,13 @@ def get_url_from_module_id(
         else run.run_id
     )
     if module_id.startswith("asset"):
-        log.debug("Getting URL for asset %s", module_id)
         asset_meta = (
             assets_metadata.get(Path(olx_path).parts[-1], {}) if assets_metadata else {}
         )
         video_meta = video_srt_metadata.get(module_id, {}) if video_srt_metadata else {}
         if video_meta:
-            log.debug("Found video metadata for %s", module_id)
+            # Link to the parent video
             return f"{root_url}/courses/{run_id}/jump_to/{video_meta.split('@')[-1]}"
-        elif module_id.endswith(".srt"):
-            log.debug("No video metadata for %s", module_id)
         middle_path = asset_meta.get("custom_md5", "")
         return f"{root_url}/{(middle_path + '/') if middle_path else ''}{module_id}"
     elif module_id.startswith("block") and is_valid_uuid(module_id.split("@")[-1]):
@@ -531,17 +528,15 @@ def parse_video_transcripts_xml(
 
 def get_video_metadata(olx_path: str, run: LearningResourceRun) -> dict:
     """
-    Get metadata for video SRT files in an OLX path
+    Get metadata for video SRT/VTT files in an OLX path
     """
     video_transcript_mapping = {}
     video_path = Path(olx_path, "video")
     if not video_path.exists():
         log.warning("No video directory found in OLX path: %s", olx_path)
         return video_transcript_mapping
     for root, _, files in os.walk(str(Path(olx_path, "video"))):
-        path = "/".join(root.split("/")[3:])
         for filename in files:
-            log.debug("Processing video file %s in %s", filename, path)
             extension_lower = Path(filename).suffix.lower()
             if extension_lower == ".xml":
                 with Path.open(Path(root, filename), "rb") as f:

learning_resources/etl/utils_test.py

@@ -711,17 +711,13 @@ def test_get_video_metadata(mocker, tmp_path, video_dir_exists):
             "learning_resources.etl.utils.parse_video_transcripts_xml",
             return_value=expected_mapping,
         )
-
         result = utils.get_video_metadata(str(olx_path), run)
 
         assert result == expected_mapping
-        # The function passes a file object to parse_video_transcripts_xml, not a Path
         assert mock_parse.call_count == 1
         call_args = mock_parse.call_args[0]
         assert call_args[0] == run
         assert call_args[1] == video_xml
-        # The third argument is a file object, so we can't easily check its exact value
-        mock_log.debug.assert_called()
     else:
         # No video directory
         result = utils.get_video_metadata(str(olx_path), run)