-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
Codecov Report
@@ Coverage Diff @@
## master #19 +/- ##
==========================================
+ Coverage 98.67% 99.62% +0.94%
==========================================
Files 10 10
Lines 227 264 +37
Branches 38 46 +8
==========================================
+ Hits 224 263 +39
+ Misses 3 1 -2
Continue to review full report at Codecov.
|
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.
Few questions for you @Blackbaud-SteveBrush .
src/app/public/modules/progress-indicator/progress-indicator.component.ts
Show resolved
Hide resolved
src/app/public/modules/progress-indicator/progress-indicator.component.ts
Outdated
Show resolved
Hide resolved
.../modules/progress-indicator/progress-indicator-title/progress-indicator-title.component.html
Show resolved
Hide resolved
fixture.detectChanges(); | ||
|
||
expect(componentInstance.activeIndex).toBe(3); | ||
expect(componentInstance.progressItems.last.isActive).toBeFalsy(); |
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.
I'm confused... How did this test pass before?
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.
It passed because the logic of the steps was incorrect. The final step has an index of 2
because there's only three items. The final step should also be marked as active
. Let me know if I'm missing something obvious!
src/app/public/modules/progress-indicator/progress-indicator.component.ts
Show resolved
Hide resolved
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! Thanks for splitting this up.
No description provided.