Skip to content

test(api): regenerate kernel_config_schema golden fixture#4002

Merged
houko merged 1 commit into
mainfrom
fix/regen-config-schema-golden
Apr 28, 2026
Merged

test(api): regenerate kernel_config_schema golden fixture#4002
houko merged 1 commit into
mainfrom
fix/regen-config-schema-golden

Conversation

@houko

@houko houko commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Recently-merged PRs added new fields to KernelConfig without regenerating the golden fixture:

The drift caused kernel_config_schema_matches_golden_fixture to fail on every open PR's CI (Test / Ubuntu, Windows, macOS), blocking ~30 in-flight fixes from merging even though their own changes were unrelated to the config schema.

Regenerated via

cargo test -p librefang-api --test config_schema_golden -- --ignored regenerate_golden --nocapture

Test plan

  • cargo test -p librefang-api --test config_schema_golden passes locally
  • CI green

Several recently-merged PRs added new fields to KernelConfig without
regenerating the golden fixture:

- app_secret_env (Messenger HMAC) from #3942
- security_token_env (Teams HMAC) from #3942
- public_base_url (mobile pairing) from #3344

The drift caused kernel_config_schema_matches_golden_fixture to fail on
every open PR's CI (Test/Ubuntu/Windows/macOS), blocking ~30 in-flight
fixes from merging even though their own changes were unrelated to the
config schema.

Regenerated via:

    cargo test -p librefang-api --test config_schema_golden \
        -- --ignored regenerate_golden --nocapture

29 lines added, no removals — purely additive new optional fields.
@houko
houko merged commit dd63dc6 into main Apr 28, 2026
14 checks passed
@houko
houko deleted the fix/regen-config-schema-golden branch April 28, 2026 15:03
@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/S 10-49 lines changed labels Apr 28, 2026
@houko houko mentioned this pull request May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-rust-required This task does not require Rust knowledge size/S 10-49 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant