safe-code-change

Safe Code Change

After a code change is made, find and fix affected tests before running validation.

Rules

• The code change is already done. Do not modify production code.
• Only update tests to match the new behavior, not the other way around.
• Do not reformat or lint-fix during test updates. Save that for validation.
• If a test change is ambiguous (unclear what the new expected behavior is), ask the user.

Step 1: Identify What Changed

git diff --name-only
git diff --stat

List the modified production files (ignore test files, configs, docs).

Step 2: Find Affected Tests

Search for imports and uses of changed functions/types across all test files:

# Find test files that import the changed package
rg "github.com/openshift/lightspeed-agentic-operator/<changed_package>" --type go -g '*_test.go'

# Find direct function/type references
rg "<ChangedFunctionOrType>" --type go -g '*_test.go'

For controller or CLI changes, also check:

• Shared test helpers or fixtures under the same package
• examples/ YAMLs only if behavior affects documented shapes (usually not test code)

Step 3: Analyze Impact on Tests

For each affected test file, check whether the change breaks existing tests:

1. Signature changes — function renamed, parameters added/removed/reordered.
2. Behavior changes — return value, error messages, side effects differ.
3. Removed code — tests for deleted functions/types need removal.
4. New code — consider whether new tests are needed (ask user if unclear).
5. Interface changes — mock implementations need updating.

Step 4: Update Tests

Apply minimal fixes to each affected test:

For Ginkgo Tests (if present):

• Update Expect() assertions to match new return values
• Update mock return values in test fixtures
• Add/remove parameters in function calls
• Update error message checks
• Adjust Eventually() timeouts if reconciliation logic changed

For Standard Go Tests (typical in this repo):