From 8e377f3ef34bb45e5b0d86b3877a8f18759ec5ea Mon Sep 17 00:00:00 2001 From: Aditya M <150046114+AdityaMantripragada@users.noreply.github.com> Date: Tue, 22 Oct 2024 08:56:36 -0700 Subject: [PATCH] Pidp 1047/1048 window scroll console error&open multiple dialogs (#602) * windows scroll console error fix * multiple dialog box fix * added back scroll top * Lint fix * revert * missing dependency --------- Co-authored-by: Panos Hatzinikolaou --- .../user-access-agreement.page.spec.ts | 3 ++- .../confirm-dialog/confirm-dialog.component.html | 1 + .../dialogs/confirm-dialog/confirm-dialog.component.ts | 10 +++++++++- 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/workspace/apps/pidp/src/app/features/profile/pages/user-access-agreement/user-access-agreement.page.spec.ts b/workspace/apps/pidp/src/app/features/profile/pages/user-access-agreement/user-access-agreement.page.spec.ts index 690b11635..ca2f6e15d 100644 --- a/workspace/apps/pidp/src/app/features/profile/pages/user-access-agreement/user-access-agreement.page.spec.ts +++ b/workspace/apps/pidp/src/app/features/profile/pages/user-access-agreement/user-access-agreement.page.spec.ts @@ -11,6 +11,7 @@ import { provideAutoSpy } from 'jest-auto-spies'; import { ApiHttpClient } from '@app/core/resources/api-http-client.service'; import { FormUtilsService } from '@app/core/services/form-utils.service'; import { LoggerService } from '@app/core/services/logger.service'; +import { UtilsService } from '@app/core/services/utils.service'; import { UserAccessAgreementResource } from './user-access-agreement-resource.service'; import { UserAccessAgreementPage } from './user-access-agreement.page'; @@ -46,6 +47,7 @@ describe('UserAccessAgreementPage', () => { provideAutoSpy(LoggerService), provideAutoSpy(Router), provideAutoSpy(UserAccessAgreementResource), + provideAutoSpy(UtilsService), ], }); @@ -61,7 +63,6 @@ describe('UserAccessAgreementPage', () => { given('user wants to go back to the previous page', () => { when('onBack is invoked', () => { component.onBack(); - then('router should navigate to root route', () => { const rootRoute = mockActivatedRoute.snapshot.data.routes.root; expect(router.navigate).toHaveBeenCalledWith([rootRoute]); diff --git a/workspace/libs/shared/ui/src/lib/components/dialogs/confirm-dialog/confirm-dialog.component.html b/workspace/libs/shared/ui/src/lib/components/dialogs/confirm-dialog/confirm-dialog.component.html index b3c2315c0..83a26a527 100644 --- a/workspace/libs/shared/ui/src/lib/components/dialogs/confirm-dialog/confirm-dialog.component.html +++ b/workspace/libs/shared/ui/src/lib/components/dialogs/confirm-dialog/confirm-dialog.component.html @@ -39,6 +39,7 @@ {{ options.cancelText }}