Skip to content

Clean up standalone imports in tests #31772

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

Clean up standalone imports in tests #31772

wants to merge 37 commits into from
+70 −893

Conversation

crisbeto
Copy link
Member

Most of the imports in our tests aren't necessary anymore after we switched everything over to standalone. These changes clean up all the usages.

@crisbeto crisbeto requested a review from a team as a code owner August 23, 2025 07:39
@crisbeto crisbeto requested review from andrewseguin and wagnermaciel and removed request for a team August 23, 2025 07:39
@crisbeto crisbeto added target: minor This PR is targeted for the next minor release merge: preserve commits When the PR is merged, a rebase and merge should be performed labels Aug 23, 2025
crisbeto added 28 commits August 25, 2025 22:13
@crisbeto
test(material/slider): clean up unnecessary imports in tests
cfcb404 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/badge): clean up unnecessary imports in tests
0f85add 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/bottom-sheet): clean up unnecessary imports in tests
4817eef 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/button): clean up unnecessary imports in tests
b3fd3fa 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/button-toggle): clean up unnecessary imports in tests
8b1670c 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/card): clean up unnecessary imports in tests
86bb118 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/checkbox): clean up unnecessary imports in tests
adb2fe8 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/chips): clean up unnecessary imports in tests
24cad5d 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/core): clean up unnecessary imports in tests
cc95fbc 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/datepicker): clean up unnecessary imports in tests
dcaf206 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/dialog): clean up unnecessary imports in tests
04e287a 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/divider): clean up unnecessary imports in tests
41f9e02 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/expansion): clean up unnecessary imports in tests
407f7dc 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/form-field): clean up unnecessary imports in tests
30c29c1 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/menu): clean up unnecessary imports in tests
5fc102b 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/list): clean up unnecessary imports in tests
2336c8b 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/progress-bar): clean up unnecessary imports in tests
55577cf 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/progress-spinner): clean up unnecessary imports in tests
7be9eaf 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/radio): clean up unnecessary imports in tests
348868c 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/sidenav): clean up unnecessary imports in tests
c898bf2 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/slide-toggle): clean up unnecessary imports in tests
274a099 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/snack-bar): clean up unnecessary imports in tests
b4e1d83 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/sort): clean up unnecessary imports in tests
09495da 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/table): clean up unnecessary imports in tests
cb5a90d 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/tabs): clean up unnecessary imports in tests
d0d068a 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/toolbar): clean up unnecessary imports in tests
62b588c 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/tooltip): clean up unnecessary imports in tests
782187f 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto
test(material/tree): clean up unnecessary imports in tests
2e84514 
Removes imports from tests that aren't necessary anymore, because we're using standalone.
@crisbeto crisbeto force-pushed the full-standalone-import-cleanup branch from 046a882 to 2e84514 Compare August 25, 2025 20:13
@crisbeto crisbeto mentioned this pull request Aug 25, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adolgachev adolgachev adolgachev approved these changes

Assignees
No one assigned
Labels
action: merge The PR is ready for merge by the caretaker area: cdk/a11y area: cdk/clipboard area: cdk/menu area: cdk/observers area: cdk/scrolling area: cdk/table area: cdk/text-field area: material/badge area: material/bottom-sheet area: material/button area: material/button-toggle area: material/card area: material/checkbox area: material/chips area: material/core area: material/datepicker area: material/dialog area: material/divider area: material/expansion area: material/form-field area: material/grid-list area: material/icon area: material/list area: material/menu area: material/progress-bar area: material/progress-spinner area: material/radio area: material/sidenav area: material/slide-toggle area: material/slider area: material/snack-bar area: material/sort area: material/table area: material/tabs area: material/toolbar area: material/tooltip area: material/tree merge: preserve commits When the PR is merged, a rebase and merge should be performed target: minor This PR is targeted for the next minor release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@crisbeto @adolgachev