You can get rid of the loader test ID by using getByText(/loading/i). And if it's a spinner instead, then you can use aria-label="loading" and getByLabel(/loading/i)
Oh, also, you can improve this as well:
let userList = await waitForElement(() => getByRole('list'))
Change that to this:
let userList = await findByRole('list')
And finally, you may like to do this for your assertion:
Jest will auto-update that code to have an inline snapshot for you and you'll get a tiny bit more confidence that it's rendering what it should (without the implementation details of how that text appears on the screen).
Inline snapshots are great! That's one new arsenal in my tooklit.
Yes that will make it more accessible as well! Thanks a lot :)
We're a place where coders share, stay up-to-date and grow their careers.
We strive for transparency and don't collect excess data.