From a554394e487467da0cf90e5ad45c1d89e5a8451e Mon Sep 17 00:00:00 2001 From: Alexander Reifschneider <alexander.reifschneider@mgm-tp.com> Date: Fri, 17 Jan 2025 13:24:41 +0100 Subject: [PATCH] Revert "Fix unit-tests "init users"" This reverts commit 116a73db77a54716650d609aed191ba04d4dc90f. --- .../lib/user-icon/user-icon.component.spec.ts | 22 ------------------- .../src/lib/user-icon/user-icon.component.ts | 3 +-- .../user-profile-in-header.component.html | 1 - 3 files changed, 1 insertion(+), 25 deletions(-) diff --git a/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.spec.ts b/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.spec.ts index 9f72097d2e..5a26a51d01 100644 --- a/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.spec.ts +++ b/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.spec.ts @@ -32,7 +32,6 @@ import { import { getElementFromFixture } from '@alfa-client/test-utils'; import { SpinnerComponent } from '@alfa-client/ui'; import { NO_NAME_MESSAGE, UserProfileResource, userProfileMessage } from '@alfa-client/user-profile-shared'; -import { SimpleChange } from '@angular/core'; import { ComponentFixture, TestBed } from '@angular/core/testing'; import { MatIcon } from '@angular/material/icon'; import { faker } from '@faker-js/faker'; @@ -120,27 +119,6 @@ describe('UserIconComponent', () => { }); }); - describe('ngOnChanges', () => { - const userProfileChange: SimpleChange = new SimpleChange(1, 2, true); - - it('should get tooltip', () => { - component.getTooltip = jest.fn(); - - component.ngOnChanges({ userProfileStateResource: userProfileChange }); - - expect(component.getTooltip).toHaveBeenCalled(); - }); - - it('should not get tooltip', () => { - component.getTooltip = jest.fn(); - component.withTooltip = false; - - component.ngOnChanges({ userProfileStateResource: userProfileChange }); - - expect(component.getTooltip).not.toHaveBeenCalled(); - }); - }); - describe('tooltip', () => { it('should return user name', () => { component.userProfileStateResource = createStateResource(userProfile); diff --git a/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.ts b/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.ts index 65f11379da..787e1e16ea 100644 --- a/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.ts +++ b/alfa-client/libs/user-profile/src/lib/user-icon/user-icon.component.ts @@ -32,14 +32,13 @@ import { isUndefined } from 'lodash-es'; }) export class UserIconComponent { @Input() userProfileStateResource: StateResource<UserProfileResource> = createEmptyStateResource<UserProfileResource>(); - @Input() withTooltip: boolean = true; readonly messageCode = MessageCode; tooltip: string; ngOnChanges(changes: SimpleChanges): void { - if (changes.userProfileStateResource && this.withTooltip) { + if (changes.userProfileStateResource) { this.tooltip = this.getTooltip(); } } diff --git a/alfa-client/libs/user-profile/src/lib/user-profile-in-header-container/user-profile-in-header/user-profile-in-header.component.html b/alfa-client/libs/user-profile/src/lib/user-profile-in-header-container/user-profile-in-header/user-profile-in-header.component.html index 48c074f478..c73890ac91 100644 --- a/alfa-client/libs/user-profile/src/lib/user-profile-in-header-container/user-profile-in-header/user-profile-in-header.component.html +++ b/alfa-client/libs/user-profile/src/lib/user-profile-in-header-container/user-profile-in-header/user-profile-in-header.component.html @@ -33,7 +33,6 @@ button-content data-test-id="user-icon" [userProfileStateResource]="currentUserResource" - [withTooltip]="false" class="user-profile-icon" > </alfa-user-icon> -- GitLab