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

fix(type): wrap type in the asyncWrapper from DOM Testing Library #303

Merged
merged 2 commits into from Jun 2, 2020

Conversation

kentcdodds
Copy link
Member

@kentcdodds kentcdodds commented Jun 2, 2020

What: wrap type in the asyncWrapper from DOM Testing Library

Why:

This fixes issues with act warnings when using type with React Testing Library (I expect the same would be necessary for angular change detection).

How: Upgrade to the latest version of DOM Testing Library (it is now required) so we can get the getConfig export to use the asyncWrapper.

Checklist:

  • Documentation N/A
  • Tests
  • Typings N/A
  • Ready to be merged

BREAKING CHANGE: The DOM Testing Library version 7.9.0 or greater is required

This fixes issues with act warnings when using `type` with React Testing
Library.
BREAKING CHANGE: The latest version of DOM Testing Library is required
@codecov
Copy link

codecov bot commented Jun 2, 2020

Codecov Report

Merging #303 into next will increase coverage by 0.01%.
The diff coverage is 92.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next     #303      +/-   ##
==========================================
+ Coverage   98.90%   98.91%   +0.01%     
==========================================
  Files           1        1              
  Lines         182      185       +3     
  Branches       56       55       -1     
==========================================
+ Hits          180      183       +3     
  Misses          2        2              
Impacted Files Coverage Δ
src/index.js 98.91% <92.00%> (+0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb33bdb...2e39cb7. Read the comment docs.

@kentcdodds kentcdodds changed the base branch from master to next June 2, 2020 18:41
@kentcdodds kentcdodds changed the base branch from next to master June 2, 2020 18:41
@kentcdodds kentcdodds merged commit 9fe76b4 into master Jun 2, 2020
@kentcdodds kentcdodds deleted the pr/fix-async-type branch June 2, 2020 18:41
@kentcdodds
Copy link
Member Author

🎉 This PR is included in version 11.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Meemaw added a commit to Meemaw/user-event that referenced this pull request Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant