Run All checks passed! + manual addition of spaces

Author: notatallshaw

tests/functional/test_fast_deps.py

@@ -32,7 +32,7 @@ def assert_installed(script: PipTestEnvironment, names: str) -> None:
 
 @mark.network
 @mark.parametrize(
-    ("requirement", "expected"),
+    "requirement, expected",
     (
         ("Paste==3.4.2", ("Paste", "six")),
         ("Paste[flup]==3.4.2", ("Paste", "six", "flup")),
@@ -47,7 +47,7 @@ def test_install_from_pypi(
 
 @mark.network
 @mark.parametrize(
-    ("requirement", "expected"),
+    "requirement, expected",
     (
         ("Paste==3.4.2", ("Paste-3.4.2-*.whl", "six-*.whl")),
         ("Paste[flup]==3.4.2", ("Paste-3.4.2-*.whl", "six-*.whl", "flup-*")),

tests/unit/test_collector.py

@@ -259,7 +259,7 @@ def test_get_simple_response_dont_log_clear_text_password(
 
 
 @pytest.mark.parametrize(
-    ("path", "expected"),
+    "path, expected",
     [
         # Test a character that needs quoting.
         ("a b", "a%20b"),
@@ -299,7 +299,7 @@ def test_clean_url_path(path: str, expected: str, is_local_path: bool) -> None:
 
 
 @pytest.mark.parametrize(
-    ("path", "expected"),
+    "path, expected",
     [
         # Test a VCS path with a Windows drive letter and revision.
         pytest.param(
@@ -322,7 +322,7 @@ def test_clean_url_path_with_local_path(path: str, expected: str) -> None:
 
 
 @pytest.mark.parametrize(
-    ("url", "clean_url"),
+    "url, clean_url",
     [
         # URL with hostname and port. Port separator should not be quoted.
         (

tests/unit/test_index.py

@@ -819,7 +819,7 @@ def test_make_candidate_evaluator(
 
 
 @pytest.mark.parametrize(
-    ("fragment", "canonical_name", "expected"),
+    "fragment, canonical_name, expected",
     [
         # Trivial.
         ("pip-18.0", "pip", 3),
@@ -851,7 +851,7 @@ def test_find_name_version_sep(
 
 
 @pytest.mark.parametrize(
-    ("fragment", "canonical_name"),
+    "fragment, canonical_name",
     [
         # A dash must follow the package name.
         ("zope.interface4.5.0", "zope-interface"),
@@ -868,7 +868,7 @@ def test_find_name_version_sep_failure(fragment: str, canonical_name: str) -> No
 
 
 @pytest.mark.parametrize(
-    ("fragment", "canonical_name", "expected"),
+    "fragment, canonical_name, expected",
     [
         # Trivial.
         ("pip-18.0", "pip", "18.0"),

tests/unit/test_network_auth.py

@@ -20,7 +20,7 @@ def reset_keyring() -> Iterable[None]:
 
 
 @pytest.mark.parametrize(
-    ["input_url", "url", "username", "password"],
+    "input_url, url, username, password",
     [
         (
             "http://user%40email.com:[email protected]/path",

tests/unit/test_network_utils.py

@@ -6,7 +6,7 @@
 
 
 @pytest.mark.parametrize(
-    ("status_code", "error_type"),
+    "status_code, error_type",
     [
         (401, "Client Error"),
         (501, "Server Error"),

tests/unit/test_pep517.py

@@ -10,7 +10,7 @@
 
 
 @pytest.mark.parametrize(
-    ("source", "expected"),
+    "source, expected",
     [
         ("pep517_setup_and_pyproject", True),
         ("pep517_setup_only", False),
@@ -45,7 +45,7 @@ def test_use_pep517_rejects_setup_cfg_only(shared_data: TestData) -> None:
 
 
 @pytest.mark.parametrize(
-    ("source", "msg"),
+    "source, msg",
     [
         ("pep517_setup_and_pyproject", "specifies a build backend"),
         ("pep517_pyproject_only", "does not have a setup.py"),
@@ -71,7 +71,7 @@ def test_disabling_pep517_invalid(shared_data: TestData, source: str, msg: str)
 
 
 @pytest.mark.parametrize(
-    ("spec",), [("./foo",), ("git+https://example.com/pkg@dev#egg=myproj",)]
+    "spec", [("./foo",), ("git+https://example.com/pkg@dev#egg=myproj",)]
 )
 def test_pep517_parsing_checks_requirements(tmpdir: Path, spec: str) -> None:
     tmpdir.joinpath("pyproject.toml").write_text(

tests/unit/test_pyproject_config.py

@@ -6,7 +6,7 @@
 
 
 @pytest.mark.parametrize(
-    ("command", "expected"),
+    "command, expected",
     [
         ("install", True),
         ("wheel", True),
@@ -39,7 +39,7 @@ def test_set_config_empty_value() -> None:
 
 
 @pytest.mark.parametrize(
-    ("passed", "expected"),
+    "passed, expected",
     [
         (["x=hello", "x=world"], {"x": ["hello", "world"]}),
         (["x=hello", "x=world", "x=other"], {"x": ["hello", "world", "other"]}),

tests/unit/test_self_check_outdated.py

@@ -16,7 +16,7 @@
 
 
 @pytest.mark.parametrize(
-    ["key", "expected"],
+    "key, expected",
     [
         (
             "/hello/world/venv",
@@ -59,7 +59,7 @@ def test_pip_self_version_check_calls_underlying_implementation(
 
 
 @pytest.mark.parametrize(
-    [
+    [  # noqa: PT006 - String representation is too long
         "installed_version",
         "remote_version",
         "stored_version",

tests/unit/test_utils.py

@@ -559,7 +559,7 @@ def test_normalize_version_info(
 
 class TestGetProg:
     @pytest.mark.parametrize(
-        ("argv", "executable", "expected"),
+        "argv, executable, expected",
         [
             ("/usr/bin/pip", "", "pip"),
             ("-c", "/usr/bin/python", "/usr/bin/python -m pip"),
@@ -1061,7 +1061,7 @@ def test_format_size(size: int, expected: str) -> None:
 
 
 @pytest.mark.parametrize(
-    ("rows", "table", "sizes"),
+    "rows, table, sizes",
     [
         ([], [], []),
         (

tests/unit/test_utils_subprocess.py

@@ -39,7 +39,7 @@ def test_format_command_args(args: CommandArgs, expected: str) -> None:
 
 
 @pytest.mark.parametrize(
-    ("stdout_only", "expected"),
+    "stdout_only, expected",
     [
         (True, ("out\n", "out\r\n")),
         (False, ("out\nerr\n", "out\r\nerr\r\n", "err\nout\n", "err\r\nout\r\n")),
@@ -312,7 +312,7 @@ def test_info_logging_with_show_stdout_true(
         )
 
     @pytest.mark.parametrize(
-        ("exit_status", "show_stdout", "extra_ok_returncodes", "log_level", "expected"),
+        "exit_status, show_stdout, extra_ok_returncodes, log_level, expected",
         [
             # The spinner should show here because show_stdout=False means
             # the subprocess should get logged at DEBUG level, but the passed