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.

Sign up for GitHub

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

Jump to bottom

Frontend Tests: Standardize userEvent usage to conform with upstream recomendations #5872

Merged
merged 3 commits into from
Jun 22, 2023
Merged

Frontend Tests: Standardize userEvent usage to conform with upstream recomendations #5872

merged 3 commits into from
Jun 22, 2023

Commits on Jun 8, 2023

  1. Use userEvent.setup and move existing cases to recommended location 

    For details, see https://testing-library.com/docs/user-event/setup .
Existing uses call `userEvent.setup()` internally in order to ease
migration to v14. The compatibility call will most likely be removed in
a future version.

This also moves most tests to userEvent from fireEvent.

Signed-off-by: Taylor Smock <tsmock@meta.com>
    @tsmock
    tsmock committed Jun 8, 2023
    Configuration menu
    Copy the full SHA
    332e425 View commit details
    Browse the repository at this point in the history

  2. Fix clipboard tests 

    Signed-off-by: Taylor Smock <tsmock@meta.com>
    @tsmock
    tsmock committed Jun 8, 2023
    Configuration menu
    Copy the full SHA
    afbe5fb View commit details
    Browse the repository at this point in the history

  3. Add missing "await" and standardize usage of userEvent

    @HelNershingThapa @tsmock
    HelNershingThapa authored and tsmock committed Jun 8, 2023
    Configuration menu
    Copy the full SHA
    ac644e0 View commit details
    Browse the repository at this point in the history