Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUMM-1625 Do not run nightly tests below iOS 11.0.0 #606

Conversation

ncreated
Copy link
Member

What and why?

🐞 This PR fixes an issue in nightly tests automation, where sometimes it was trying to run unit tests on iOS 10.x.

How?

Our nightly tests automation looks at all installed and available for installation Simulators in host environment (CI machine). Then it picks few Simulators (randomly) and runs tests (if Simulator is pre-installed) or installs the Simulator and then runs tests. Because iOS 10.x simulator can be installed on macOS Catalina, it was sometimes picked by our automation.

I added MIN_SUPPORTED_IOS_VERSION variable to filter out unsupported versions from test plan.

Also, I added dump_environment() function to print more information about the Simulators, Runtimes and Devices when automation fails. This should be handy for debugging issues like iOS 15.0 Simulator not being visible by xcode-install.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference

This is to prevent CI failures, when too old simulators can be installed on the host (CI machine).
@ncreated ncreated requested a review from a team as a code owner September 23, 2021 11:28
@ncreated ncreated self-assigned this Sep 23, 2021
Copy link
Member

@xgouchet xgouchet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@maxep maxep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@ncreated ncreated merged commit eaed1cf into master Sep 24, 2021
@ncreated ncreated deleted the ncreated/RUMM-1625-do-not-run-nightly-tests-below-supported-OS-versions branch October 25, 2021 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants