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.

Sign up for GitHub

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

Jump to bottom

Document Flutter ios testing support #3066

Merged
merged 22 commits into from
Jan 27, 2025
Merged

Document Flutter ios testing support #3066

merged 22 commits into from
Jan 27, 2025

Conversation

si-net
Copy link
Collaborator

@si-net si-net commented Jan 16, 2025 •
edited by bahrimootaz
Loading

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation fix (typos, incorrect content, missing content, etc.)

@si-net si-net requested a review from a team as a code owner January 16, 2025 14:47
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

6 similar comments
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

bahrimootaz
bahrimootaz reviewed Jan 17, 2025
View reviewed changes
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md
saucectl run
```

For further configuration options and info on how to use `saucectl` visit [/docs/mobile-apps/automated-testing/espresso-xcuitest/xcuitest.md](/docs/mobile-apps/automated-testing/espresso-xcuitest/xcuitest.md)
Copy link
Contributor

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.

Copy link
Collaborator Author

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

Copy link
Collaborator Author

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.

Copy link

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 to xcTestRunFile parameter in the file and the saucectl init command.

Maybe in the part we recommend using saucectl init we could give a heads up there. Like You'll need to provide the test app or ipa file, and the xctestrun file created as described previously. saucectl init can be invoked interactive or non interactive. If using non interactive these are the minimum required parameters:

saucectl init --no-prompt -r us-west-1 xctest --xctest-run-file FlutterTest.xctestrun --app FlutterTestApp-resigned.ipa --device name=iPhone,private=true

Copy link
Collaborator Author

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. UseReal DevicenotVirtual Device``

docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

1 similar comment
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

si-net
si-net commented Jan 17, 2025
View reviewed changes
docs/dev/api/rdc.md Outdated Show resolved Hide resolved
si-net
si-net commented Jan 17, 2025
View reviewed changes
docs/dev/api/rdc.md Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

2 similar comments
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

thiagosaucelabs
thiagosaucelabs reviewed Jan 22, 2025
View reviewed changes
Copy link
Contributor

@thiagosaucelabs thiagosaucelabs left a comment

Choose a reason for hiding this comment

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

Still looking through this.

docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
docs/mobile-apps/automated-testing/flutter/flutter-integration-testing-ios.md Outdated Show resolved Hide resolved
@thiagosaucelabs thiagosaucelabs force-pushed the mb-flutter_integration_test_ios branch from 18748af to 432aabe Compare January 22, 2025 16:30
@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@github-actions GitHub Actions
Copy link

Deploy preview ready for 3066!
https://docs.dev.saucelabs.net/pr-preview/pr-3066

@bahrimootaz bahrimootaz merged commit 9088411 into main Jan 27, 2025
3 checks passed
@bahrimootaz bahrimootaz deleted the mb-flutter_integration_test_ios branch January 27, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bahrimootaz bahrimootaz bahrimootaz approved these changes

@thiagosaucelabs thiagosaucelabs Awaiting requested review from thiagosaucelabs

@rillgen-saucelabs rillgen-saucelabs Awaiting requested review from rillgen-saucelabs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@si-net @rillgen-saucelabs @thiagosaucelabs @bahrimootaz