refactor: helper: Fix WSL normalized_file_paths related tests.

Addressed issues with file path separators `\\` specific to WSL environments.
zulip · Jun 27, 2024 · 40d4425 · 40d4425
1 parent 440a3fa
commit 40d4425
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/tests/helper/test_helper.py b/tests/helper/test_helper.py
@@ -573,6 +573,7 @@ def test_process_media(
     )
     mocked_open_media = mocker.patch(MODULE + ".open_media")
     mocker.patch(MODULE + ".PLATFORM", platform)
+    mocker.patch("zulipterminal.platform_code.PLATFORM", platform)
     mocker.patch("zulipterminal.core.Controller.show_media_confirmation_popup")
 
     process_media(controller, link)

diff --git a/zulipterminal/helper.py b/zulipterminal/helper.py
@@ -784,6 +784,7 @@ def process_media(controller: Any, link: str) -> None:
         controller.view.set_footer_text, "Downloading your media..."
     )
     media_path = download_media(controller, link, show_download_status)
+    media_path = normalized_file_path(media_path)
     tool = ""
 
     # TODO: Add support for other platforms as well.
@@ -825,7 +826,7 @@ def download_media(
                     show_download_status()
 
         controller.report_success([" Downloaded ", ("bold", media_name)])
-        return normalized_file_path(local_path)
+        return local_path
 
 
 @asynch