fix(igx-ts): fix spec files (#1661)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: damyanpetev <damyanpetev@users.noreply.github.com>

Author: 3 people

packages/igx-templates/igx-ts/bullet-graph/default/files/src/app/__path__/__filePrefix__.spec.ts

@@ -18,7 +18,7 @@ describe('<%=ClassName%>', () => {
     component = fixture.componentInstance;
     fixture.detectChanges();
     // disable animation
-    component.bulletGraph.transitionDuration = 0;
+    component.bulletGraph().transitionDuration = 0;
   });
 
   it('should create', () => {

packages/igx-templates/igx-ts/linear-gauge/default/files/src/app/__path__/__filePrefix__.spec.ts

@@ -22,6 +22,6 @@ describe('<%=ClassName%>', () => {
   it('should create', () => {
     expect(component).toBeTruthy();
     // disable animation
-    component.linearGauge.transitionDuration = 0;
+    component.linearGauge().transitionDuration = 0;
   });
 });

packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/auth.guard.spec.ts

@@ -1,4 +1,7 @@
+import { TestBed } from '@angular/core/testing';
+import { Router } from '@angular/router';
 import { AuthGuard } from './auth.guard';
+import { UserStore } from './services/user-store';
 
 describe('AuthGuard', () => {
   let mockRouter: any;
@@ -13,20 +16,28 @@ describe('AuthGuard', () => {
     mockUserService = {
       currentUser: true
     };
+
+    TestBed.configureTestingModule({
+      providers: [
+        AuthGuard,
+        { provide: Router, useValue: mockRouter },
+        { provide: UserStore, useValue: mockUserService }
+      ]
+    });
   });
 
   it('Should properly initialize', () => {
-    const authGuard = new AuthGuard(mockRouter, mockUserService);
+    const authGuard = TestBed.inject(AuthGuard);
     expect(authGuard).toBeDefined();
   });
 
-  it(`Should properly call 'canActivate'`, () => {
-    const authGuard = new AuthGuard(mockRouter, mockUserService);
+  it(`Should return true from 'canActivate' for authenticated user`, () => {
+    const authGuard = TestBed.inject(AuthGuard);
     const mockSpy = vi.fn();
     expect(authGuard.canActivate(mockSpy as any, mockSpy as any)).toEqual(true);
   });
-  it(`Should properly call 'canActivate'`, () => {
-    const authGuard = new AuthGuard(mockRouter, mockUserService);
+  it(`Should redirect and return false from 'canActivate' for unauthenticated user`, () => {
+    const authGuard = TestBed.inject(AuthGuard);
     const mockSpy = vi.fn();
     mockUserService.currentUser = false;
     vi.spyOn(mockRouter, 'navigate');

packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-bar/login-bar.spec.ts

@@ -1,4 +1,3 @@
-import { Component } from '@angular/core';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
 import { NoopAnimationsModule } from '@angular/platform-browser/animations';
@@ -14,28 +13,11 @@ import {
   IgxRippleModule,
   IgxToggleModule
 } from 'igniteui-angular';
-import { LoginDialog } from '../login-dialog/login-dialog';
 import { ExternalAuth } from '../services/external-auth';
 import { UserStore } from '../services/user-store';
 import { provideAuthentication } from '../provide-authentication';
 import { LoginBar } from './login-bar';
 
-@Component({
-  selector: 'app-login-dialog',
-  template: '',
-  imports: [RouterTestingModule,
-	  IgxAvatarModule,
-	  IgxButtonModule,
-	  IgxDialogModule,
-	  IgxDropDownModule,
-	  IgxIconModule,
-	  IgxRippleModule,
-	  IgxToggleModule]
-})
-class TestLoginDialog extends LoginDialog {
-  override open() { }
-}
-
 describe('LoginBar', () => {
   let component: LoginBar;
   let fixture: ComponentFixture<LoginBar>;
@@ -64,8 +46,7 @@ describe('LoginBar', () => {
         IgxIconModule,
         IgxRippleModule,
         IgxToggleModule,
-		LoginBar,
-		TestLoginDialog
+        LoginBar
       ],
       providers: [
         ...provideAuthentication(),
@@ -103,13 +84,10 @@ describe('LoginBar', () => {
   });
 
   it('should open dialog on button click (not logged)', () => {
-    // override ViewChild:
-    component.loginDialog = new TestLoginDialog();
-
     const button = fixture.debugElement.query(By.css('button'));
-    vi.spyOn(component.loginDialog, 'open');
+    vi.spyOn(component.loginDialog(), 'open');
     button.triggerEventHandler('click', {});
-    expect(component.loginDialog.open).toHaveBeenCalled();
+    expect(component.loginDialog().open).toHaveBeenCalled();
   });
 
   it('should open drop down on button click (logged in)', async () => {
@@ -122,7 +100,7 @@ describe('LoginBar', () => {
     const button = fixture.debugElement.query(By.css('button'));
     button.triggerEventHandler('click', {});
     await fixture.whenStable();
-    expect(component.igxDropDown.collapsed).toBeFalsy();
+    expect(component.igxDropDown().collapsed).toBeFalsy();
   });
 
   it('should handle user menu items', async () => {
@@ -133,11 +111,12 @@ describe('LoginBar', () => {
     vi.spyOn(userStore, 'clearCurrentUser');
     vi.spyOn(externalAuth, 'logout');
 
-    component.igxDropDown.open();
-    component.igxDropDown.setSelectedItem(0);
+    const dropdown = component.igxDropDown();
+    dropdown.open();
+    dropdown.setSelectedItem(0);
     expect(router.navigate).toHaveBeenCalledWith(['/auth/profile']);
 
-    component.igxDropDown.setSelectedItem(1);
+    dropdown.setSelectedItem(1);
     expect(router.navigate).toHaveBeenCalledWith(['/home']);
     expect(userStore.clearCurrentUser).toHaveBeenCalled();
     expect(externalAuth.logout).toHaveBeenCalled();

packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/services/jwt.spec.ts

@@ -2,42 +2,59 @@ import { Observable } from 'rxjs';
 import { take } from 'rxjs/operators';
 import { decodeBase64Url, decodeJWT, parseUser } from './jwt-util';
 import { JwtInterceptor } from './jwt.interceptor';
+import { TestBed } from '@angular/core/testing';
+import { UserStore } from './user-store';
 
 describe('JWT Tests', () => {
+  let interceptor: JwtInterceptor;
+  let mockUserStore: any;
+
+  beforeEach(() => {
+    mockUserStore = {
+      currentUser: null
+    };
+
+    TestBed.configureTestingModule({
+      providers: [
+        JwtInterceptor,
+        { provide: UserStore, useValue: mockUserStore }
+      ]
+    });
+
+    interceptor = TestBed.inject(JwtInterceptor);
+  });
+
   afterEach(() => { vi.restoreAllMocks(); });
 
   describe(`JWT Interceptor`, () => {
-    const mockService = {} as any;
-    const jwtInterceptor = new JwtInterceptor(mockService);
-
     it(`Should properly initialize`, () => {
-      expect(jwtInterceptor).toBeTruthy();
+      expect(interceptor).toBeTruthy();
     });
 
     it(`Should properly handle 'intercept'`, () => {
       const mockRequest = {
-        clone: () => { }
+        clone: vi.fn().mockReturnValue({})
       } as any;
       const mockNext = {
-        handle: () => new Observable()
+        handle: vi.fn(() => new Observable(sub => sub.complete()))
       } as any;
-      vi.spyOn(mockNext, 'handle');
-      mockService.currentUser = false;
+
+      mockUserStore.currentUser = false;
       // call w/o current user
-      jwtInterceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe(() => { });
+      interceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe();
       expect(mockNext.handle).toHaveBeenCalledTimes(1);
       expect(mockNext.handle).toHaveBeenCalledWith(mockRequest);
-      mockService.currentUser = {
+      mockUserStore.currentUser = {
         token: ''
       };
-      jwtInterceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe(() => { });
+      interceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe();
       expect(mockNext.handle).toHaveBeenCalledTimes(2);
       expect(mockNext.handle).toHaveBeenCalledWith(mockRequest);
-      mockService.currentUser = {
+      mockUserStore.currentUser = {
         token: 'test_token'
       };
-      vi.spyOn(mockRequest, 'clone');
-      jwtInterceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe(() => { });
+
+      interceptor.intercept(mockRequest, mockNext).pipe(take(1)).subscribe();
       expect(mockRequest.clone).toHaveBeenCalledWith({
         setHeaders: {
           Authorization: `Bearer test_token`