Skip to content

test: stabilize headless browser CI - #3484

Merged
ibgreen merged 2 commits into
masterfrom
codex/mvt-polygon-test-stabilization
Jul 31, 2026
Merged

test: stabilize headless browser CI#3484
ibgreen merged 2 commits into
masterfrom
codex/mvt-polygon-test-stabilization

Conversation

@ibgreen

@ibgreen ibgreen commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Normalize projected GeoJSON polygon coordinates before comparing headless-browser output.
  • Launch Chromium with --disable-dev-shm-usage so the full browser suite does not crash when CI shared memory is constrained.
  • Preserve the existing assertions and test coverage.

Root cause

The headless workflow exposed two independent stability problems while testing the unrelated Thrift dependency update in #3463:

  1. Chromium projection results can differ at insignificant floating-point precision, making the MVT polygon deep-equality assertion brittle.
  2. After that assertion was stabilized, Chromium exited partway through the full suite under CI shared-memory pressure, leaving Vitest with only a browser-disconnect error.

Validation

@ibgreen ibgreen changed the title test(mvt): stabilize polygon comparison test: stabilize headless browser CI Jul 31, 2026
@ibgreen
ibgreen marked this pull request as ready for review July 31, 2026 13:43
@ibgreen
ibgreen merged commit 5578963 into master Jul 31, 2026
9 checks passed
@ibgreen
ibgreen deleted the codex/mvt-polygon-test-stabilization branch July 31, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant