Thanks to visit codestin.com
Credit goes to github.com

Skip to content

(Test cross-imports) No legacy Material imports in tests #177412

@justinmc

Description

@justinmc

Umbrella issue: #177028
Design doc: https://docs.google.com/document/d/1UHxALQqCbmgjnM1RNV9xE2pK3IGyx-UktGX1D7hYCjs/edit?tab=t.0

Any test with a Material import that isn't strictly necessary should remove it. This will help with our general test organization and also help remove a devDependency after Material and Cupertino are decoupled from the framework.

How to fix

  1. Find a test listed below that is not already fixed (checked) and that does not have a PR already linked.
  2. Hopefully all you need to do is to remove the Material import and make sure the tests still pass.
  3. When you open a PR, reference this issue as "Part of " instead of "Fixes", to avoid it being prematurely closed.

Tests to fix:

Located in Widgets, imported Material

Located in Widgets, imported Cupertino

Located in Cupertino, imported Material

Metadata

Metadata

Assignees

No one assigned

    Labels

    P1High-priority issues at the top of the work lista: tests"flutter test", flutter_test, or one of our testsc: proposalA detailed proposal for a change to Flutterframeworkflutter/packages/flutter repository. See also f: labels.good first issueRelatively approachable for first-time contributorsteam-frameworkOwned by Framework teamtriaged-frameworkTriaged by Framework team

    Type

    No type

    Projects

    Status

    In Progress

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions