You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Remove or harden the test-only deploy ID inventory #1146
The test-only DEPLOY_VALIDATED_INTEGRATION_IDS constant in crates/trusted-server-core/src/config.rs duplicates the registered integration builder IDs. Its test currently checks only that no registered builder is missing from the constant; a stale extra ID can remain in the constant without failing. This was identified during the PR #1049 documentation refresh, which added a separate set-equality check, but the redundant inventory and one-directional assertion remain in the current checkout.
Proposed fix
Remove the redundant constant and derive the coverage assertion from a single source of truth, or keep it and make the test compare both sets. Preserve the intended guarantee that every registered integration has deploy validation.
Done when
A stale extra ID in the test inventory fails the relevant test, or no duplicated inventory remains.
A missing deploy-validated registered builder also fails.
Activity