STY: Apply ruff/flake8-pytest-style rule PT006

PT006 Wrong type passed to first argument of `@pytest.mark.parametrize`; expected `tuple`

Author: DimitriPapadopoulos

nipype/algorithms/tests/test_CompCor.py

@@ -28,7 +28,7 @@ def close_up_to_column_sign(a, b, rtol=1e-05, atol=1e-08, equal_nan=False):
 
 
 @pytest.mark.parametrize(
-    "a, b, close",
+    ("a", "b", "close"),
     [
         ([[0.1, 0.2], [0.3, 0.4]], [[-0.1, 0.2], [-0.3, 0.4]], True),
         ([[0.1, 0.2], [0.3, 0.4]], [[-0.1, 0.2], [0.3, -0.4]], False),

nipype/algorithms/tests/test_stats.py

@@ -22,7 +22,7 @@ def test_ActivationCount(tmpdir):
 
 
 @pytest.mark.parametrize(
-    "threshold, above_thresh",
+    ("threshold", "above_thresh"),
     [
         (1, 15.865),  # above one standard deviation (one side)
         (2, 2.275),  # above two standard deviations (one side)

nipype/interfaces/ants/tests/test_base.py

@@ -20,6 +20,6 @@
 # fmt: on
 
 
-@pytest.mark.parametrize("raw_info, version", ANTS_VERSIONS)
+@pytest.mark.parametrize(("raw_info", "version"), ANTS_VERSIONS)
 def test_version_parser(raw_info, version):
     assert Info.parse_version(raw_info) == version

nipype/interfaces/base/tests/test_resource_monitor.py

@@ -55,7 +55,9 @@ class UseResources(CommandLine):
 
 @pytest.mark.skip(reason="inconsistent readings")
 @pytest.mark.skipif(os.getenv("CI_SKIP_TEST", False), reason="disabled in CI tests")
-@pytest.mark.parametrize("mem_gb,n_procs", [(0.5, 3), (2.2, 8), (0.8, 4), (1.5, 1)])
+@pytest.mark.parametrize(
+    ("mem_gb", "n_procs"), [(0.5, 3), (2.2, 8), (0.8, 4), (1.5, 1)]
+)
 def test_cmdline_profiling(tmpdir, mem_gb, n_procs, use_resource_monitor):
     """
     Test runtime profiler correctly records workflow RAM/CPUs consumption
@@ -80,7 +82,9 @@ def test_cmdline_profiling(tmpdir, mem_gb, n_procs, use_resource_monitor):
 @pytest.mark.skipif(
     True, reason="test disabled temporarily, until function profiling works"
 )
-@pytest.mark.parametrize("mem_gb,n_procs", [(0.5, 3), (2.2, 8), (0.8, 4), (1.5, 1)])
+@pytest.mark.parametrize(
+    ("mem_gb", "n_procs"), [(0.5, 3), (2.2, 8), (0.8, 4), (1.5, 1)]
+)
 def test_function_profiling(tmpdir, mem_gb, n_procs, use_resource_monitor):
     """
     Test runtime profiler correctly records workflow RAM/CPUs consumption

nipype/interfaces/fsl/tests/test_base.py

@@ -60,7 +60,7 @@ def test_FSLCommand2():
 
 @pytest.mark.skipif(no_fsl(), reason="fsl is not installed")
 @pytest.mark.parametrize(
-    "args, desired_name",
+    ("args", "desired_name"),
     [
         ({}, {"file": "foo.nii.gz"}),  # just the filename
         # filename with suffix

nipype/interfaces/fsl/tests/test_preprocess.py

@@ -588,7 +588,7 @@ def setup_fugue(tmpdir):
 
 @pytest.mark.skipif(no_fsl(), reason="fsl is not installed")
 @pytest.mark.parametrize(
-    "attr, out_file",
+    ("attr", "out_file"),
     [
         (
             {

nipype/interfaces/tests/test_dcm2nii.py

@@ -5,7 +5,7 @@
 
 
 @pytest.mark.parametrize(
-    "fname, extension, search_crop",
+    ("fname", "extension", "search_crop"),
     [
         ("output_1", ".txt", False),
         ("output_w_[]_meta_1", ".json", False),

nipype/interfaces/tests/test_io.py

@@ -104,7 +104,7 @@ def test_s3datagrabber():
 
 
 @pytest.mark.parametrize(
-    "SF_args, inputs_att, expected",
+    ("SF_args", "inputs_att", "expected"),
     [
         (
             {"templates": templates1},

nipype/interfaces/utility/tests/test_base.py

@@ -37,7 +37,7 @@ def test_rename(tmpdir):
 
 
 @pytest.mark.parametrize(
-    "args, expected", [({}, ([0], [1, 2, 3])), ({"squeeze": True}, (0, [1, 2, 3]))]
+    ("args", "expected"), [({}, ([0], [1, 2, 3])), ({"squeeze": True}, (0, [1, 2, 3]))]
 )
 def test_split(tmpdir, args, expected):
     tmpdir.chdir()
@@ -52,7 +52,7 @@ def test_split(tmpdir, args, expected):
 
 
 @pytest.mark.parametrize(
-    "args, kwargs, in_lists, expected",
+    ("args", "kwargs", "in_lists", "expected"),
     [
         ([3], {}, [0, [1, 2], [3, 4, 5]], [0, 1, 2, 3, 4, 5]),
         ([0], {}, None, None),

nipype/pipeline/engine/tests/test_engine.py

@@ -16,7 +16,7 @@
 # XXX - SG I'll create a graphical version of these tests and actually
 # ensure that all connections are tested later
 @pytest.mark.parametrize(
-    "iterables, expected",
+    ("iterables", "expected"),
     [
         ({"1": None}, (1, 0)),  # test1
         ({"1": dict(input1=lambda: [1, 2], input2=lambda: [1, 2])}, (4, 0)),  # test2
@@ -34,7 +34,7 @@ def test_1mod(iterables, expected):
 
 
 @pytest.mark.parametrize(
-    "iterables, expected",
+    ("iterables", "expected"),
     [
         ({"1": {}, "2": dict(input1=lambda: [1, 2])}, (3, 2)),  # test3
         ({"1": dict(input1=lambda: [1, 2]), "2": {}}, (4, 2)),  # test4
@@ -58,7 +58,7 @@ def test_2mods(iterables, expected):
 
 
 @pytest.mark.parametrize(
-    "iterables, expected, connect",
+    ("iterables", "expected", "connect"),
     [
         (
             {"1": {}, "2": dict(input1=lambda: [1, 2]), "3": {}},