react-select-event icon indicating copy to clipboard operation
react-select-event copied to clipboard

Migrate to using @testing-library/user-event

Open artursvonda opened this issue 1 year ago • 3 comments

The userEvent better represents actions taken by user/browser and actually allows us to get rid of some hacks.

Also, testing now with React v18.

And removed redundant act calls as well since, both, fireEvent and userEvent wrap calls in act implicitly (when required).

artursvonda avatar May 17 '23 08:05 artursvonda

Love to see this change as it also solves issue with act() wrapping fireEvent() in rse.clear() method that throws RTL warnings - your change seams to be working fine w/o that issue

aroslaniec avatar Oct 25 '23 11:10 aroslaniec

I've tested these changes locally on a large codebase I'm migrating to React 18 and can confirm it's fixed bugs which arose from the migration

kirkobyte avatar Dec 09 '23 15:12 kirkobyte

@romgain do you think this can be merged and published to npm?

chiptus avatar Feb 06 '24 18:02 chiptus