From 40d4425c7227f41c540c441184e151f93ae7b7e5 Mon Sep 17 00:00:00 2001 From: rsashank Date: Wed, 5 Jun 2024 23:52:40 +0530 Subject: [PATCH] refactor: helper: Fix WSL normalized_file_paths related tests. Addressed issues with file path separators `\\` specific to WSL environments. --- tests/helper/test_helper.py | 1 + zulipterminal/helper.py | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/helper/test_helper.py b/tests/helper/test_helper.py index 2e32e5c10f..216b03e16a 100644 --- 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 index 76d0a0158d..a71d055b74 100644 --- 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