-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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 [email protected] Fix template manifest test (flutter/flutter#148616) 2024-05-18 [email protected] Disable shuffling in the flutter_tools create_test suite (flutter/flutter#148619) 2024-05-18 [email protected] Move `Feedback` to widgets layer (flutter/flutter#148523) 2024-05-18 [email protected] Roll Flutter Engine from 93f1b5a146ed to 552a965b707b (4 revisions) (flutter/flutter#148595) 2024-05-18 [email protected] Swap crash reporting with unified analytics (flutter/flutter#148525) 2024-05-17 [email protected] Roll Flutter Engine from 5b3bf9a950b2 to 93f1b5a146ed (3 revisions) (flutter/flutter#148581) 2024-05-17 [email protected] Migrate the flutter/flutter wiki to docs/unsorted_wiki (flutter/flutter#148562) 2024-05-17 [email protected] Roll Flutter Engine from 17decdf12557 to 5b3bf9a950b2 (3 revisions) (flutter/flutter#148567) 2024-05-17 [email protected] Shift a test on MotoG4 to staging, add it on Mokey (flutter/flutter#148552) 2024-05-17 [email protected] Revert "`if` chains � `switch` expressions" (flutter/flutter#148556) 2024-05-17 [email protected] Roll Packages from 87a02e3 to ae4dd32 (9 revisions) (flutter/flutter#148555) 2024-05-17 [email protected] Roll Flutter Engine from 7d244ab0348a to 17decdf12557 (1 revision) (flutter/flutter#148549) 2024-05-17 [email protected] 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 [email protected],[email protected] 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 [email protected] Fix template manifest test (flutter/flutter#148616) 2024-05-18 [email protected] Disable shuffling in the flutter_tools create_test suite (flutter/flutter#148619) 2024-05-18 [email protected] Move `Feedback` to widgets layer (flutter/flutter#148523) 2024-05-18 [email protected] Roll Flutter Engine from 93f1b5a146ed to 552a965b707b (4 revisions) (flutter/flutter#148595) 2024-05-18 [email protected] Swap crash reporting with unified analytics (flutter/flutter#148525) 2024-05-17 [email protected] Roll Flutter Engine from 5b3bf9a950b2 to 93f1b5a146ed (3 revisions) (flutter/flutter#148581) 2024-05-17 [email protected] Migrate the flutter/flutter wiki to docs/unsorted_wiki (flutter/flutter#148562) 2024-05-17 [email protected] Roll Flutter Engine from 17decdf12557 to 5b3bf9a950b2 (3 revisions) (flutter/flutter#148567) 2024-05-17 [email protected] Shift a test on MotoG4 to staging, add it on Mokey (flutter/flutter#148552) 2024-05-17 [email protected] Revert "`if` chains � `switch` expressions" (flutter/flutter#148556) 2024-05-17 [email protected] Roll Packages from 87a02e3 to ae4dd32 (9 revisions) (flutter/flutter#148555) 2024-05-17 [email protected] Roll Flutter Engine from 7d244ab0348a to 17decdf12557 (1 revision) (flutter/flutter#148549) 2024-05-17 [email protected] 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 [email protected],[email protected] 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.