From 33a138809a7a6dd45ebb19268e075c5dea1e63ff Mon Sep 17 00:00:00 2001 From: Shivangi Shree Date: Tue, 28 Jul 2026 22:56:11 +0530 Subject: [PATCH] [ACS-8427] Fix search behaviour on closing (#5310) * [ACS-8427] Fix search behaviour on closing * [ACS-8427] Use navigateBack function name instead of exitSearch * [ACS-8427] Fix function name in unit test description --- .../search/search-navigation.service.spec.ts | 15 ++++++++++++--- .../search/search-navigation.service.ts | 2 +- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/projects/aca-content/src/lib/components/search/search-navigation.service.spec.ts b/projects/aca-content/src/lib/components/search/search-navigation.service.spec.ts index 6b62c6324..24dc3e220 100644 --- a/projects/aca-content/src/lib/components/search/search-navigation.service.spec.ts +++ b/projects/aca-content/src/lib/components/search/search-navigation.service.spec.ts @@ -50,12 +50,21 @@ describe('SearchNavigationService', () => { expect(routerNavigate).not.toHaveBeenCalledWith([service.previousRoute]); }); - it('should navigate to saved route when exitSearch function is called', () => { - const routerNavigate = spyOn(router, 'navigate'); + it('should navigate to saved route when navigateBack function is called', () => { + const routerNavigateByUrl = spyOn(router, 'navigateByUrl'); service.saveRoute('/personal-files'); service.navigateBack(); - expect(routerNavigate).toHaveBeenCalledWith([service.previousRoute]); + expect(routerNavigateByUrl).toHaveBeenCalledWith(service.previousRoute); + }); + + it('should navigate to saved route including query params when navigateBack function is called', () => { + const routerNavigateByUrl = spyOn(router, 'navigateByUrl'); + const savedRoute = '/personal-files/details/some-node-id?location=%2Fpersonal-files'; + service.saveRoute(savedRoute); + service.navigateBack(); + + expect(routerNavigateByUrl).toHaveBeenCalledWith(savedRoute); }); it('should navigate to Search when navigateToSearch function is called', () => { diff --git a/projects/aca-content/src/lib/components/search/search-navigation.service.ts b/projects/aca-content/src/lib/components/search/search-navigation.service.ts index 6530ccdea..b8682a4d8 100644 --- a/projects/aca-content/src/lib/components/search/search-navigation.service.ts +++ b/projects/aca-content/src/lib/components/search/search-navigation.service.ts @@ -52,7 +52,7 @@ export class SearchNavigationService { navigateBack(): void { if (this.previousRoute) { - this.router.navigate([this.previousRoute]); + this.router.navigateByUrl(this.previousRoute); } else { this.router.navigate(['/personal-files']); }