PR feedback

Signed-off-by: Bernát Gábor <[email protected]>

Author: gaborbernat

src/tox/config/set_env.py

@@ -24,22 +24,17 @@ def __init__(  # noqa: C901, PLR0912
         from .loader.replacer import MatchExpression, find_replace_expr  # noqa: PLC0415
 
         if isinstance(raw, dict):
-            # TOML 'file' attribute is to be handled separately later
-            self._raw = dict(raw)
-            if "file" in raw:
+            self._raw = raw
+            if "file" in raw:  # environment files to be handled later
                 self._env_files.append(raw["file"])
                 self._raw.pop("file")
-
             return
-
         if isinstance(raw, list):
             self._raw = reduce(lambda a, b: {**a, **b}, raw)
             return
-
         for line in raw.splitlines():  # noqa: PLR1702
             if line.strip():
-                # INI 'file|' attribute is to be handled separately later
-                if line.startswith("file|"):
+                if line.startswith("file|"):  # environment files to be handled later
                     self._env_files.append(line[len("file|") :])
                 else:
                     try:

tests/config/test_set_env.py

@@ -51,14 +51,14 @@ def test_set_env_bad_line() -> None:
         SetEnv("A", "py", "py", Path())
 
 
-_ConfType = Literal["ini", "toml"]
+ConfigFileFormat = Literal["ini", "toml"]
 
 
 class EvalSetEnv(Protocol):
     def __call__(
         self,
         config: str,
-        conf_type: _ConfType = "ini",
+        conf_type: ConfigFileFormat = "ini",
         extra_files: dict[str, Any] | None = ...,
         from_cwd: Path | None = ...,
     ) -> SetEnv: ...
@@ -68,14 +68,11 @@ def __call__(
 def eval_set_env(tox_project: ToxProjectCreator) -> EvalSetEnv:
     def func(
         config: str,
-        conf_type: _ConfType = "ini",
+        conf_type: ConfigFileFormat = "ini",
         extra_files: dict[str, Any] | None = None,
         from_cwd: Path | None = None,
     ) -> SetEnv:
-        if conf_type == "ini":
-            prj = tox_project({"tox.ini": config, **(extra_files or {})})
-        else:
-            prj = tox_project({"tox.toml": config, **(extra_files or {})})
+        prj = tox_project({f"tox.{conf_type}": config, **(extra_files or {})})
         result = prj.run("c", "-k", "set_env", "-e", "py", from_cwd=None if from_cwd is None else prj.path / from_cwd)
         result.assert_success()
         set_env: SetEnv = result.env_conf("py")["set_env"]
@@ -172,7 +169,7 @@ def test_set_env_honor_override(eval_set_env: EvalSetEnv) -> None:
     ],
 )
 def test_set_env_environment_file(
-    conf_type: _ConfType, config: str, eval_set_env: EvalSetEnv, monkeypatch: MonkeyPatch
+    conf_type: ConfigFileFormat, config: str, eval_set_env: EvalSetEnv, monkeypatch: MonkeyPatch
 ) -> None:
     env_file = """
     A=1
@@ -211,7 +208,7 @@ def test_set_env_environment_file(
     ],
 )
 def test_set_env_environment_file_combined_with_normal_setting(
-    conf_type: _ConfType, config: str, eval_set_env: EvalSetEnv, monkeypatch: MonkeyPatch
+    conf_type: ConfigFileFormat, config: str, eval_set_env: EvalSetEnv, monkeypatch: MonkeyPatch
 ) -> None:
     env_file = """
     A=1