-
Notifications
You must be signed in to change notification settings - Fork 4
Removed labeled-by attribute from file input #223
Conversation
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.
@Blackbaud-AlexKingman - just a couple small comments.
fixture.componentInstance.labelText = 'Something different'; | ||
fixture.detectChanges(); | ||
fixture.whenStable().then(() => { | ||
expect(fixture.nativeElement).toBeAccessible(); |
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.
We should now be using expectAsync
on these.
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.
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.
Done.
src/app/public/modules/file-attachment/file-attachment.component.spec.ts
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #223 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 29 29
Lines 869 869
Branches 162 162
=========================================
Hits 869 869 Continue to review full report at Codecov.
|
src/app/public/modules/file-attachment/fixtures/file-attachment.component.fixture.ts
Outdated
Show resolved
Hide resolved
…t.component.fixture.ts Co-authored-by: John Lyons <John.Lyons@blackbaud.com>
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
#222