Skip to content

Revert "Treat hidden IndexedStack children as offstage for test finder"#124035

Closed
CaseyHillers wants to merge 1 commit intomasterfrom
revert-123129-indexed-stack-hide-from-finder-v2
Closed

Revert "Treat hidden IndexedStack children as offstage for test finder"#124035
CaseyHillers wants to merge 1 commit intomasterfrom
revert-123129-indexed-stack-hide-from-finder-v2

Conversation

@CaseyHillers
Copy link
Contributor

Reverts #123129

This is breaking Google Testing. 2 tests are failing to find widgets that used to be findable. They're able to find when skipOffstage is set to false.

@flutter-dashboard flutter-dashboard bot added a: tests "flutter test", flutter_test, or one of our tests d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. c: contributor-productivity Team-specific productivity, code health, technical debt. labels Apr 3, 2023
@goderbauer
Copy link
Member

This shouldn't be necessary as a g3fix for those failures has been scheduled. We can leave it open in case the g3fix fails for some reason, though.

@guidezpl guidezpl deleted the revert-123129-indexed-stack-hide-from-finder-v2 branch April 18, 2024 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants