Skip to content

Commit 7a5b42f

Browse files
chrisirhcpkozlowski-opensource
authored andcommitted
test(tabs): get test coverage to 100%
- Add one removal of a non-last tab to test one last branch - This excludes the deprecation message for the tabs directive.
1 parent 672a557 commit 7a5b42f

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

Diff for: src/tabs/test/tabs.spec.js

+17
Original file line numberDiff line numberDiff line change
@@ -443,16 +443,33 @@ describe('tabs', function() {
443443
expectTitles(['1', 'tab 1', 'tab 2', 'tab 3']);
444444
expectContents(['Hello', 'content 1', 'content 2', 'content 3']);
445445

446+
// Select last tab
446447
titles().find('a').eq(3).click();
447448
expect(contents().eq(3)).toHaveClass('active');
448449
expect(titles().eq(3)).toHaveClass('active');
449450

451+
// Remove last tab
450452
scope.$apply('list = [1,2]');
451453
expectTitles(['1', 'tab 1', 'tab 2']);
452454
expectContents(['Hello', 'content 1', 'content 2']);
453455

456+
// "tab 2" is now selected
454457
expect(titles().eq(2)).toHaveClass('active');
455458
expect(contents().eq(2)).toHaveClass('active');
459+
460+
// Select 2nd tab ("tab 1")
461+
titles().find('a').eq(1).click();
462+
expect(titles().eq(1)).toHaveClass('active');
463+
expect(contents().eq(1)).toHaveClass('active');
464+
465+
// Remove 2nd tab
466+
scope.$apply('list = [2]');
467+
expectTitles(['1', 'tab 2']);
468+
expectContents(['Hello', 'content 2']);
469+
470+
// New 2nd tab is now selected
471+
expect(titles().eq(1)).toHaveClass('active');
472+
expect(contents().eq(1)).toHaveClass('active');
456473
}));
457474
});
458475

0 commit comments

Comments
 (0)