refactor(tests): convert test_import.py tests to use NamedTuple fixtures

Author: tony

tests/cli/test_import.py

@@ -15,8 +15,28 @@
     import pathlib
 
 
-@pytest.mark.parametrize("cli_args", [(["import"])])
+class ImportTestFixture(t.NamedTuple):
+    """Test fixture for basic tmuxp import command tests."""
+
+    test_id: str
+    cli_args: list[str]
+
+
+IMPORT_TEST_FIXTURES: list[ImportTestFixture] = [
+    ImportTestFixture(
+        test_id="basic_import",
+        cli_args=["import"],
+    ),
+]
+
+
+@pytest.mark.parametrize(
+    list(ImportTestFixture._fields),
+    IMPORT_TEST_FIXTURES,
+    ids=[test.test_id for test in IMPORT_TEST_FIXTURES],
+)
 def test_import(
+    test_id: str,
     cli_args: list[str],
     tmp_path: pathlib.Path,
     monkeypatch: pytest.MonkeyPatch,
@@ -29,24 +49,40 @@ def test_import(
     assert "teamocil" in result.out
 
 
+class ImportTeamocilTestFixture(t.NamedTuple):
+    """Test fixture for tmuxp import teamocil command tests."""
+
+    test_id: str
+    cli_args: list[str]
+    inputs: list[str]
+
+
+IMPORT_TEAMOCIL_TEST_FIXTURES: list[ImportTeamocilTestFixture] = [
+    ImportTeamocilTestFixture(
+        test_id="import_teamocil_config_file",
+        cli_args=["import", "teamocil", "./.teamocil/config.yaml"],
+        inputs=["\n", "y\n", "./la.yaml\n", "y\n"],
+    ),
+    ImportTeamocilTestFixture(
+        test_id="import_teamocil_config_file_exists",
+        cli_args=["import", "teamocil", "./.teamocil/config.yaml"],
+        inputs=["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
+    ),
+    ImportTeamocilTestFixture(
+        test_id="import_teamocil_config_name",
+        cli_args=["import", "teamocil", "config"],
+        inputs=["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
+    ),
+]
+
+
 @pytest.mark.parametrize(
-    ("cli_args", "inputs"),
-    [
-        (
-            ["import", "teamocil", "./.teamocil/config.yaml"],
-            ["\n", "y\n", "./la.yaml\n", "y\n"],
-        ),
-        (
-            ["import", "teamocil", "./.teamocil/config.yaml"],
-            ["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
-        ),
-        (
-            ["import", "teamocil", "config"],
-            ["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
-        ),
-    ],
+    list(ImportTeamocilTestFixture._fields),
+    IMPORT_TEAMOCIL_TEST_FIXTURES,
+    ids=[test.test_id for test in IMPORT_TEAMOCIL_TEST_FIXTURES],
 )
 def test_import_teamocil(
+    test_id: str,
     cli_args: list[str],
     inputs: list[str],
     tmp_path: pathlib.Path,
@@ -76,24 +112,40 @@ def test_import_teamocil(
     assert new_config_yaml.exists()
 
 
+class ImportTmuxinatorTestFixture(t.NamedTuple):
+    """Test fixture for tmuxp import tmuxinator command tests."""
+
+    test_id: str
+    cli_args: list[str]
+    inputs: list[str]
+
+
+IMPORT_TMUXINATOR_TEST_FIXTURES: list[ImportTmuxinatorTestFixture] = [
+    ImportTmuxinatorTestFixture(
+        test_id="import_tmuxinator_config_file",
+        cli_args=["import", "tmuxinator", "./.tmuxinator/config.yaml"],
+        inputs=["\n", "y\n", "./la.yaml\n", "y\n"],
+    ),
+    ImportTmuxinatorTestFixture(
+        test_id="import_tmuxinator_config_file_exists",
+        cli_args=["import", "tmuxinator", "./.tmuxinator/config.yaml"],
+        inputs=["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
+    ),
+    ImportTmuxinatorTestFixture(
+        test_id="import_tmuxinator_config_name",
+        cli_args=["import", "tmuxinator", "config"],
+        inputs=["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
+    ),
+]
+
+
 @pytest.mark.parametrize(
-    ("cli_args", "inputs"),
-    [
-        (
-            ["import", "tmuxinator", "./.tmuxinator/config.yaml"],
-            ["\n", "y\n", "./la.yaml\n", "y\n"],
-        ),
-        (
-            ["import", "tmuxinator", "./.tmuxinator/config.yaml"],
-            ["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
-        ),
-        (
-            ["import", "tmuxinator", "config"],
-            ["\n", "y\n", "./exists.yaml\n", "./la.yaml\n", "y\n"],
-        ),
-    ],
+    list(ImportTmuxinatorTestFixture._fields),
+    IMPORT_TMUXINATOR_TEST_FIXTURES,
+    ids=[test.test_id for test in IMPORT_TMUXINATOR_TEST_FIXTURES],
 )
 def test_import_tmuxinator(
+    test_id: str,
     cli_args: list[str],
     inputs: list[str],
     tmp_path: pathlib.Path,