wp-media / wp-rocket-e2e

Playwright E2E testing repo
GNU General Public License v3.0
0 stars 0 forks source link

Mobile assertion bails out early and passes after desktop assertion fails. #93

Closed jeawhanlee closed 5 months ago

jeawhanlee commented 5 months ago

Before submitting an issue please check that you’ve completed the following steps:

Describe the bug Mobile assertion steps fails early if fail message is not empty due to failed assertions step from desktop scenario.

To Reproduce Steps to reproduce the behavior:

  1. Run npm run test:lcp
  2. Have desktop assertion fail.
  3. See that mobile will always pass ( because the assertion step bails out early )

Expected behavior Mobile assertion step should not bail out early when desktop assertion fails.

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

Acceptance Criteria (for WP Media team use only) Clear instructions for developers, to be added before the grooming