Removed ClientRouter.getUrl implementation

seznam · Jun 4, 2024 · 27468dc · 27468dc
1 parent d459fd2
commit 27468dc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/packages/core/src/router/ClientRouter.ts b/packages/core/src/router/ClientRouter.ts
@@ -106,13 +106,6 @@ export class ClientRouter extends AbstractRouter {
     return this;
   }
 
-  /**
-   * @inheritDoc
-   */
-  getUrl() {
-    return this._window.getUrl();
-  }
-
   /**
    * @inheritDoc
    */

diff --git a/packages/core/src/router/__tests__/ClientRouterSpec.ts b/packages/core/src/router/__tests__/ClientRouterSpec.ts
@@ -35,7 +35,7 @@ describe('ima.core.router.ClientRouter', () => {
       30000
     );
 
-    jest.spyOn(router, 'getPath').mockReturnValue('/routePath');
+    jest.spyOn(router, 'getPath').mockReturnValue('/routePath?foo=bar');
 
     router.init(routerConfig);
   });
@@ -50,11 +50,13 @@ describe('ima.core.router.ClientRouter', () => {
   });
 
   it('should be return actual url', () => {
-    jest.spyOn(window, 'getUrl').mockImplementation();
+    jest.spyOn(router, 'getBaseUrl');
+    jest.spyOn(router, 'getPath');
 
-    router.getUrl();
+    expect(router.getUrl()).toBe('http://locahlost:3002/routePath?foo=bar');
 
-    expect(window.getUrl).toHaveBeenCalled();
+    expect(router.getBaseUrl).toHaveBeenCalled();
+    expect(router.getPath).toHaveBeenCalled();
   });
 
   it('should add listener to popState event, click event', () => {