[NAE-2125] Remove URI service usage from admin and menu items

- Added unit tests for `AbstractBreadcrumbsComponent` to validate redirection logic and error handling.
- Enhanced test coverage for `AbstractNavigationDoubleDrawerComponent` by adding tests for menu state, sorting, and resize handling.
- Updated `MockUriResourceService` with consistent root path conventions.
- Extended `MockUserService` with logout and empty user functionality.

Author: machacjozef

projects/netgrif-components-core/src/lib/navigation/breadcrumbs/abstract-breadcrumbs.component.spec.ts

@@ -0,0 +1,106 @@
+import { TestBed } from '@angular/core/testing';
+import { Router } from '@angular/router';
+import { LoggerService } from '../../logger/services/logger.service';
+import { PathService } from '../service/path.service';
+import { CaseResourceService } from '../../resources/engine-endpoint/case-resource.service';
+import { ActivatedRoute } from '@angular/router';
+import { DynamicNavigationRouteProviderService } from '../../routing/dynamic-navigation-route-provider/dynamic-navigation-route-provider.service';
+import { TranslateService } from '@ngx-translate/core';
+import {AbstractBreadcrumbsComponent} from "./abstract-breadcrumbs.component";
+
+class TestBreadcrumbsComponent extends AbstractBreadcrumbsComponent {
+    constructor(
+        pathService,
+        caseResourceService,
+        activatedRoute,
+        router,
+        dynamicRoutingService,
+        translateService,
+        logger
+    ) {
+        super(
+            pathService,
+            caseResourceService,
+            activatedRoute,
+            router,
+            dynamicRoutingService,
+            translateService,
+            logger
+        );
+    }
+    public setRedirectUrls(map: Map<string, Array<string>>) {
+        (this as any).redirectUrls = map;
+    }
+}
+
+
+describe('AbstractBreadcrumbsComponent', () => {
+    let component: TestBreadcrumbsComponent;
+    let mockRouter: any;
+    let mockPathService: any;
+    let mockLogger: any;
+
+    beforeEach(() => {
+        mockRouter = jasmine.createSpyObj('Router', ['navigate']);
+        mockRouter.navigate.and.returnValue(Promise.resolve(true));
+
+        mockPathService = {
+            activePath: '/test',
+            splitPath: jasmine.createSpy('splitPath').and.returnValue(['', 'test']),
+            reset: jasmine.createSpy('reset')
+        };
+        mockLogger = jasmine.createSpyObj('LoggerService', ['error']);
+        const mockCaseResourceService = { searchCases: jasmine.createSpy('searchCases') };
+        const mockActivatedRoute = { snapshot: { params: { filterCaseId: 'filter1' } } };
+        const mockDynamicRoutingService = { route: '/dummy-route' };
+        const mockTranslateService = { currentLang: 'en-US' };
+
+        TestBed.configureTestingModule({
+            providers: [
+                { provide: Router, useValue: mockRouter },
+                { provide: PathService, useValue: mockPathService },
+                { provide: LoggerService, useValue: mockLogger },
+                { provide: CaseResourceService, useValue: mockCaseResourceService },
+                { provide: ActivatedRoute, useValue: mockActivatedRoute },
+                { provide: DynamicNavigationRouteProviderService, useValue: mockDynamicRoutingService },
+                { provide: TranslateService, useValue: mockTranslateService },
+            ]
+        });
+
+        component = new TestBreadcrumbsComponent(
+            TestBed.inject(PathService),
+            TestBed.inject(CaseResourceService),
+            TestBed.inject(ActivatedRoute),
+            TestBed.inject(Router),
+            TestBed.inject(DynamicNavigationRouteProviderService),
+            TestBed.inject(TranslateService),
+            TestBed.inject(LoggerService)
+        );
+
+        component.setRedirectUrls(new Map());
+        component.redirectOnClick = true;
+    });
+
+
+    it('should navigate to target if redirect target exists', () => {
+        component.setRedirectUrls(new Map([['/test', ['/somewhere']]]));
+        component.redirect();
+        expect(mockRouter.navigate).toHaveBeenCalledWith(['/somewhere']);
+        expect(mockLogger.error).not.toHaveBeenCalled();
+    });
+
+    it('should log error if redirect target does not exist', () => {
+        component.setRedirectUrls(new Map());
+        component.redirect();
+        expect(mockRouter.navigate).not.toHaveBeenCalled();
+        expect(mockLogger.error).toHaveBeenCalledWith("Missing required data for redirecting breadcrumbs.");
+    });
+
+    it('should not navigate if redirectOnClick is false', () => {
+        component.redirectOnClick = false;
+        component.setRedirectUrls(new Map([['/test', ['/somewhere']]]));
+        component.redirect();
+        expect(mockRouter.navigate).not.toHaveBeenCalled();
+        expect(mockLogger.error).not.toHaveBeenCalled();
+    });
+});

projects/netgrif-components-core/src/lib/navigation/navigation-double-drawer/abstract-navigation-double-drawer.spec.ts

@@ -38,8 +38,10 @@ import {AbstractNavigationDoubleDrawerComponent} from './abstract-navigation-dou
 import {TranslateService} from "@ngx-translate/core";
 import {CaseResourceService} from "../../resources/engine-endpoint/case-resource.service";
 import {MockCaseResourceService} from "../../utility/tests/mocks/mock-case-resource.service";
+import {MenuOrder} from "../model/navigation-configs";
+import {MockUserService} from "../../utility/tests/mocks/mock-user.service";
 
