Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[google_maps_flutter] ios: re-enable test with popup #5312

Merged

Conversation

cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Apr 20, 2022

Re-able the test that triggers system alert as it seems not failing anymore in newest iOS versions.

fixes: flutter/flutter#93325

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@cyanglaz
Copy link
Contributor Author

cyanglaz commented May 3, 2022

Need to wait until flutter/engine#32820 going to stable before we can land this, so that the test can pass on both stable and master

@cyanglaz
Copy link
Contributor Author

cyanglaz commented Jul 12, 2022

The engine roll commit is: flutter/flutter@dd65b6b

We can merge when is commit is in stable branch.

@stuartmorgan
Copy link
Contributor

We can merge when is commit is in stable branch.

That day is upon us :)

@cyanglaz cyanglaz force-pushed the enable_google_map_interface_test branch from 94c585a to f0ff4e7 Compare November 30, 2022 00:41
@cyanglaz cyanglaz marked this pull request as ready for review November 30, 2022 00:41
Copy link
Contributor

@stuartmorgan stuartmorgan left a comment

Choose a reason for hiding this comment

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

LGTM!

@cyanglaz cyanglaz added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 30, 2022
@auto-submit auto-submit bot merged commit 89ad5a9 into flutter:main Dec 1, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 1, 2022
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 1, 2022
* e911e0564 b8f7f1f [flutter_releases] Flutter stable 3.3.9 Framework Cherrypicks (#115856) (flutter/plugins#6751)

* bdf1a98a1 Roll Flutter from 0eb2d51 to cb23473 (13 revisions) (flutter/plugins#6755)

* 614d7c0df Roll Flutter from cb23473 to ff59250 (3 revisions) (flutter/plugins#6757)

* 2be1da958 Roll Flutter from ff59250 to 17c1dbc (31 revisions) (flutter/plugins#6764)

* 381541628 [in_app_purchase_storekit] Add support for macOS (flutter/plugins#6517)

* eb796fe2e Roll Flutter from 17c1dbc to b2672fe (26 revisions) (flutter/plugins#6766)

* 89ad5a9d6 enable test (flutter/plugins#5312)
@ditman
Copy link
Member

ditman commented Dec 1, 2022

cyanglaz pushed a commit that referenced this pull request Dec 2, 2022
This reverts commit 89ad5a9.
cyanglaz pushed a commit that referenced this pull request Dec 2, 2022
mit-mit pushed a commit to mit-mit/flutter that referenced this pull request Dec 6, 2022
…#116356)

* e911e0564 b8f7f1f [flutter_releases] Flutter stable 3.3.9 Framework Cherrypicks (flutter#115856) (flutter/plugins#6751)

* bdf1a98a1 Roll Flutter from 0eb2d51 to cb23473 (13 revisions) (flutter/plugins#6755)

* 614d7c0df Roll Flutter from cb23473 to ff59250 (3 revisions) (flutter/plugins#6757)

* 2be1da958 Roll Flutter from ff59250 to 17c1dbc (31 revisions) (flutter/plugins#6764)

* 381541628 [in_app_purchase_storekit] Add support for macOS (flutter/plugins#6517)

* eb796fe2e Roll Flutter from 17c1dbc to b2672fe (26 revisions) (flutter/plugins#6766)

* 89ad5a9d6 enable test (flutter/plugins#5312)
shogohida pushed a commit to shogohida/flutter that referenced this pull request Dec 7, 2022
…#116356)

* e911e0564 b8f7f1f [flutter_releases] Flutter stable 3.3.9 Framework Cherrypicks (flutter#115856) (flutter/plugins#6751)

* bdf1a98a1 Roll Flutter from 0eb2d51 to cb23473 (13 revisions) (flutter/plugins#6755)

* 614d7c0df Roll Flutter from cb23473 to ff59250 (3 revisions) (flutter/plugins#6757)

* 2be1da958 Roll Flutter from ff59250 to 17c1dbc (31 revisions) (flutter/plugins#6764)

* 381541628 [in_app_purchase_storekit] Add support for macOS (flutter/plugins#6517)

* eb796fe2e Roll Flutter from 17c1dbc to b2672fe (26 revisions) (flutter/plugins#6766)

* 89ad5a9d6 enable test (flutter/plugins#5312)
auto-submit bot pushed a commit that referenced this pull request Dec 12, 2022
…6783)

* reland

fix

fix test for iOS 16

fix

fix typos

* format

* update changelog
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…#116356)

* e911e0564 b8f7f1f [flutter_releases] Flutter stable 3.3.9 Framework Cherrypicks (flutter#115856) (flutter/plugins#6751)

* bdf1a98a1 Roll Flutter from 0eb2d51 to cb23473 (13 revisions) (flutter/plugins#6755)

* 614d7c0df Roll Flutter from cb23473 to ff59250 (3 revisions) (flutter/plugins#6757)

* 2be1da958 Roll Flutter from ff59250 to 17c1dbc (31 revisions) (flutter/plugins#6764)

* 381541628 [in_app_purchase_storekit] Add support for macOS (flutter/plugins#6517)

* eb796fe2e Roll Flutter from 17c1dbc to b2672fe (26 revisions) (flutter/plugins#6766)

* 89ad5a9d6 enable test (flutter/plugins#5312)
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
…5312" (flutter#6783)

* reland

fix

fix test for iOS 16

fix

fix typos

* format

* update changelog
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: google_maps_flutter platform-ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[google_maps_flutter] XCUITests stuck when handling permission popups
3 participants