From 0c0b1c890aa760a24bb90f081aedfa01d4596e5e Mon Sep 17 00:00:00 2001 From: OZGCloud <ozgcloud@mgm-tp.com> Date: Mon, 18 Nov 2024 10:17:45 +0100 Subject: [PATCH] OZG-4310 missing tests --- .../user-add-formservice.spec.ts | 20 +++++++++++++++++++ .../users-roles/users-roles.component.spec.ts | 13 ++++++++++++ 2 files changed, 33 insertions(+) diff --git a/alfa-client/libs/admin/settings/src/lib/users-roles/user-add-form/user-add-formservice.spec.ts b/alfa-client/libs/admin/settings/src/lib/users-roles/user-add-form/user-add-formservice.spec.ts index 9ab65b7d35..82048540fd 100644 --- a/alfa-client/libs/admin/settings/src/lib/users-roles/user-add-form/user-add-formservice.spec.ts +++ b/alfa-client/libs/admin/settings/src/lib/users-roles/user-add-form/user-add-formservice.spec.ts @@ -68,6 +68,26 @@ describe('UserAddFormService', () => { }); }); + describe('disableUncheckedCheckboxes', () => { + it('if control value is false then control should be disabled', () => { + const control: AbstractControl = alfaGroup.get(UserAddFormService.LOESCHEN); + control.setValue(false); + + formService.disableUncheckedCheckboxes(alfaGroup); + + expect(control.disabled).toBe(true); + }); + + it('if control value is true then control should NOT be disabled', () => { + const control: AbstractControl = alfaGroup.get(UserAddFormService.LOESCHEN); + control.setValue(true); + + formService.disableUncheckedCheckboxes(alfaGroup); + + expect(control.disabled).toBe(false); + }); + }); + describe('updateCheckboxStates', () => { it('if control value is false then control should be disabled', () => { const control: AbstractControl = alfaGroup.get(UserAddFormService.LOESCHEN); diff --git a/alfa-client/libs/admin/settings/src/lib/users-roles/users-roles.component.spec.ts b/alfa-client/libs/admin/settings/src/lib/users-roles/users-roles.component.spec.ts index eb4bbe0774..d7c6ca0764 100644 --- a/alfa-client/libs/admin/settings/src/lib/users-roles/users-roles.component.spec.ts +++ b/alfa-client/libs/admin/settings/src/lib/users-roles/users-roles.component.spec.ts @@ -1,3 +1,4 @@ +import { createStateResource } from '@alfa-client/tech-shared'; import { Mock, mock } from '@alfa-client/test-utils'; import { ComponentFixture, TestBed } from '@angular/core/testing'; import { Router } from '@angular/router'; @@ -5,6 +6,7 @@ import { ButtonWithSpinnerComponent } from '@ods/component'; import { MailboxIconComponent, PersonIconComponent } from '@ods/system'; import { MockComponent, MockPipe } from 'ng-mocks'; import { ROUTES } from '../../../../shared'; +import { createUser } from '../../../test/user/user'; import { ToUserNamePipe } from '../user/to-user-name.pipe'; import { UserService } from './user.service'; import { UsersRolesComponent } from './users-roles.component'; @@ -40,6 +42,17 @@ describe('UsersRolesComponent', () => { }); describe('component', () => { + describe('ngOnInit', () => { + it('should get users from userService', () => { + const userState = createStateResource([createUser()]); + userService.get.mockReturnValueOnce(userState); + + component.ngOnInit(); + + expect(component.users$).toBe(userState); + }); + }); + describe('navigateToAddUser', () => { it('should navigate to add user', () => { const routerSpy = jest.spyOn(router as any, 'navigate'); -- GitLab