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.

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

[webview_flutter] Fix macOS 14+ test hangs #7953

Conversation

stuartmorgan
Copy link
Contributor

Starting in macOS 14, webview_flutter integration tests hang due to being unable to connect to the test server. Based on local experimentation, in appears that we are not able to connect when binding to the "any" address, which gives us an IP address of 0.0.0.0, but using the loopback address (which is all we should need; the point is to accept local connections from the test itself) does work.

Since our test fleet is a mixed macOS 13/14 fleet, this should fix flaky timeouts of macOS platform tests in CI, in addition to fixing local runs.

Fixes flutter/flutter#157449

Pre-launch Checklist

Starting in macOS 14, webview_flutter integration tests hang due to
being unable to connect to the test server. Based on local
experimentation, in appears that we are not able to connect when binding
to the "any" address, which gives us an IP address of 0.0.0.0, but using
the loopback address (which is all we should need; the point is to
accept local connections from the test itself) does work.

Fixes flutter/flutter#157449
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 29, 2024
@auto-submit auto-submit bot merged commit ab404b5 into flutter:main Oct 29, 2024
76 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 30, 2024
flutter/packages@028027e...7cc1caa

2024-10-30 [email protected] [tool] Update build-all app creation (flutter/packages#7955)
2024-10-30 [email protected] Disable two lint rules which are being removed. (flutter/packages#7959)
2024-10-29 [email protected] Bump `path_provider_android` to `^2.2.5` due to Android v1 embedder. (flutter/packages#7935)
2024-10-29 [email protected] Roll Flutter from 3fb95c7 to 42132e8 (19 revisions) (flutter/packages#7958)
2024-10-29 [email protected] [webview_flutter] Fix macOS 14+ test hangs (flutter/packages#7953)

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] 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: webview_flutter platform-ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[webview] Integration tests hang locally on macOS
2 participants