-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Reland "[google_maps_flutter] ios: re-enable test with popup #5312" #6783
Reland "[google_maps_flutter] ios: re-enable test with popup #5312" #6783
Conversation
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 once tests are happy.
[button tap]; | ||
return; | ||
} | ||
XCUICoordinate *coordinate = [button coordinateWithNormalizedOffset:CGVectorMake(0, 0)]; |
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.
Hopefully this doesn't cause flakiness.
auto label is removed for flutter/plugins, pr: 6783, due to - The status or check suite repo_checks has failed. Please fix the issues identified (or deflake) before re-applying this label. |
8a22895
to
0ea7e3f
Compare
No versioning update: PRs that only affect tests. |
@@ -2,9 +2,9 @@ | |||
// Use of this source code is governed by a BSD-style license that can be | |||
// found in the LICENSE file. | |||
|
|||
@import CoreLocation; |
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 don't need the entire GoogleMaps sdk to run the tests. Only CoreLocation is necessary.
* 13818023c [camera] Attempt to fix flaky new Android test (flutter/plugins#6831) * da4321d01 [google_maps_flutter] Modified `README.md` to fix minor syntax issues (flutter/plugins#6631) * e8c9731f1 Roll Flutter from eefbe85 to bd0791b (25 revisions) (flutter/plugins#6832) * 2eb616545 Reland "[google_maps_flutter] ios: re-enable test with popup #5312" (flutter/plugins#6783) * 738bd91d8 Update FlutterFire link (flutter/plugins#6835) * ec2041f82 Roll Flutter from bd0791b to 15af817 (27 revisions) (flutter/plugins#6837)
…#116996) * 13818023c [camera] Attempt to fix flaky new Android test (flutter/plugins#6831) * da4321d01 [google_maps_flutter] Modified `README.md` to fix minor syntax issues (flutter/plugins#6631) * e8c9731f1 Roll Flutter from eefbe85 to bd0791b (25 revisions) (flutter/plugins#6832) * 2eb616545 Reland "[google_maps_flutter] ios: re-enable test with popup flutter#5312" (flutter/plugins#6783) * 738bd91d8 Update FlutterFire link (flutter/plugins#6835) * ec2041f82 Roll Flutter from bd0791b to 15af817 (27 revisions) (flutter/plugins#6837)
…5312" (flutter#6783) * reland fix fix test for iOS 16 fix fix typos * format * update changelog
Reland #5312
The accessibility type for the flutter text buttons are changed from buttons to texts.
Fixes flutter/flutter#93325
Pre-launch Checklist
dart format
.)[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style.///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.