-
Notifications
You must be signed in to change notification settings - Fork 6.8k
docs(material/chips): add chips harness example #20942
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.
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
Conversation
4fe6c5f
to
7bc9531
Compare
); | ||
|
||
it('should get whether a chip list is disabled', async () => { | ||
const chipLists = await loader.getAllHarnesses(MatChipListHarness); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you're only testing the first list, you can use loader.getHarness
instead. It'll help you avoid repeating [0]
below. The same goes for the other tests that only look for one instance.
7bc9531
to
b442538
Compare
Changed to using |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
(cherry picked from commit 554c775)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
No description provided.