-xdescribe('AbstractNavigationDoubleDrawerComponent', () => {
+describe('AbstractNavigationDoubleDrawerComponent', () => {
     let component: TestDrawerComponent;
     let fixture: ComponentFixture<TestDrawerComponent>;
 
@@ -66,13 +68,14 @@ xdescribe('AbstractNavigationDoubleDrawerComponent', () => {
                 {provide: UserPreferenceService, useClass: MockUserPreferenceService},
                 {provide: CaseResourceService, useClass: MockCaseResourceService},
                 {provide: UriResourceService, useClass: MockUriResourceService},
+                {provide: UserService, useClass: MockUserService},
             ],
             schemas: [CUSTOM_ELEMENTS_SCHEMA],
         }).compileComponents();
         fixture = TestBed.createComponent(TestDrawerComponent);
         component = fixture.componentInstance;
-        spyOn(component, 'toggleLeftMenu');
-        spyOn(component, 'toggleRightMenu');
+        spyOn(component, 'toggleLeftMenu').and.callThrough();
+        spyOn(component, 'toggleRightMenu').and.callThrough();
         fixture.detectChanges();
     }));
 
@@ -123,8 +126,45 @@ xdescribe('AbstractNavigationDoubleDrawerComponent', () => {
 
     it('should check the menu state', () => {
         expect(component.isOnZeroLevel()).toBeTruthy();
-        expect(component.isLeftItemsEmpty).toBeTruthy();
-        expect(component.isRightItemsEmpty).toBeTruthy();
+        expect(component.isLeftItemsEmpty()).toBeTruthy();
+        expect(component.isRightItemsEmpty()).toBeTruthy();
+    });
+
+    it('should load more items', () => {
+        component.moreItems = [{id: 'a'}, {id: 'b'}, {id: 'c'}] as any;
+        component.rightItems = [];
+        component.loadMoreItems();
+        expect(component.rightItems.length).toBeGreaterThan(0);
+    });
+
+    it('should switch order and change sorting', () => {
+        component.rightItems = [
+            {navigation: {title: 'Z'}, id: '1'} as any,
+            {navigation: {title: 'A'}, id: '2'} as any
+        ];
+        component.leftItems = [
+            {navigation: {title: 'B'}, id: '3'} as any,
+            {navigation: {title: 'C'}, id: '4'} as any
+        ];
+        component.itemsOrder = MenuOrder.Ascending;
+        component.switchOrder();
+        console.log(MenuOrder);
+        expect(component.itemsOrder).toBe(MenuOrder.Descending as any);
+    });
+
+
+    it('should return id in itemsTrackBy', () => {
+        expect(component.itemsTrackBy(1, {id: 'test'} as any)).toBe('test');
+    });
+
+    it('should return node path in uriNodeTrackBy', () => {
+        expect(component.uriNodeTrackBy(2, {path: '/test'} as any)).toBe('/test');
+    });
+
+    it('should handle resize event', () => {
+        const event = {rectangle: {width: 420}} as any;
+        component.onResizeEvent(event);
+        expect(component.configRightMenu.width).toBe(420);
     });
 
 });
@@ -152,5 +192,3 @@ class TestDrawerComponent extends AbstractNavigationDoubleDrawerComponent {
             _log, _config, _pathService, _caseResourceService, _impersonationUserSelect, _impersonation, _dynamicRouteProviderService);
     }
 }
-
-

projects/netgrif-components-core/src/lib/utility/tests/mocks/mock-uri-resource.service.ts

@@ -9,13 +9,13 @@ import {ResourceProvider} from '../../../resources/resource-provider.service';
 export class MockUriResourceService extends UriResourceService {
 
     static TEST1_ID = 'test1';
-    static TEST1_PATH = 'root/test1';
+    static TEST1_PATH = '/test1';
     static TEST2_ID = 'test2';
-    static TEST2_PATH = 'root/test2';
+    static TEST2_PATH = '/test2';
 
     private _root: UriNodeResource = {
-        id: 'root',
-        path: 'root',
+        id: '/',
+        path: '/',
         name: 'root',
         parentId: null,
         parent: undefined,
@@ -28,7 +28,7 @@ export class MockUriResourceService extends UriResourceService {
         id: MockUriResourceService.TEST1_ID,
         path: MockUriResourceService.TEST1_PATH,
         name: MockUriResourceService.TEST1_ID,
-        parentId: 'root',
+        parentId: '/',
         parent: this._root,
         childrenId: undefined,
         children: undefined,
@@ -39,7 +39,7 @@ export class MockUriResourceService extends UriResourceService {
         id: MockUriResourceService.TEST2_ID,
         path: MockUriResourceService.TEST2_PATH,
         name: MockUriResourceService.TEST2_ID,
-        parentId: 'root',
+        parentId: '/',
         parent: this._root,
         childrenId: undefined,
         children: undefined,
@@ -61,7 +61,7 @@ export class MockUriResourceService extends UriResourceService {
     }
 
     getNodesByParent(parentId: string): Observable<Array<UriNodeResource>> {
-        if (parentId === 'root') return of([this._test1Node, this._test2Node]);
+        if (parentId === '/') return of([this._test1Node, this._test2Node]);
         return of([this._root]);
     }
 

projects/netgrif-components-core/src/lib/utility/tests/mocks/mock-user.service.ts

@@ -1,5 +1,5 @@
 import {Injectable} from '@angular/core';
-import {Observable, ReplaySubject} from 'rxjs';
+import {Observable, of, ReplaySubject} from 'rxjs';
 import {User} from '../../../user/models/user';
 
 @Injectable()
@@ -50,4 +50,12 @@ export class MockUserService {
         return this.isUserEmpty(this.user)
     }
 
+    logout() {
+        this._user = this.emptyUser()
+        return of({});
+    }
+
+    protected emptyUser() {
+        return new User('', '', '', '', '', '', [], [], [], []);
+    }
 }