Remove RouterLink dependency and update tests to expect events on routing tabs

Co-authored-by: Lipata <[email protected]>

Author: Copilot

projects/igniteui-angular/src/lib/tabs/bottom-nav/bottom-nav.component.spec.ts

@@ -388,7 +388,7 @@ describe('IgxBottomNav', () => {
                 indexChangingSpy = spyOn(bottomNav.selectedIndexChanging, 'emit');
             }));
 
-            it('Validate the events are not fired on clicking tab headers before pressing enter/space key.', fakeAsync(() => {
+            it('Validate the events are fired on clicking tab headers for routing tabs.', fakeAsync(() => {
                 fixture.ngZone.run(() => router.initialNavigation());
                 tick();
                 expect(location.path()).toBe('/');
@@ -398,28 +398,20 @@ describe('IgxBottomNav', () => {
                 });
                 tick();
                 expect(location.path()).toBe('/view2');
-                expect(bottomNav.selectedIndex).toBe(-1);
-
-                expect(indexChangingSpy).not.toHaveBeenCalled();
-                expect(indexChangeSpy).not.toHaveBeenCalled();
-                expect(itemChangeSpy).not.toHaveBeenCalled();
-
-                headers[1].dispatchEvent(KEY_ENTER_EVENT);
-                tick(200);
-                fixture.detectChanges();
+                expect(bottomNav.selectedIndex).toBe(1);
 
-                expect(itemChangeSpy).toHaveBeenCalledWith({
-                    owner: bottomNav,
-                    oldItem: undefined,
-                    newItem: tabItems[1]
-                });
                 expect(indexChangingSpy).toHaveBeenCalledWith({
                     owner: bottomNav,
                     cancel: false,
                     oldIndex: -1,
                     newIndex: 1
                 });
                 expect(indexChangeSpy).toHaveBeenCalledWith(1);
+                expect(itemChangeSpy).toHaveBeenCalledWith({
+                    owner: bottomNav,
+                    oldItem: undefined,
+                    newItem: tabItems[1]
+                });
             }));
         });
     });

projects/igniteui-angular/src/lib/tabs/tab-header.directive.ts

@@ -1,6 +1,5 @@
 
-import { Directive, ElementRef, HostBinding, HostListener, Optional, inject } from '@angular/core';
-import { RouterLink } from '@angular/router';
+import { Directive, ElementRef, HostBinding, HostListener } from '@angular/core';
 import { PlatformUtil } from '../core/utils';
 import { IgxTabItemDirective } from './tab-item.directive';
 import { IgxTabHeaderBase, IgxTabsBase } from './tabs.base';
@@ -18,13 +17,7 @@ export abstract class IgxTabHeaderDirective implements IgxTabHeaderBase {
         public tab: IgxTabItemDirective,
         private elementRef: ElementRef<HTMLElement>,
         protected platform: PlatformUtil
-    ) { 
-        // Inject RouterLink directive if present
-        this.routerLink = inject(RouterLink, { optional: true });
-    }
-
-    /** @hidden */
-    private routerLink?: RouterLink;
+    ) { }
 
     /** @hidden */
     @HostBinding('attr.tabindex')
@@ -47,11 +40,7 @@ export abstract class IgxTabHeaderDirective implements IgxTabHeaderBase {
     /** @hidden */
     @HostListener('click')
     public onClick() {
-        // For routing tabs, let the RouterLink handle navigation and don't select the tab immediately
-        // For other tabs (content tabs or action tabs), allow immediate selection
-        if (!this.routerLink) {
-            this.tabs.selectTab(this.tab, true);
-        }
+        this.tabs.selectTab(this.tab, true);
     }
 
     /** @hidden */

projects/igniteui-angular/src/lib/tabs/tabs/tabs.component.spec.ts

@@ -1089,7 +1089,7 @@ describe('IgxTabs', () => {
                 indexChangingSpy = spyOn(tabs.selectedIndexChanging, 'emit');
             }));
 
-            it('Validate the events are not fired on clicking tab headers before pressing enter/space key.', fakeAsync(() => {
+            it('Validate the events are fired on clicking tab headers for routing tabs.', fakeAsync(() => {
                 fixture.ngZone.run(() => router.initialNavigation());
                 tick();
                 expect(location.path()).toBe('/');
@@ -1099,15 +1099,7 @@ describe('IgxTabs', () => {
                 });
                 tick();
                 expect(location.path()).toBe('/view2');
-                expect(tabs.selectedIndex).toBe(-1);
-
-                expect(indexChangingSpy).not.toHaveBeenCalled();
-                expect(indexChangeSpy).not.toHaveBeenCalled();
-                expect(itemChangeSpy).not.toHaveBeenCalled();
-
-                headers[1].dispatchEvent(KEY_ENTER_EVENT);
-                tick(200);
-                fixture.detectChanges();
+                expect(tabs.selectedIndex).toBe(1);
 
                 expect(indexChangingSpy).toHaveBeenCalledWith({
                     owner: tabs,