Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: migrate from karma (deprecated) to web-test-runner #210

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

larabr
Copy link
Collaborator

@larabr larabr commented Feb 28, 2025

Karma has been deprecated for some time, and the wtr playwright integration makes it easier to run e.g. specific browser versions.
Since wtr does not integrate with webpack, we also switch to esbuild and rollup for the test bundling.

Also, fix #135 .

TODO:

  • Might switch to a pre-building step using webpack over rollup to align with the monorepo

BN.js uses a legacy export mode that needs special handling.
Also, using the noble-curves API improves readability, as the scalar clamping
scalar clamping needs to be explicitly disabled for the proxy transformation.
@larabr larabr force-pushed the tests-drop-bnjs branch 2 times, most recently from ee54eae to fa220c6 Compare February 28, 2025 13:04
Also drop webpack, as wtr only integrates with rollup and esbuild.
Due to X25519 issues specific to the WebKit build.
To catch issues earlier.

Playwright's Webkit and Chromium builds do not include beta channels
but already point to pre-release branches (see e.g. https://playwright.dev/docs/browsers#chromium).
Copy link
Member

@twiss twiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In case you're planning to rebase/merge rather than squash: there's a typo/reduplicated "scalar clamping" in the first commit message. Other than that all LGTM.

@larabr larabr marked this pull request as draft March 5, 2025 10:54
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.

Tests: incorrect line references reported on error
2 participants