-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Fix template manifest test #148616
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
Fix template manifest test #148616
Conversation
72a3f01 to
11b4488
Compare
a810413 to
d2a7a67
Compare
|
To add more context, here are some logs from a failure: https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8747578640646662689/+/u/run_test.dart_for_tool_tests_shard_and_subshard_commands/stdout This is probably a flake depending on test order execution. I notice a lot of tests in this file tamper with the global |
andrewkolos
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. Thanks!
flutter/flutter@00425ef...adf279f 2024-05-18 nate.w5687@gmail.com Fix template manifest test (flutter/flutter#148616) 2024-05-18 jason-simmons@users.noreply.github.com Disable shuffling in the flutter_tools create_test suite (flutter/flutter#148619) 2024-05-18 victorsanniay@gmail.com Move `Feedback` to widgets layer (flutter/flutter#148523) 2024-05-18 engine-flutter-autoroll@skia.org Roll Flutter Engine from 93f1b5a146ed to 552a965b707b (4 revisions) (flutter/flutter#148595) 2024-05-18 andrewrkolos@gmail.com Swap crash reporting with unified analytics (flutter/flutter#148525) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5b3bf9a950b2 to 93f1b5a146ed (3 revisions) (flutter/flutter#148581) 2024-05-17 katelovett@google.com Migrate the flutter/flutter wiki to docs/unsorted_wiki (flutter/flutter#148562) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 17decdf12557 to 5b3bf9a950b2 (3 revisions) (flutter/flutter#148567) 2024-05-17 zanderso@users.noreply.github.com Shift a test on MotoG4 to staging, add it on Mokey (flutter/flutter#148552) 2024-05-17 zanderso@users.noreply.github.com Revert "`if` chains � `switch` expressions" (flutter/flutter#148556) 2024-05-17 engine-flutter-autoroll@skia.org Roll Packages from 87a02e3 to ae4dd32 (9 revisions) (flutter/flutter#148555) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 7d244ab0348a to 17decdf12557 (1 revision) (flutter/flutter#148549) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from a19d3722922d to 7d244ab0348a (4 revisions) (flutter/flutter#148546) 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 Please CC rmistry@google.com,stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: 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
The issue was fixed by flutter#148616
flutter/flutter@00425ef...adf279f 2024-05-18 nate.w5687@gmail.com Fix template manifest test (flutter/flutter#148616) 2024-05-18 jason-simmons@users.noreply.github.com Disable shuffling in the flutter_tools create_test suite (flutter/flutter#148619) 2024-05-18 victorsanniay@gmail.com Move `Feedback` to widgets layer (flutter/flutter#148523) 2024-05-18 engine-flutter-autoroll@skia.org Roll Flutter Engine from 93f1b5a146ed to 552a965b707b (4 revisions) (flutter/flutter#148595) 2024-05-18 andrewrkolos@gmail.com Swap crash reporting with unified analytics (flutter/flutter#148525) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5b3bf9a950b2 to 93f1b5a146ed (3 revisions) (flutter/flutter#148581) 2024-05-17 katelovett@google.com Migrate the flutter/flutter wiki to docs/unsorted_wiki (flutter/flutter#148562) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 17decdf12557 to 5b3bf9a950b2 (3 revisions) (flutter/flutter#148567) 2024-05-17 zanderso@users.noreply.github.com Shift a test on MotoG4 to staging, add it on Mokey (flutter/flutter#148552) 2024-05-17 zanderso@users.noreply.github.com Revert "`if` chains � `switch` expressions" (flutter/flutter#148556) 2024-05-17 engine-flutter-autoroll@skia.org Roll Packages from 87a02e3 to ae4dd32 (9 revisions) (flutter/flutter#148555) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from 7d244ab0348a to 17decdf12557 (1 revision) (flutter/flutter#148549) 2024-05-17 engine-flutter-autoroll@skia.org Roll Flutter Engine from a19d3722922d to 7d244ab0348a (4 revisions) (flutter/flutter#148546) 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 Please CC rmistry@google.com,stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: 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
'flutter create should tool exit if the template manifest cannot be read'fails consistently, as shown by #148614.The test expects a
ToolExitwith the message "Unable to read the template manifest", but depending on how the test is being run, a different exception ("Cannot create a project within the Flutter SDK") is sometimes thrown first.This pull request relocates the test project to
dev/to prevent the extraneous error.