Update tests in material library for Material 3 by default #128300
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
flutter-dashboard
bot
added
f: material design
github-actions
bot
added
f: scrolling
QuncCccccc
force-pushed
the
update_material_tests
branch
from
08c41cf to
b5de9bc
Compare
25 tasks
QuncCccccc
force-pushed
the
update_material_tests
branch
from
daf3e0f to
73ec599
Compare
QuncCccccc
changed the title
QuncCccccc
force-pushed
the
update_material_tests
branch
from
b72543a to
71e7fb8
Compare
QuncCccccc
changed the title
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Jun 14, 2023
auto-submit bot
pushed a commit
to flutter/packages
that referenced
this pull request
Jun 14, 2023
Manual roll requested by [email protected] flutter/flutter@09b7e56...95be76a 2023-06-14 [email protected] [web] Migrate framework away from dart:html and package:js (flutter/flutter#128580) 2023-06-14 [email protected] Fixed slider value indicator not disappearing after a bit on desktop platform when slider is clicked not dragged (flutter/flutter#128137) 2023-06-14 [email protected] Inline AbstractNode into SemanticsNode and Layer (flutter/flutter#128467) 2023-06-14 [email protected] Roll Flutter Engine from 727b4413fe6f to 2d8d5ecfe4a8 (5 revisions) (flutter/flutter#128842) 2023-06-14 [email protected] Roll Flutter Engine from 66a5761412f9 to 727b4413fe6f (10 revisions) (flutter/flutter#128841) 2023-06-14 [email protected] Roll Flutter Engine from b6bf3a6f1ccd to 66a5761412f9 (1 revision) (flutter/flutter#128813) 2023-06-13 [email protected] Fix syntax error in docstring (flutter/flutter#128692) 2023-06-13 [email protected] Update unit tests in material library for Material 3 (flutter/flutter#128725) 2023-06-13 [email protected] [flutter_tools] Suppress git output in flutter channel (flutter/flutter#128475) 2023-06-13 [email protected] Fix ensureVisible and default focus traversal for reversed scrollables (flutter/flutter#128756) 2023-06-13 [email protected] Roll Flutter Engine from f9a0a0dafeea to b6bf3a6f1ccd (2 revisions) (flutter/flutter#128797) 2023-06-13 [email protected] Update rest of the unit tests in material library for Material 3 (flutter/flutter#128747) 2023-06-13 [email protected] Update tests in material library for Material 3 by default (flutter/flutter#128300) 2023-06-13 [email protected] Update misc tests for Material3 (flutter/flutter#128712) 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],[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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Aug 16, 2023
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Aug 17, 2023
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Aug 17, 2023
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Aug 17, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates some of the unit tests in the
packages/flutter/test/materialfolder so that they'll pass if ThemeData.useMaterial3 defaults to true.The files in this PR have both M2 and M3 tests, so we don't need to update these files in step 3 of #91605. Will have another PR that only sets
useMaterial3to false and will need to be updated with a M3 version.Also fixes some issues such as in
navigation_drawer_test.dartandnavigation_bar_test.dart, aThemeDataobject (ThemeData.light().copyWith(useMaterial3: true)) is passed to Theme.data, but when we compare the actual value and expected value, we use another ThemeData object (ThemeData()) whoseuseMaterial3is false. It might be because they are new widgets, so they can pass before.In the
switch_test.dart, we already included M3 tests, so the only thing I do here is to mark theuseMaterial3to false in M2 tests.Pre-launch Checklist
///).