Skip to content

Remove material import from banner_test.dart#181274

Closed
navaronbracke wants to merge 1 commit into
flutter:masterfrom
navaronbracke:checked_mode_banner_test_no_material
Closed

Remove material import from banner_test.dart#181274
navaronbracke wants to merge 1 commit into
flutter:masterfrom
navaronbracke:checked_mode_banner_test_no_material

Conversation

@navaronbracke

Copy link
Copy Markdown
Contributor

This PR removes material imports from /widgets/banner_test.dart

Closes #181261
Part of #177415
Depends on #180456 for TestWidgetsApp

List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@github-actions github-actions Bot added the framework flutter/packages/flutter repository. See also f: labels. label Jan 21, 2026
@justinmc justinmc self-requested a review January 21, 2026 21:40
@navaronbracke

Copy link
Copy Markdown
Contributor Author

Closing in favor of #181261, since using WidgetsApp was enough there.

@navaronbracke navaronbracke deleted the checked_mode_banner_test_no_material branch January 22, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant