Fix CLI and localization unit tests for colored terminal output.

Disable ANSI styling in CLI test invocations and isolate the Windows locale fallback test from registry lookups so CI passes reliably.

Author: sankalps0549

tests/api/unit_tests/test_cli_basics.py

@@ -27,7 +27,7 @@ def test_help_runs_without_synergy(self):
         with patch("moldflow_cli.context.get_synergy") as mock_ctx_synergy, patch(
             "moldflow_cli.factories.get_synergy"
         ) as mock_fact_synergy:
-            result = runner.invoke(app, ["--help"])
+            result = runner.invoke(app, ["--no-color", "--help"])
         assert result.exit_code == 0
         # Help should not need to touch Synergy/COM at all.
         mock_ctx_synergy.assert_not_called()
@@ -40,7 +40,7 @@ def test_help_runs_without_synergy(self):
     def test_help_shows_global_no_color_option(self):
         """Global help should advertise the output-styling toggle."""
         app = build_cli_app()
-        result = runner.invoke(app, ["--help"])
+        result = runner.invoke(app, ["--no-color", "--help"])
         assert result.exit_code == 0
         assert "--no-color" in result.stdout
 
@@ -50,9 +50,9 @@ def test_subcommand_help_runs_without_synergy(self):
         with patch("moldflow_cli.context.get_synergy") as mock_ctx_synergy, patch(
             "moldflow_cli.factories.get_synergy"
         ) as mock_fact_synergy:
-            list_help = runner.invoke(app, ["list", "--help"])
-            describe_help = runner.invoke(app, ["describe", "--help"])
-            invoke_help = runner.invoke(app, ["invoke", "--help"])
+            list_help = runner.invoke(app, ["--no-color", "list", "--help"])
+            describe_help = runner.invoke(app, ["--no-color", "describe", "--help"])
+            invoke_help = runner.invoke(app, ["--no-color", "invoke", "--help"])
 
         assert list_help.exit_code == 0
         assert describe_help.exit_code == 0

tests/api/unit_tests/test_cli_describe.py

@@ -823,7 +823,7 @@ def test_list_human_output_shows_explicit_message_for_empty_filtered_results():
 def test_list_with_describe_requires_structured_output():
     """Expanded list metadata should stay opt-in for JSON/YAML flows only."""
     app = build_cli_app()
-    result = runner.invoke(app, ["list", "--with-describe"])
+    result = runner.invoke(app, ["--no-color", "list", "--with-describe"])
     assert result.exit_code != 0
     combined = (result.stdout or "") + (getattr(result, "stderr", "") or "")
     assert "--with-describe requires --json or --yaml" in combined
@@ -855,7 +855,7 @@ def test_list_json_allows_zero_max_results():
 def test_list_rejects_negative_max_results():
     """List should reject negative result caps explicitly."""
     app = build_cli_app()
-    result = runner.invoke(app, ["list", "--json", "--max-results", "-1"])
+    result = runner.invoke(app, ["--no-color", "list", "--json", "--max-results", "-1"])
     assert result.exit_code != 0
     combined = (result.stdout or "") + (getattr(result, "stderr", "") or "")
     assert "--max-results must be greater than or equal to 0" in combined

tests/api/unit_tests/test_cli_invoke_execute.py

@@ -90,7 +90,7 @@ def _get_cli_test_manager(self) -> CLITestManager:
     setattr(moldflow, "CLITestManager", CLITestManager)
     setattr(moldflow.Synergy, "cli_test_manager", property(_get_cli_test_manager))
     try:
-        result = runner.invoke(app, ["invoke", "cli_test_manager"])
+        result = runner.invoke(app, ["--no-color", "invoke", "cli_test_manager"])
         assert result.exit_code != 0
         combined = (result.stdout or "") + (getattr(result, "stderr", "") or "")
         lowered = combined.lower()

tests/api/unit_tests/test_cli_invoke_parse.py

@@ -697,7 +697,7 @@ def cli_echo(self, value: str) -> str:
         with patch("moldflow_cli.context.get_synergy") as mock_ctx_synergy, patch(
             "moldflow_cli.factories.get_synergy"
         ) as mock_fact_synergy:
-            r = runner.invoke(app, ["invoke", "synergy.cli_echo", '{"value":'])
+            r = runner.invoke(app, ["--no-color", "invoke", "synergy.cli_echo", '{"value":'])
 
         assert r.exit_code != 0
         combined = (
@@ -1856,7 +1856,7 @@ def open_project(self, path: int):
 def test_invoke_help_shows_arg_syntax():
     """Invoke --help should demonstrate the step.param.attr=value routing syntax."""
     app = build_cli_app()
-    result = runner.invoke(app, ["invoke", "--help"])
+    result = runner.invoke(app, ["--no-color", "invoke", "--help"])
     help_text = result.stdout.lower()
     assert result.exit_code == 0
     assert "find_plot_by_name.plot_name" in help_text or "step.param.attr" in help_text

tests/core/test_localization.py

@@ -127,10 +127,13 @@ def test_set_language_env(self, locale):
         assert _(TEST_STRING) == TEST_TRANSLATION_DICT[locale]
         del os.environ[LOCALE_ENVIRONMENT_VARIABLE_NAME]
 
+    @pytest.mark.usefixtures("environment_locale")
     def test_set_language_windows_locale_fallback(self):
         """
         Test set_language falls back to the Windows user locale.
         """
-        with patch("moldflow.localization._get_windows_locale_name", return_value="ja-JP"):
+        with patch("moldflow.localization.winreg.OpenKey", side_effect=FileNotFoundError), patch(
+            "moldflow.localization._get_windows_locale_name", return_value="ja-JP"
+        ):
             _ = set_language(version=TEST_VERSION)
         assert _(TEST_STRING) == "テスト文字列"