Evaluate current usage of `cy.wait()` in our E2E test suite

[lukeheath]

Goal

Determine if our current usage of cy.wait() is necessary, and if not, what is a better alternative?

The concern we have is that cy.wait() is arbitrary and can cause flakiness with our E2E test suite. We previously worked to remove all cy.wait() instances, but recently they have started showing up in our tests again. Is this acceptable?

We should determine in what cases cy.wait() is acceptable for use, and when it is not.