Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Document Flutter ios testing support #3066
base: main
Are you sure you want to change the base?
Document Flutter ios testing support #3066
Changes from all commits
c8f208d
7760eb2
a8f8bb3
ea004fe
cdb0aea
9f8b0dd
ac5564b
fed3285
bde10f1
7174e7d
c193eb5
5683c91
2cc7abf
ec1f2af
b6ea82d
592854f
8ddd402
97731ab
432aabe
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Check warning on line 38 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L38
Raw output
Check warning on line 42 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L42
Raw output
Check warning on line 45 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L45
Raw output
Check warning on line 45 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L45
Raw output
Check warning on line 65 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L65
Raw output
Check warning on line 69 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L69
Raw output
Check warning on line 84 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L84
Raw output
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should change the saucectl docs to reflect xctests too, this is something that can make customer confused and open SRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we need to find a smart way to do this, I don't want to copy the entire xcuitest config page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thinking about this more, I believe we should not update the doc in the
xcuitest config page
until we at least support format version 2 of the xctestrun file.Otherwise people will assume that we have generic XCTest support and wonder when their XCTests don't work on Sauce.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah the confusion would come from the replacement from
appTest
toxcTestRunFile
parameter in the file and thesaucectl init
command.Maybe in the part we recommend using
saucectl init
we could give a heads up there. LikeYou'll need to provide the test app or ipa file, and the xctestrun file created as described previously.
saucectl initcan be invoked interactive or non interactive. If using non interactive these are the minimum required parameters:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mention that in this comment:
follow the steps to configure your xctest, with your
.app/
.ipafile and the
.xctestruntest config. Use
Real Devicenot
Virtual Device``Check warning on line 86 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L86
Raw output
Check warning on line 90 in docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
GitHub Actions / vale
[vale] docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md#L90
Raw output