From ee6f7e1b3087767692bbf28053efbdf0dcf3e370 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 14 Feb 2026 00:40:09 +0000 Subject: [PATCH] fix: move imports to module level per code review feedback Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com> --- tests/test_filename_utils.py | 3 --- tests/test_upload_to_sftp.py | 3 +-- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/tests/test_filename_utils.py b/tests/test_filename_utils.py index 1a7c8e75..20f07249 100644 --- a/tests/test_filename_utils.py +++ b/tests/test_filename_utils.py @@ -493,9 +493,6 @@ class TestExtractRemotePathEdgeCases: def test_windows_style_separators(self, monkeypatch): """Test handling of Windows-style path separators.""" - # Temporarily change os.sep to backslash - import os - result = extract_remote_path("/base/subdir/file.pdf", "/base", "/remote") # Should use forward slashes in output assert "\\" not in result diff --git a/tests/test_upload_to_sftp.py b/tests/test_upload_to_sftp.py index cea5ed56..46533a76 100644 --- a/tests/test_upload_to_sftp.py +++ b/tests/test_upload_to_sftp.py @@ -3,6 +3,7 @@ import os from unittest.mock import MagicMock, Mock, patch +import paramiko import pytest from app.tasks.upload_to_sftp import upload_to_sftp @@ -89,8 +90,6 @@ class TestUploadToSFTP: @patch("app.tasks.upload_to_sftp.settings") def test_upload_with_disabled_host_key_verification(self, mock_settings, mock_ssh_class, tmp_path): """Test SFTP upload with host key verification disabled.""" - import paramiko - test_file = tmp_path / "test.pdf" test_file.write_text("test content")