Skip to content

Commit 6f9d767

Browse files
mateusjmafjhonyeduardo
authored andcommitted
test(navbar): corrige a falha no teste do método validateMenuLogo
Criado um novo "fake this" e inserido na função `call` da chamada do método `validateMenuLogo`. Fixes DTHFUI-1674
1 parent fbb61b8 commit 6f9d767

File tree

1 file changed

+40
-26
lines changed

1 file changed

+40
-26
lines changed

projects/ui/src/lib/components/po-navbar/po-navbar.component.spec.ts

Lines changed: 40 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -539,48 +539,62 @@ describe('PoNavbarComponent:', () => {
539539

540540
});
541541

542-
describe('validateMenuLogo', () => {
543-
544-
const logo = 'logo';
545-
546-
const fakeMenu = {
547-
logo,
548-
changeDetector: { detectChanges: () => {} }
549-
};
542+
describe('validateMenuLogo:', () => {
550543

551544
it(`should set 'menu.logo' as 'undefined' and call 'menu.changeDetector.detectChanges' if has 'logo' and 'menu.logo'`, () => {
552-
component.logo = logo;
553-
component.menu = <any>fakeMenu;
545+
const fakeThis = {
546+
logo: 'logo',
547+
menu: {
548+
logo: 'logo',
549+
changeDetector: {
550+
detectChanges: () => {}
551+
}
552+
}
553+
};
554554

555-
spyOn(component.menu.changeDetector, 'detectChanges');
556-
component['validateMenuLogo']();
557-
fixture.detectChanges();
555+
spyOn(fakeThis.menu.changeDetector, <any> 'detectChanges');
556+
component['validateMenuLogo'].call(fakeThis);
558557

559-
expect(component.menu.logo).toBeUndefined();
560-
expect(component.menu.changeDetector.detectChanges).toHaveBeenCalled();
558+
expect(fakeThis.menu.logo).toBeUndefined();
559+
expect(fakeThis.menu.changeDetector.detectChanges).toHaveBeenCalled();
561560
});
562561

563562
it(`shouldn't call 'menu.changeDetector.detectChanges' if doesn't have 'menu.logo'`, () => {
564-
component.logo = logo;
565-
component.menu = <any>fakeMenu;
566-
component.menu.logo = undefined;
563+
const fakeThis = {
564+
logo: 'logo',
565+
menu: {
566+
logo: undefined,
567+
changeDetector: {
568+
detectChanges: () => {}
569+
}
570+
}
571+
};
567572

568-
spyOn(component.menu.changeDetector, 'detectChanges');
573+
spyOn(fakeThis.menu.changeDetector, 'detectChanges');
569574

570-
component['validateMenuLogo']();
575+
component['validateMenuLogo'].call(fakeThis);
571576

572-
expect(component.menu.changeDetector.detectChanges).not.toHaveBeenCalled();
577+
expect(fakeThis.menu.logo).toBeUndefined();
578+
expect(fakeThis.menu.changeDetector.detectChanges).not.toHaveBeenCalled();
573579
});
574580

575581
it(`shouldn't call 'menu.changeDetector.detectChanges' if doesn't have 'logo'`, () => {
576-
component.logo = undefined;
577-
component.menu = <any>fakeMenu;
582+
const fakeThis = {
583+
logo: undefined,
584+
menu: {
585+
logo: 'logo',
586+
changeDetector: {
587+
detectChanges: () => {}
588+
}
589+
}
590+
};
578591

579-
spyOn(component.menu.changeDetector, 'detectChanges');
592+
spyOn(fakeThis.menu.changeDetector, 'detectChanges');
580593

581-
component['validateMenuLogo']();
594+
component['validateMenuLogo'].call(fakeThis);
582595

583-
expect(component.menu.changeDetector.detectChanges).not.toHaveBeenCalled();
596+
expect(fakeThis.logo).toBeUndefined();
597+
expect(fakeThis.menu.changeDetector.detectChanges).not.toHaveBeenCalled();
584598
});
585599
});
586600

0 commit comments

Comments
 (0)