Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@michaelw85 I think the issue is that ... IF NOT EXISTS isn't supported on that older MySQL version.

@michaelw85 I think the issue is that ... IF NOT EXISTS isn't supported on that older MySQL version.

Yes I finally figured that out 😓 I'm looking at other options. There's no option to skip this MySQL version yet looking at the code.

We could set up this second user more centrally in our wp-cli-tests package: https://github.com/wp-cli/wp-cli-tests/blob/main/bin/install-package-tests

We could set up this second user more centrally in our wp-cli-tests package: https://github.com/wp-cli/wp-cli-tests/blob/main/bin/install-package-tests

That might be an easy fix, but I don't think a lot of tests would be done with this user apart from this specific scenario.
Is there a way to conditionally skip this test for 5.6?

@swissspidy Added wp-cli/wp-cli-tests#266 to make it possible to set minimum mysql requirements

@michaelw85 I just tagged v5.0.1 of wp-cli/wp-cli-tests 👍

@swissspidy Tests are finally green. First I added the 5.7 mysql requirement tag, but I also have to add the plain @require-mysql which I didn't expect. I assume there's a different piece of code that determines the include part of the behat tags I haven't touched. I don't have time right now but I think this should be updated to match other tag behavior.