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-1770 test_repetition_mode is added for Xcode 13 stack #662

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

buranmert
Copy link
Contributor

@buranmert buranmert commented Nov 15, 2021

What and why?

Bitrise's xcode-test step deprecates should_retry_test_on_fail input in favor of test_repetition_mode: 'retry_on_failure

How?

Our main bitrise.yml uses the new input parameter.
nightly-unit-tests tool sets the input parameter accordingly.

TODO

  • Once the PR is approved, I will update our stack in Bitrise and re-run CI

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

@buranmert buranmert self-assigned this Nov 15, 2021
@buranmert buranmert force-pushed the buranmert/RUMM-1770-update-bitrise-stack branch from 3746b9f to 3c129dd Compare November 15, 2021 17:04
@buranmert buranmert marked this pull request as ready for review November 15, 2021 17:07
@buranmert buranmert requested a review from a team as a code owner November 15, 2021 17:07
@@ -91,6 +91,11 @@ def __rendered_template(self):

template = template.replace('## <MACOS VERSION> ##', self.host_os_version)

if self.host_os_version.startswith('11'): # macOS 11.X Big Sur
template = template.replace('## <RETRY_ON_FAIL> ##', "test_repetition_mode: 'retry_on_failure'")
Copy link
Member

Choose a reason for hiding this comment

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

Don't we need to specify maximum_test_repetitions as well? Or does it have a default value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, default value is 3, reasonable i guess.?
i didn't want to replace multiple lines in yaml in order not to deal with indentation issues.

Copy link
Member

Choose a reason for hiding this comment

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

Totally 👍 for keeping it simple like that.

Copy link
Member

@ncreated ncreated left a comment

Choose a reason for hiding this comment

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

👌

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.

LGTM

@buranmert buranmert merged commit 90a7ab3 into master Nov 17, 2021
@buranmert buranmert deleted the buranmert/RUMM-1770-update-bitrise-stack branch November 17, 2021 10:25
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.

3 participants