-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[platform_view]use cmd to uninstall the app #111139
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
[platform_view]use cmd to uninstall the app #111139
Conversation
868c676
to
ff1e10a
Compare
ff1e10a
to
85b8f0a
Compare
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.
LGTM, thanks for circling back on this. If it gets flaky again I give up, let's go back to the hack and move on.
finger crossed: https://chromium-swarm.appspot.com/task?id=5d32c1d1a13cbe10 |
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.
LGTM.
@@ -15,6 +15,17 @@ Future<void> main() async { | |||
final String projectDirectory = '${flutterDirectory.path}/dev/integration_tests/ios_platform_view_tests'; | |||
|
|||
await inDirectory(projectDirectory, () async { | |||
|
|||
// To address "Failed to terminate" failure. |
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.
nits: link the issue.
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.
oops. it got landed before my force push. will just leave it as it is.
We can simply uninstall using the flutter command, as pointed out by @jmagman
led: https://chromium-swarm.appspot.com/task?id=5d32c1d1a13cbe10
Fixes #109697
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.