fix: add no-await-sync-events exception for userEvent.keyboard

[ph-fritsche]

Added exception for the new API proposed in testing-library/user-event#581

Note that with the change userEvent.type (and userEvent.keyboard) will actually return no Promise if the delay is not >0 so that the typing and the actual return value will match.

Further it might be worth discussing if this rule should be disabled for .ts and .tsx files as ts(80007) and @typescript-eslint/await-thenable already warn here and curating a manual list has no benefits when there already are warnings based on the type.

[Belco90]

Hey @ph-fritsche, thanks for your contribution! We need to wait until testing-library/user-event#581 gets merged.

Meanwhile, could you add some additional tests to check userEvent.keyboard is reported or not as intended?

About disabling the rule in TS environments: the plugin won't detect that for now, so it's up to the user to decide if they want to enable or disable the rule.

[gndelia]

should we update a test to include the keyboard method ? 🤔

[Belco90]

I just realized this rule is still pending to be refactored for v4. @ph-fritsche I can include userEvent.keyboard there when I refactor it. What do you think?

[ph-fritsche]

@Belco90 I've added the changes necessary. Should be ready to be merged now. 😃

[Belco90]

LGTM, thanks for your PR and nice work adding this new feature to userEvent 🚀

[github-actions]

🎉 This PR is included in version 3.10.2 🎉

The release is available on:

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

Your semantic-release bot 📦🚀

[Belco90]

@ph-fritsche sorry mate, I forgot to add you as a contributor!

@all-contributors please add @ph-fritsche for code

[allcontributors]

@Belco90

I could not determine your intention.

Basic usage: @all-contributors please add @Someone for code, doc and infra

For other usages see the documentation

[Belco90]

@all-contributors please add @ph-fritsche for code

[allcontributors]

@Belco90

I've put up a pull request to add @ph-fritsche! 🎉