Improve WooCommerce Test Reliability

[n7studios]

Summary

Improves reliability of WooCommerce tests by:

• waiting for the success notice to display before interacting with it,
• removing the "Your store does not appear to be using a secure connection" notice, which shifts the DOM, resulting in the mouseOver event hovering over the wrong row in a table.

Testing

Existing tests pass.

Checklist

• I have written a test and included it in this PR
• I have run all tests and they pass
• The code passes when running the PHP CodeSniffer
• Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
• Security and Sanitization requirements have been followed
• I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

[github-actions]

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

[noelherrick]

This is a bit problematic as it could hide a regression. What are your thoughts on wrapping this in a retry instead? It looks like Codeception has some built in "retry" logic (at least 4.x does).

[n7studios]

The retry logic is still flaky at times, which is why I went for this method. I don't think it'll hide a regression - it just ensures that the row actions are displayed automatically, rather than being displayed on mouse over.

We're not testing the row actions themselves - they're built in to WordPress. This is so we can reliably get WordPress' Quick Edit form to display. What's being tested is that the Quick Edit form's values are saved after changing, which follows this line.