Make sure test results are archived when test suites fail

[crazytonyli]

Description

The buildkite script stops running after test suite fails, but we still need the test result, well especially when any tests failed. This PR took the approach used in WordPress-iOS repo

Testing instructions

Make sure test results are archived when test failed. I'll update this description with screenshots of buildkite steps later.

Screenshots

N/A

---

• I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

[wpmobilebot]

You can test the changes from this Pull Request by:

• Clicking here or scanning the QR code below to access App Center
• Then installing the build number pr7647-0dcfec0 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

[AliSoftware]

What do you think of also adding the echo "^^^ +++" on non-zero exit code (like is done here) so that Buildkite will have the "Testing" section expanded and open by default — instead of the "Zipping test results" one — when the test failed, to avoid confusion for anyone checking the logs and seeing the failure looking like it was on the zipping instead of the testing step?

See Buildkite docs here

[crazytonyli]

Great idea. I've copied this feature over in 0dcfec0

[peril-woocommerce]

	Warnings
⚠️	This PR is assigned to a milestone which is closing in less than 4 days Please, make sure to get it merged by then or assign it to a later expiring milestone

Generated by 🚫 dangerJS