test(cli): add cat_file_edits_steering_enabled to session applicator fixture

Keeps empty_args Namespace aligned with new CLI dest for cat file edits steering (Windows CRLF normalization may widen the diff).

Made-with: Cursor

Author: Mateusz

tests/unit/core/cli_support/applicators/test_session_applicator.py

@@ -34,15 +34,15 @@ def applicator(self):
     @pytest.fixture
     def empty_args(self) -> CliArgs:
         """Create empty CLI arguments namespace."""
-        return argparse.Namespace(
-            disable_interactive_mode=None,
-            force_set_project=None,
-            project_dir_resolution_model=None,
-            project_dir_resolution_mode=None,
-            project_dir_resolution_filesystem_mode=None,
-            disable_default_openrouter_project_dir_resolution_fallback=None,
-            disable_interactive_commands=None,
-            quality_verifier_model=None,
+        return argparse.Namespace(
+            disable_interactive_mode=None,
+            force_set_project=None,
+            project_dir_resolution_model=None,
+            project_dir_resolution_mode=None,
+            project_dir_resolution_filesystem_mode=None,
+            disable_default_openrouter_project_dir_resolution_fallback=None,
+            disable_interactive_commands=None,
+            quality_verifier_model=None,
             quality_verifier_frequency=None,
             quality_verifier_ttft_timeout_seconds=None,
             quality_verifier_tool_followup_weight=None,
@@ -55,13 +55,14 @@ def empty_args(self) -> CliArgs:
             planning_phase_reasoning_effort=None,
             planning_phase_thinking_budget=None,
             pytest_full_suite_steering_enabled=None,
+            cat_file_edits_steering_enabled=None,
             pytest_context_saving_enabled=None,
             test_execution_reminder_enabled=None,
             fix_think_tags_enabled=None,
-            disable_dangerous_git_commands_protection=None,
-            disable_double_ampersand_fixes_for_windows=None,
-            disable_auto_continue_removal=None,
-            droid_path_fix_enabled=None,
+            disable_dangerous_git_commands_protection=None,
+            disable_double_ampersand_fixes_for_windows=None,
+            disable_auto_continue_removal=None,
+            droid_path_fix_enabled=None,
             tool_access_allowed_tools=None,
             tool_access_blocked_tools=None,
             tool_access_default_policy=None,
@@ -111,7 +112,7 @@ def test_apply_force_set_project(
             assert overrides["session"].get("force_set_project") is True
             assert os.environ.get("FORCE_SET_PROJECT") == "true"
 
-    def test_apply_planning_phase_enabled(
+    def test_apply_planning_phase_enabled(
         self,
         applicator,
         empty_args: CliArgs,
@@ -123,62 +124,62 @@ def test_apply_planning_phase_enabled(
         applicator.apply(empty_args, overrides, resolution)
 
         assert "session" in overrides
-        assert "planning_phase" in overrides["session"]
-        assert overrides["session"]["planning_phase"].get("enabled") is True
-
-    def test_apply_project_dir_resolution_filesystem_mode(
-        self,
-        applicator,
-        empty_args: CliArgs,
-        overrides: CliOverrides,
-        resolution: ParameterResolution,
-    ) -> None:
-        """Test that project_dir_resolution_filesystem_mode is applied correctly."""
-        empty_args.project_dir_resolution_filesystem_mode = "disabled"
-        applicator.apply(empty_args, overrides, resolution)
-
-        assert "session" in overrides
-        assert (
-            overrides["session"].get("project_dir_resolution_filesystem_mode")
-            == "disabled"
-        )
-        cli_records = resolution.latest_by_source(ParameterSource.CLI)
-        assert "session.project_dir_resolution_filesystem_mode" in cli_records
-        assert (
-            cli_records["session.project_dir_resolution_filesystem_mode"].origin
-            == "--project-dir-resolution-filesystem-mode"
-        )
-
-    def test_apply_disable_default_openrouter_project_dir_resolution_fallback(
-        self,
-        applicator,
-        empty_args: CliArgs,
-        overrides: CliOverrides,
-        resolution: ParameterResolution,
-    ) -> None:
-        empty_args.disable_default_openrouter_project_dir_resolution_fallback = True
-        applicator.apply(empty_args, overrides, resolution)
-
-        assert "session" in overrides
-        assert (
-            overrides["session"].get(
-                "disable_default_openrouter_project_dir_resolution_fallback"
-            )
-            is True
-        )
-        cli_records = resolution.latest_by_source(ParameterSource.CLI)
-        assert (
-            "session.disable_default_openrouter_project_dir_resolution_fallback"
-            in cli_records
-        )
-        assert (
-            cli_records[
-                "session.disable_default_openrouter_project_dir_resolution_fallback"
-            ].origin
-            == "--disable-default-openrouter-project-dir-resolution-fallback"
-        )
-
-    def test_apply_quality_verifier_ttft_timeout_seconds(
+        assert "planning_phase" in overrides["session"]
+        assert overrides["session"]["planning_phase"].get("enabled") is True
+
+    def test_apply_project_dir_resolution_filesystem_mode(
+        self,
+        applicator,
+        empty_args: CliArgs,
+        overrides: CliOverrides,
+        resolution: ParameterResolution,
+    ) -> None:
+        """Test that project_dir_resolution_filesystem_mode is applied correctly."""
+        empty_args.project_dir_resolution_filesystem_mode = "disabled"
+        applicator.apply(empty_args, overrides, resolution)
+
+        assert "session" in overrides
+        assert (
+            overrides["session"].get("project_dir_resolution_filesystem_mode")
+            == "disabled"
+        )
+        cli_records = resolution.latest_by_source(ParameterSource.CLI)
+        assert "session.project_dir_resolution_filesystem_mode" in cli_records
+        assert (
+            cli_records["session.project_dir_resolution_filesystem_mode"].origin
+            == "--project-dir-resolution-filesystem-mode"
+        )
+
+    def test_apply_disable_default_openrouter_project_dir_resolution_fallback(
+        self,
+        applicator,
+        empty_args: CliArgs,
+        overrides: CliOverrides,
+        resolution: ParameterResolution,
+    ) -> None:
+        empty_args.disable_default_openrouter_project_dir_resolution_fallback = True
+        applicator.apply(empty_args, overrides, resolution)
+
+        assert "session" in overrides
+        assert (
+            overrides["session"].get(
+                "disable_default_openrouter_project_dir_resolution_fallback"
+            )
+            is True
+        )
+        cli_records = resolution.latest_by_source(ParameterSource.CLI)
+        assert (
+            "session.disable_default_openrouter_project_dir_resolution_fallback"
+            in cli_records
+        )
+        assert (
+            cli_records[
+                "session.disable_default_openrouter_project_dir_resolution_fallback"
+            ].origin
+            == "--disable-default-openrouter-project-dir-resolution-fallback"
+        )
+
+    def test_apply_quality_verifier_ttft_timeout_seconds(
         self,
         applicator,
         empty_args: CliArgs,
@@ -195,28 +196,28 @@ def test_apply_quality_verifier_ttft_timeout_seconds(
                 overrides["session"].get("quality_verifier_ttft_timeout_seconds")
                 == 11.5
             )
-            assert os.environ.get("QUALITY_VERIFIER_TTFT_TIMEOUT_SECONDS") == "11.5"
-
-    def test_apply_disable_auto_continue_removal(
-        self,
-        applicator,
-        empty_args: CliArgs,
-        overrides: CliOverrides,
-        resolution: ParameterResolution,
-    ) -> None:
-        """Test disable flag maps to session.auto_continue_removal_enabled=False."""
-        empty_args.disable_auto_continue_removal = True
-
-        applicator.apply(empty_args, overrides, resolution)
-
-        assert "session" in overrides
-        assert overrides["session"].get("auto_continue_removal_enabled") is False
-        cli_records = resolution.latest_by_source(ParameterSource.CLI)
-        assert "session.auto_continue_removal_enabled" in cli_records
-        assert (
-            cli_records["session.auto_continue_removal_enabled"].origin
-            == "--disable-auto-continue-removal"
-        )
+            assert os.environ.get("QUALITY_VERIFIER_TTFT_TIMEOUT_SECONDS") == "11.5"
+
+    def test_apply_disable_auto_continue_removal(
+        self,
+        applicator,
+        empty_args: CliArgs,
+        overrides: CliOverrides,
+        resolution: ParameterResolution,
+    ) -> None:
+        """Test disable flag maps to session.auto_continue_removal_enabled=False."""
+        empty_args.disable_auto_continue_removal = True
+
+        applicator.apply(empty_args, overrides, resolution)
+
+        assert "session" in overrides
+        assert overrides["session"].get("auto_continue_removal_enabled") is False
+        cli_records = resolution.latest_by_source(ParameterSource.CLI)
+        assert "session.auto_continue_removal_enabled" in cli_records
+        assert (
+            cli_records["session.auto_continue_removal_enabled"].origin
+            == "--disable-auto-continue-removal"
+        )
 
     def test_apply_tool_access_overrides(
         self,