Fix CI test failures by updating assertions to use result.output instead of result.stderr

- Updated failing tests to check result.output instead of result.stderr since Click's test runner combines stdout/stderr unless specifically configured
- Added environment variables to disable color output in both unit and integration test steps
- This resolves the "stderr not separately captured" errors in CI

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: ritwik-g

.github/workflows/ci.yml

@@ -45,10 +45,16 @@ jobs:
     - name: Run unit tests
       run: |
         uv run pytest tests/ -v --cov=helm_values_manager --cov-report=xml --cov-report=term
+      env:
+        FORCE_COLOR: "0"
+        NO_COLOR: "1"
 
     - name: Run integration tests
       run: |
         uv run pytest tests/test_integration.py -v
+      env:
+        FORCE_COLOR: "0"
+        NO_COLOR: "1"
 
     - name: Upload coverage to Codecov
       if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.12'

tests/test_generator.py

@@ -312,8 +312,8 @@ def test_generate_command_validation_fails(tmp_path):
         result = runner.invoke(app, ["generate", "--env", "dev"])
 
         assert result.exit_code == 1
-        assert "Validation failed" in result.stderr
-        assert "Missing required value: required-field" in result.stderr
+        assert "Validation failed" in result.output
+        assert "Missing required value: required-field" in result.output
 
 
 def test_generate_command_missing_env_var(tmp_path):
@@ -345,7 +345,7 @@ def test_generate_command_missing_env_var(tmp_path):
         result = runner.invoke(app, ["generate", "--env", "prod"])
 
         assert result.exit_code == 1
-        assert "Environment variable 'MISSING_VAR' not found" in result.stderr
+        assert "Environment variable 'MISSING_VAR' not found" in result.output
 
 
 def test_generate_command_complex_example(tmp_path, monkeypatch):

tests/test_integration.py

@@ -348,7 +348,7 @@ def test_error_recovery_workflow(tmp_path):
         # 6. Try to generate with validation errors
         result = runner.invoke(app, ["generate", "--env", "test"])
         assert result.exit_code == 1
-        assert "Validation failed" in result.stderr
+        assert "Validation failed" in result.output
 
 
 def test_schema_evolution_workflow(tmp_path):

tests/test_validate.py

@@ -15,7 +15,7 @@ def test_validate_missing_schema(tmp_path):
         app, ["validate", "--env", "dev", "--schema", str(tmp_path / "missing.json")]
     )
     assert result.exit_code == 1
-    assert "Schema file not found" in result.stderr
+    assert "Schema file not found" in result.output
 
 
 def test_validate_invalid_schema_json(tmp_path):
@@ -25,7 +25,7 @@ def test_validate_invalid_schema_json(tmp_path):
 
     result = runner.invoke(app, ["validate", "--env", "dev", "--schema", str(schema_file)])
     assert result.exit_code == 1
-    assert "Invalid JSON in schema file" in result.stderr
+    assert "Invalid JSON in schema file" in result.output
 
 
 def test_validate_valid_schema_only(tmp_path):
@@ -77,7 +77,7 @@ def test_validate_duplicate_keys(tmp_path):
 
     result = runner.invoke(app, ["validate", "--env", "dev", "--schema", str(schema_file)])
     assert result.exit_code == 1
-    assert "Duplicate key: database" in result.stderr
+    assert "Duplicate key: database" in result.output
 
 
 def test_validate_duplicate_paths(tmp_path):
@@ -106,7 +106,7 @@ def test_validate_duplicate_paths(tmp_path):
 
     result = runner.invoke(app, ["validate", "--env", "dev", "--schema", str(schema_file)])
     assert result.exit_code == 1
-    assert "Duplicate path: database.host" in result.stderr
+    assert "Duplicate path: database.host" in result.output
 
 
 def test_validate_invalid_type(tmp_path):
@@ -128,7 +128,7 @@ def test_validate_invalid_type(tmp_path):
 
     result = runner.invoke(app, ["validate", "--env", "dev", "--schema", str(schema_file)])
     assert result.exit_code == 1
-    assert "Invalid schema" in result.stderr or "Invalid type" in result.stderr
+    assert "Invalid schema" in result.output or "Invalid type" in result.output
 
 
 def test_validate_values_type_mismatch(tmp_path):
@@ -161,7 +161,7 @@ def test_validate_values_type_mismatch(tmp_path):
         ["validate", "--env", "dev", "--schema", str(schema_file), "--values", str(values_file)],
     )
     assert result.exit_code == 1
-    assert "Type mismatch for port" in result.stderr
+    assert "Type mismatch for port" in result.output
 
 
 def test_validate_missing_required_value(tmp_path):
@@ -192,7 +192,7 @@ def test_validate_missing_required_value(tmp_path):
         ["validate", "--env", "prod", "--schema", str(schema_file), "--values", str(values_file)],
     )
     assert result.exit_code == 1
-    assert "Missing required value: database-host" in result.stderr
+    assert "Missing required value: database-host" in result.output
 
 
 def test_validate_secret_structure(tmp_path):
@@ -226,7 +226,7 @@ def test_validate_secret_structure(tmp_path):
         ["validate", "--env", "dev", "--schema", str(schema_file), "--values", str(values_file)],
     )
     assert result.exit_code == 1
-    assert "Invalid secret structure for api-key" in result.stderr
+    assert "Invalid secret structure for api-key" in result.output
 
 
 def test_validate_valid_secret(tmp_path, monkeypatch):
@@ -293,7 +293,7 @@ def test_validate_unknown_key_in_values(tmp_path):
         ["validate", "--env", "dev", "--schema", str(schema_file), "--values", str(values_file)],
     )
     assert result.exit_code == 1
-    assert "Unknown key: unknown-key" in result.stderr
+    assert "Unknown key: unknown-key" in result.output
 
 
 def test_validate_shows_environment_name_in_errors(tmp_path):
@@ -333,5 +333,5 @@ def test_validate_shows_environment_name_in_errors(tmp_path):
     )
 
     assert result.exit_code == 1
-    assert "Missing required value: database-host" in result.stderr
-    assert "staging" in result.stderr
+    assert "Missing required value: database-host" in result.output
+    assert "staging" in result.output