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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(web): conversion of Web gesture-engine browser-tests for @web/test-runner use 馃弮 #11454

Merged
merged 6 commits into from
May 22, 2024

Conversation

jahorton
Copy link
Contributor

This one came with a couple of extra wrinkles:

  1. Webkit provides Touch and TouchEvent constructors, but doesn't like us actually using them... at least not in the engine-only form used by playwright. So, I had to do a bit of extra handling for touch emulation.
  2. Rather than make things complex, I simply copied the "host fixture" page as a direct, test-hosting fixture .spec.html. We do lose the fixture auto-update, but the .spec.html page is pretty similar to it.
    • I'm not too worried about maintenance being high-cost, though I am disappointed that it is higher cost than before.

@keymanapp-test-bot skip

@jahorton jahorton requested a review from mcdurdin as a code owner May 15, 2024 08:06
Copy link
Contributor Author

Choose a reason for hiding this comment

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

A direct copy of the host-fixture's CSS, now that we're using an edited copy of the HTML page as the test-hosting fixture page for browser-oriented auto-testing.

@github-actions github-actions bot added web/ and removed web/ labels May 15, 2024
Base automatically changed from chore/web/kbd-proc-test-migration to master May 22, 2024 14:55
@jahorton jahorton merged commit 5640cfa into master May 22, 2024
19 checks passed
@jahorton jahorton deleted the chore/web/gesture-test-migration branch May 22, 2024 14:55
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 18.0.41-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants