Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Jul 23, 2019

Fix flutter web tests

/cc @liyuqian @jonahwilliams

Fix flutter web tests

/cc @liyuqian @jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@dnfield dnfield merged commit 6603dbd into master Jul 23, 2019
@dnfield dnfield deleted the dnfield-patch-1 branch July 23, 2019 17:14
Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

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

RSLGTM.

That line looks good to me although I'm not sure what it does and what problem it is fixing...

@dnfield
Copy link
Contributor Author

dnfield commented Jul 23, 2019

Running the web tests are guarded right now to be disabled by default. You need to enable them via config or an environment variable to actually run them. All CI was failing previously because of that change upstream in the framework.

@liyuqian
Copy link
Contributor

Thanks for the explanation!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 23, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jul 24, 2019
flutter/engine@8ed5da8...ef99738

git log 8ed5da8..ef99738 --no-merges --oneline
ef99738 Added a DartExecutor API for querying # of pending channel callbacks (flutter/engine#10021)
a0ec528 Embedding testing app (flutter/engine#10007)
2bf1506 Disable DartLifecycleTest::ShuttingDownTheVMShutsDownAllIsolates in runtime_unittests. (flutter/engine#10064)
8edd257 Roll fuchsia/sdk/core/linux-amd64 from M5an7VPM8DiCcNcKe6J0CkAtLk8X9oMeJUqGOrZATIsC to XqtWTBni4xpYCTr7gqU7rFTuXNY1TZ_zOqBJrZM8c_kC (flutter/engine#10061)
e32bdf5 Roll src/third_party/skia 00c680d2bb7c..e11dfd3da4d7 (18 commits) (flutter/engine#10062)
6603dbd Update .cirrus.yml (flutter/engine#10056)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.
johnsonmh pushed a commit to johnsonmh/flutter that referenced this pull request Jul 30, 2019
flutter/engine@8ed5da8...ef99738

git log 8ed5da8..ef99738 --no-merges --oneline
ef99738 Added a DartExecutor API for querying # of pending channel callbacks (flutter/engine#10021)
a0ec528 Embedding testing app (flutter/engine#10007)
2bf1506 Disable DartLifecycleTest::ShuttingDownTheVMShutsDownAllIsolates in runtime_unittests. (flutter/engine#10064)
8edd257 Roll fuchsia/sdk/core/linux-amd64 from M5an7VPM8DiCcNcKe6J0CkAtLk8X9oMeJUqGOrZATIsC to XqtWTBni4xpYCTr7gqU7rFTuXNY1TZ_zOqBJrZM8c_kC (flutter/engine#10061)
e32bdf5 Roll src/third_party/skia 00c680d2bb7c..e11dfd3da4d7 (18 commits) (flutter/engine#10062)
6603dbd Update .cirrus.yml (flutter/engine#10056)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants