test(ui): realign dashboard vitest suites with this cycle's contract changes (release base-reds r5) - #15115
Merged
diegosouzapw merged 1 commit intoSep 29, 2026
Conversation
- next-intl mocks now provide useLocale: PoolSchedulesCard (#14466) and ProxyHealthCell (#14807) call useLocale(), so pool-card, quota-share-page, quota-share-delete-error and ProxyRegistryManager-credential-autofill crashed on render. - use-stream-metrics: tps is generation throughput over (totalMs - ttftMs) since #13373 (#13130); expected value is 15 / 1.8 s. - compressionPanel: the GET fixture now reports enginesExplicit like the real API, and the preview assertion follows #14529/#14700 (header-less requests downgrade lossy rtk to session-dedup -> lite).
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clears the Vitest reds on the release PR #11442 (CI run 36562032271, job 109385627722). Test-only change: every failure came from an intentional contract change merged in this cycle, not from a production defect.
pool-card(6),quota-share-page(7),quota-share-delete-error(5)PoolSchedulesCardnow callsuseLocale(); the tests'next-intlmock only exporteduseTranslations, so every render threwNo "useLocale" export is defined on the "next-intl" mockuseLocale: () => "en"to the mocksProxyRegistryManager-credential-autofill(2)ProxyHealthCelluse-stream-metrics(2)15 / 1.8compressionPanel"derived-pipeline preview reflects the enabled engines"enginesExplicitis set. The real GET returns it (src/lib/db/compression.ts), but the fixture did not. With the flag set, rtk (lossy) is downgraded tosession-dedup → litefor a request without a header (#14529). The assertion was already stale before #14700, which I checked by running it against63e029e^enginesExplicit: true; the assertion checks forsession-dedup → liteand for no rtk or cavemanProof: all 6 files pass together (36/36), in 2 runs in a row. Before the fix, 23 tests failed on the tip
9bbf6ef.Gates:
typecheck:core,check:open-sse-typecheck,check:dashboard-typecheck(within baseline), eslint on the changed files,check-file-sizeafter the commit all pass.Not fixed here (investigated):
main(run 35650527896, 2026-09-21, exit 143 after 1.5h onomniroute-113-5). Locally the tip andorigin/mainboth ran past a 25-min timeout at the same step. This predates this cycle's merges and is a problem with the runner or the Turbopack environment, not with the diff.cancelled, not failed, because a newer push superseded run 36561991270. In the current run 36562032271 both Bun jobs (ubuntu and windows) pass.— TBD.npm cion Node 22 / npm 10.9.8 rejects the lockfile because of how npm 10 resolves nested overrides (undici@7.29.1 does not satisfy undici@6.29.0,brace-expansion@2.1.4 does not satisfy 1.1.21, missingconcat-map). It has failed on every run since 2026-09-18. The last green run was 2026-09-16. This needs a separate lockfile or workflow decision.