chore(e2e): refresh Cypress fixtures so MiniMax M3 appears in dropdown#458
Conversation
The E2E_FIXTURES=1 cypress build serves availability.json (and the rest of packages/app/cypress/fixtures/api/) instead of querying the DB. After #456 added MiniMax M3 to the dashboard, the model-architecture.cy.ts spec gained a new MiniMax M3 describe block that tries to click `MiniMax M3` in the model dropdown — but the fixture availability.json predated M3, so GlobalFilterContext.availableModels filtered MiniMax M3 out of the dropdown and the `cy.contains('MiniMax M3').click()` in the `before all` hook timed out. The leftover open Radix popper then took out the gpt-oss and DeepSeek V4 Pro suites with `[role="combobox"]` not visible. Refreshed via `pnpm --filter app capture:fixtures` (captures from https://inferencex.semianalysis.com). Verified locally: `E2E_FIXTURES=1 pnpm start` + `cypress run --spec model-architecture.cy.ts` → 45/45 passing. Co-authored-by: Alec Ibarra <adibarra@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This comment has been minimized.
This comment has been minimized.
|
@claude keep fixing the cypress tests until they fully pass without issues |
|
Claude finished @adibarra's task in 6m 58s —— View job Fixed failing Cypress tests
Root causePR #435 changed the default inference model to The sibling test Changes (commit
|
After PR #435 made DeepSeek-V4-Pro the default model, the refreshed availability fixture (which includes only FP4 for dsv4) caused `effectivePrecisions` to intersect the URL-seeded ['fp8'] with the available set and fall back to FP4. The precision-multiselect then rendered FP4 instead of the expected FP8. Add `g_model=DeepSeek-R1-0528` to the two affected specs so the URL selection is meaningful — matching the workaround already used by the "multiple URL params" test in url-params.cy.ts. Co-authored-by: Alec Ibarra <adibarra@users.noreply.github.com>
|
@claude linter/formatter failing fix |
|
Claude finished @adibarra's task in 2m 5s —— View job Linter/formatter fixed
Result
Committed as |
Co-authored-by: Alec Ibarra <adibarra@users.noreply.github.com>
Note
Low Risk
Test-only URL and comment changes; no production code paths affected.
Overview
After Cypress fixtures made the default inference model (DeepSeek-V4-Pro) FP4-only, two e2e visits that relied on the default model no longer exercised the precision behavior under test.
line-labels.cy.tsnow opens/inferencewithg_model=DeepSeek-R1-0528alongsidei_linelabel=1andi_prec=fp4,fp8, so both precisions surviveeffectivePrecisionsand the spec can assert dual-precision line labels.url-params.cy.tsuses the same model fori_prec=fp8, so the precision multiselect shows FP8 instead of falling back to FP4. Comments document why the default model is insufficient.Reviewed by Cursor Bugbot for commit ca7c284. Bugbot is set up for automated code reviews on this repo. Configure here.