-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[google_maps_flutter] Disable failing iOS tests #5629
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
[google_maps_flutter] Disable failing iOS tests #5629
Conversation
Disables the tests that have started failing for unknown OOB reasons, in order to fix the tree. flutter/flutter#139825
ditman
left a comment
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! but maybe only skip if isIOS on the core plugin?
| }); | ||
| }, | ||
| // TODO(stuartmorgan): Re-enable; see https://github.com/flutter/flutter/issues/139825 | ||
| skip: true); |
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.
| skip: true); | |
| skip: isIOS); |
| }); | ||
| }, | ||
| // TODO(stuartmorgan): Re-enable; see https://github.com/flutter/flutter/issues/139825 | ||
| skip: true); |
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.
| skip: true); | |
| skip: isIOS); |
| }); | ||
| }, | ||
| // TODO(stuartmorgan): Re-enable; see https://github.com/flutter/flutter/issues/139825 | ||
| skip: true); |
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.
| skip: true); | |
| skip: isIOS); |
|
Doh, I meant to make those iOS when switching plugins and then completely forgot. Thanks! |
flutter/packages@6cd0657...cb6dbcd 2023-12-09 [email protected] [multicast_dns] Adjust README (flutter/packages#5529) 2023-12-09 [email protected] [tool] Use ^ for Dart SDK (flutter/packages#5623) 2023-12-09 [email protected] [google_sign_in_web] Migrate to pkg:web. (flutter/packages#5612) 2023-12-08 [email protected] [google_maps_flutter] Disable failing iOS tests (flutter/packages#5629) 2023-12-08 [email protected] Add `parse` constructors for the `BenchmarkResults` and `BenchmarkScore` classes (flutter/packages#5614) 2023-12-08 [email protected] [path_provider_android] Disable `getExternalStorageDirectories (type: ...)` test (flutter/packages#5619) 2023-12-08 [email protected] [ci] Fix indentation in labeler.yml (flutter/packages#5625) 2023-12-08 [email protected] Update labeler to 5.0.0, fix yml format (flutter/packages#5580) 2023-12-08 [email protected] [url_launcher] Adds documentation that a launch needs to be triggered by a user action (flutter/packages#5143) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Disables the tests that have started failing for unknown OOB reasons, in order to fix the tree.
flutter/flutter#139825