Wrap everything in eventWrapper

[kentcdodds]

Normally, we call fireEvent to trigger events, and that should already be wrapped in the eventWrapper. If the user's using @testing-library/react, then this eventWrapper will be act.

However, we don't always use fireEvent and if this were to trigger a state update:

user-event/src/focus.js

Line 10 in 8b15930

element.focus()

Then we're toast.

So I see two solutions to this:

1. Wrap every function in eventWrapper
2. Find everywhere we're triggering events and wrap them individually

I think if option 1 will work (and I think it will) then that's the easiest and will allow us to avoid issues of people forgetting to add it in the future.

[kentcdodds]

I'm almost done with this. Going with option 1 because I verified it'll work :)

[kentcdodds]

🎉 This issue has been resolved in version 12.0.9 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