Skip to content

Commit

Permalink
Merge pull request #158 from bullhorn/f/toolTip
Browse files Browse the repository at this point in the history
fix for file tooltip not displaying
  • Loading branch information
acrowe2 authored Aug 29, 2017
2 parents 28622b1 + c2a8d2e commit 35433d6
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/app/modal/modal.controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ class CareerPortalModalController {
// 2: EEOC Race/Ethnicity
// 3: EEOC Veteran
// 4: EEOC Disability
if (toolTipType) {
if (toolTipType || toolTipType === 0) {
this.isToolTipHidden = false;
this.currentToolTip = toolTipType;
}
Expand Down
9 changes: 9 additions & 0 deletions test/unit/controllers/modal.controller.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,15 @@ describe('Controller: CareerPortalModalController', () => {
expect(vm.isToolTipHidden).toBe(true);
expect(vm.currentToolTip).toBe(0);
});
it('should be able to set the tooltip to 0.', () => {
expect(vm.showTooltip).toBeDefined();
expect(vm.isToolTipHidden).toBe(true);
expect(vm.currentToolTip).toBe(0);
vm.showTooltip(1);
vm.showTooltip(0);
expect(vm.isToolTipHidden).toBe(false);
expect(vm.currentToolTip).toBe(0);
});
});

describe('Function: hideTooltip()', () => {
Expand Down

0 comments on commit 35433d6

Please sign in to comment.