Skip to content

Revert "Reland: Removes single window assumptions from flutter_test" - #122193

Merged
auto-submit[bot] merged 1 commit into
masterfrom
revert-122060-issues/117481
Mar 8, 2023
Merged

Revert "Reland: Removes single window assumptions from flutter_test"#122193
auto-submit[bot] merged 1 commit into
masterfrom
revert-122060-issues/117481

Conversation

@CaseyHillers

Copy link
Copy Markdown
Contributor

Reverts #122060

See b/272195069

Google Testing shows this has triggered a large number of failures. I haven't finished running TGPs, but this landed right next to another large scale breakage. Pre-emptively reverting, and we can reland when the roll isn't backed up.

@flutter-dashboardflutter-dashboardBot added a: tests "flutter test", flutter_test, or one of our tests f: focus Focus traversal, gaining or losing focus p: material_ui material_ui package in flutter/packages framework flutter/packages/flutter repository. See also f: labels. labels Mar 8, 2023

@goderbauergoderbauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@CaseyHillersCaseyHillers added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 8, 2023
@goderbauer

Copy link
Copy Markdown
Member

For future archeologists: The failures that caused this revert were likely prompted by the accidental deletion of the framedata getter from TestPlatformDispatcher: 96951c8#r1129776358

@pdblasi-google

Copy link
Copy Markdown
Contributor

For future archeologists: The failures that caused this revert were likely prompted by the accidental deletion of the framedata getter from TestPlatformDispatcher: 96951c8#r1129776358

Man, I really want to write a test/bot like we were talking about. Monitor that these test classes actually do match the signatures they're implementing...

@auto-submit
auto-submitBot merged commit 0091601 into masterMar 8, 2023
@auto-submit
auto-submitBot deleted the revert-122060-issues/117481 branch March 8, 2023 19:42
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 9, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
hannah-hyj pushed a commit to hannah-hyj/flutter that referenced this pull request Mar 11, 2023
…flutter#122060)" (flutter#122193)
Revert "Reland: Removes single window assumptions from `flutter_test`"
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 11, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 12, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests"flutter test", flutter_test, or one of our testsautosubmitMerge PR when tree becomes green via auto submit Appf: focusFocus traversal, gaining or losing focusframeworkflutter/packages/flutter repository. See also f: labels.p: material_uimaterial_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@CaseyHillers@goderbauer@pdblasi-google