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

test: identify browser/jsdom differences #1091

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

test: identify browser/jsdom differences #1091

wants to merge 12 commits into from

Conversation

ph-fritsche
Copy link
Member

What:

Identify the differences between Chrome and Jsdom environment.

Why:

Determine if failing tests in Chrome are the result of wrong assumptions in our tests or wrong implementations in our source.

Checklist:

  • Ready to be merged

@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 27, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 2463be9:

Sandbox Source
userEvent-dom Configuration
userEvent-react Configuration

@codecov
Copy link

codecov bot commented Dec 27, 2022

Codecov Report

Merging #1091 (2463be9) into main (7a305de) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1091   +/-   ##
=======================================
  Coverage   99.46%   99.47%           
=======================================
  Files          92       92           
  Lines        2070     2085   +15     
  Branches      714      706    -8     
=======================================
+ Hits         2059     2074   +15     
  Misses         11       11           
Impacted Files Coverage Δ
src/system/keyboard.ts 100.00% <100.00%> (ø)
src/system/pointer/index.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ph-fritsche ph-fritsche mentioned this pull request Mar 29, 2023
5 tasks
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.

None yet

1 participant