(airflow) branch main updated: Enable PT006 rule to sftp Provider test (#57924)

[shahar]

This is an automated email from the ASF dual-hosted git repository.

shahar pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 4d7c077e450 Enable PT006 rule to sftp Provider test (#57924)
4d7c077e450 is described below

commit 4d7c077e450366ad97597ce8edcd2237eaa79213
Author: ChenChen Lai <72776271+0l...@users.noreply.github.com>
AuthorDate: Sat Nov 8 02:28:03 2025 +0800

    Enable PT006 rule to sftp Provider test (#57924)
---
 providers/sftp/tests/unit/sftp/hooks/test_sftp.py     | 6 +++---
 providers/sftp/tests/unit/sftp/operators/test_sftp.py | 4 ++--
 providers/sftp/tests/unit/sftp/sensors/test_sftp.py   | 4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/providers/sftp/tests/unit/sftp/hooks/test_sftp.py 
b/providers/sftp/tests/unit/sftp/hooks/test_sftp.py
index 55a9fb9dc62..2342b93d2f0 100644
--- a/providers/sftp/tests/unit/sftp/hooks/test_sftp.py
+++ b/providers/sftp/tests/unit/sftp/hooks/test_sftp.py
@@ -442,7 +442,7 @@ class TestSFTPHook:
         assert hook.key_file == TEST_KEY_FILE
 
     @pytest.mark.parametrize(
-        "path, exists",
+        ("path", "exists"),
         [
             (TMP_DIR_FOR_TESTS, True),
             (TMP_FILE_FOR_TESTS, True),
@@ -456,7 +456,7 @@ class TestSFTPHook:
         assert result == exists
 
     @pytest.mark.parametrize(
-        "path, prefix, delimiter, match",
+        ("path", "prefix", "delimiter", "match"),
         [
             ("test/path/file.bin", None, None, True),
             ("test/path/file.bin", "test", None, True),
@@ -762,7 +762,7 @@ class TestSFTPHookAsync:
         mock_connect.assert_called_with(**expected_connection_details)
 
     @pytest.mark.parametrize(
-        "mock_port, mock_host_key",
+        ("mock_port", "mock_host_key"),
         [
             (22, "ssh-ed25519 
AAAAC3NzaC1lZDI1NTE5AAAAIFe8P8lk5HFfL/rMlcCMHQhw1cg+uZtlK5rXQk2C4pOY"),
             (2222, 
"AAAAC3NzaC1lZDI1NTE5AAAAIFe8P8lk5HFfL/rMlcCMHQhw1cg+uZtlK5rXQk2C4pOY"),
diff --git a/providers/sftp/tests/unit/sftp/operators/test_sftp.py 
b/providers/sftp/tests/unit/sftp/operators/test_sftp.py
index 984815fef1a..df9460fe764 100644
--- a/providers/sftp/tests/unit/sftp/operators/test_sftp.py
+++ b/providers/sftp/tests/unit/sftp/operators/test_sftp.py
@@ -593,7 +593,7 @@ class TestSFTPOperator:
             ).execute(None)
 
     @pytest.mark.parametrize(
-        "operation, expected",
+        ("operation", "expected"),
         TEST_GET_PUT_PARAMS,
     )
     @mock.patch("airflow.providers.ssh.hooks.ssh.SSHHook.get_conn", 
spec=paramiko.SSHClient)
@@ -624,7 +624,7 @@ class TestSFTPOperator:
         assert lineage.outputs == expected[1]
 
     @pytest.mark.parametrize(
-        "operation, expected",
+        ("operation", "expected"),
         TEST_GET_PUT_PARAMS,
     )
     @mock.patch("airflow.providers.ssh.hooks.ssh.SSHHook.get_conn", 
spec=paramiko.SSHClient)
diff --git a/providers/sftp/tests/unit/sftp/sensors/test_sftp.py 
b/providers/sftp/tests/unit/sftp/sensors/test_sftp.py
index c7f84ee1794..aa8a3497713 100644
--- a/providers/sftp/tests/unit/sftp/sensors/test_sftp.py
+++ b/providers/sftp/tests/unit/sftp/sensors/test_sftp.py
@@ -242,7 +242,7 @@ class TestSFTPSensor:
         assert not output
 
     @pytest.mark.parametrize(
-        "op_args, op_kwargs,",
+        ("op_args", "op_kwargs"),
         [
             pytest.param(("op_arg_1",), {"key": "value"}),
             pytest.param((), {}),
@@ -274,7 +274,7 @@ class TestSFTPSensor:
         }
 
     @pytest.mark.parametrize(
-        "op_args, op_kwargs,",
+        ("op_args", "op_kwargs"),
         [
             pytest.param(("op_arg_1",), {"key": "value"}),
             pytest.param((), {}),