chore: cxspa-13736 - migrate 14 remaining libs from karma to vitest

core-libs/storefront/cms-components/navigation/navigation/navigation-ui.component.ts

@@ -27,7 +27,8 @@ import {
   filter,
   take,
 } from 'rxjs/operators';
-import { BREAKPOINT, BreakpointService } from '../../../layout';
+import { BREAKPOINT } from '../../../layout/config/layout-config';
+import { BreakpointService } from '../../../layout/breakpoint/breakpoint.service';
 import { GenericLinkComponent } from '../../../shared/components/generic-link/generic-link.component';
 import { IconComponent } from '../../misc/icon/icon.component';
 import { ICON_TYPE } from '../../misc/icon/index';

core-libs/storefront/shared/components/ng-select-a11y/ng-select-a11y.directive.ts

@@ -31,7 +31,8 @@ import {
 } from '@spartacus/core';
 import { filter, merge, take } from 'rxjs';
 import { map } from 'rxjs/operators';
-import { BREAKPOINT, BreakpointService } from '../../../layout';
+import { BREAKPOINT } from '../../../layout/config/layout-config';
+import { BreakpointService } from '../../../layout/breakpoint/breakpoint.service';
 
 const ARIA_LABEL = 'aria-label';
 const ARIA_HIDDEN = 'aria-hidden';

feature-libs/cart/base/components/cart-shared/cart-item-list/cart-item-list.component.spec.ts

@@ -671,9 +671,13 @@ describe('CartItemListComponent', () => {
     });
 
     it('should not call _setItems when neither contextRequiresRerender nor isItemsChanged are true', () => {
-      configureTestingModule().overrideProvider(OutletContextData, {
-        useValue: { context$ },
-      });
+      configureTestingModule()
+        .overrideProvider(OutletContextData, {
+          useValue: { context$ },
+        })
+        .overrideProvider(FeatureToggles, {
+          useValue: { a11yPreventCartItemsFormRedundantRecreation: true },
+        });
       TestBed.compileComponents();
       stubServiceAndCreateComponent();
 
@@ -682,9 +686,6 @@ describe('CartItemListComponent', () => {
       // and feature toggle is explicitly enabled (dual-token issue under Vite)
       component.readonly = mockContext.readonly;
       fixture.componentRef.setInput('items', mockContext.items);
-      (component as any)['featureToggles'] = {
-        a11yPreventCartItemsFormRedundantRecreation: true,
-      };
 
       const spySetItems = vi.spyOn(<any>component, '_setItems');
       component.ngOnInit();

feature-libs/cart/base/core/facade/active-cart.service.spec.ts

@@ -1,6 +1,7 @@
 import { TestBed } from '@angular/core/testing';
 import { Cart, MultiCartFacade, OrderEntry } from '@spartacus/cart/base/root';
 import {
+  FeatureToggles,
   getLastValueSync,
   OCC_CART_ID_CURRENT,
   OCC_USER_ID_ANONYMOUS,
@@ -22,7 +23,10 @@ import {
 } from 'rxjs';
 import { take } from 'rxjs/operators';
 import { vi } from 'vitest';
-import { provideMockFeatureToggles } from '@spartacus/core/testing/mock-feature-toggles';
+import {
+  MockFeatureTogglesController,
+  provideMockFeatureToggles,
+} from '@spartacus/core/testing/mock-feature-toggles';
 import { ActiveCartService } from './active-cart.service';
 
 const userId$ = new BehaviorSubject<string>(OCC_USER_ID_ANONYMOUS);
@@ -72,9 +76,7 @@ const MockWindowRef = {
       store[key] = `${value}`;
     },
     removeItem: (key: string): void => {
-      if (key in store) {
-        store[key] = undefined;
-      }
+      delete store[key];
     },
   },
   isBrowser(): boolean {
@@ -343,7 +345,7 @@ describe('ActiveCartService', () => {
         'oAuthRedirectCodeFlow'
       );
 
-      expect(storedOauthFlowKey).toBeUndefined();
+      expect(storedOauthFlowKey).toBeNull();
     });
   });
 
@@ -743,7 +745,6 @@ describe('ActiveCartService', () => {
     // context (spartacus⚿<baseSite>⚿<key>). `pendingGuestCartMerge` is the
     // (protected) key held by ActiveCartStatePersistenceService.
     const STORAGE_KEY = `spartacus⚿${BASE_SITE}⚿pendingGuestCartMerge`;
-
     beforeEach(() => {
       winRef?.localStorage?.removeItem(STORAGE_KEY);
       TestBed.resetTestingModule();
@@ -758,10 +759,13 @@ describe('ActiveCartService', () => {
             provide: SiteContextParamsService,
             useValue: { getValues: () => of([BASE_SITE]) },
           },
-          provideMockFeatureToggles({
-            authorizationCodeFlowByDefault: true,
-            mergeGuestCartOnCodeFlowLogin: true,
-          }),
+          {
+            provide: FeatureToggles,
+            useValue: {
+              authorizationCodeFlowByDefault: true,
+              mergeGuestCartOnCodeFlowLogin: true,
+            },
+          },
         ],
       });
       service = TestBed.inject(ActiveCartService);
@@ -831,7 +835,7 @@ describe('ActiveCartService', () => {
 
     describe('guestCartMerge', () => {
       it('should add the persisted entries and clear storage without deleting the guest cart', () => {
-        vi.spyOn(multiCartFacade, 'deleteCart');
+        vi.spyOn(multiCartFacade, 'deleteCart').mockImplementation(() => {});
         vi.spyOn(service as any, 'addEntriesGuestMerge').mockImplementation(
           () => {}
         );

feature-libs/cart/tsconfig.spec.json

@@ -40,7 +40,7 @@
       ]
     }
   },
-  "files": ["setup-test.ts"],
+  "files": ["../../testing/setup-vitest.ts"],
   "include": [
     "**/*.ts",
     "../../core-libs/core/src/**/*.ts",

feature-libs/organization/account-summary/components/details/document/account-summary-document.component.spec.ts

@@ -1,3 +1,4 @@
+import { vi } from 'vitest';
 import { Component, EventEmitter, Input, Output } from '@angular/core';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
@@ -33,7 +34,6 @@ import {
   DocumentStatus,
   FilterByOptions,
 } from '@spartacus/organization/account-summary/root';
-import createSpy = jasmine.createSpy;
 
 import { RouterModule } from '@angular/router';
 import { MockFeatureDirective } from 'core-libs/storefront/shared/test/mock-feature-directive';
@@ -89,7 +89,7 @@ class MockAccountSummaryFacade implements Partial<AccountSummaryFacade> {
 }
 
 class MockFileDownloadService {
-  download = createSpy('MockFileDownloadService.download Spy');
+  download = vi.fn();
 }
 
 class MockLanguageService {
@@ -169,8 +169,8 @@ describe('AccountSummaryDocumentComponent', () => {
 
   it('Should change page and sort', () => {
     // Spy functions to ensure new documents are being fetched
-    spyOn<any>(component, 'updateQueryParams').and.callThrough();
-    spyOn(accountSummaryFacade, 'getDocumentList').and.callThrough();
+    vi.spyOn<any, any>(component, 'updateQueryParams');
+    vi.spyOn(accountSummaryFacade, 'getDocumentList');
 
     // By default page will be 0
     expect(component._queryParams.page).toEqual(0);
@@ -212,8 +212,8 @@ describe('AccountSummaryDocumentComponent', () => {
 
   it('should change filters', () => {
     // Spy functions to ensure new documents are being fetched
-    spyOn<any>(component, 'updateQueryParams').and.callThrough();
-    spyOn(accountSummaryFacade, 'getDocumentList').and.callThrough();
+    vi.spyOn<any, any>(component, 'updateQueryParams');
+    vi.spyOn(accountSummaryFacade, 'getDocumentList');
 
     // Change the filters
     const status = DocumentStatus.CLOSED;
@@ -258,7 +258,7 @@ describe('AccountSummaryDocumentComponent', () => {
 
     // Call addNamesToSortModel with two sort options
     const sorts: Array<SortModel> = [{ code: 'abc' }, { code: 'def' }];
-    spyOn(translationService, 'translate').and.returnValue(of('test'));
+    vi.spyOn(translationService, 'translate').mockReturnValue(of('test'));
     component['addNamesToSortModel'](sorts);
 
     // Expect that translate was called twice
@@ -279,25 +279,25 @@ describe('AccountSummaryDocumentComponent', () => {
 
     const tableHeaders = tableElement.queryAll(By.css('th'));
     expect(tableHeaders?.length).toEqual(8);
-    expect(tableHeaders[0].properties.innerText).toEqual(
+    expect(tableHeaders[0].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.id'
     );
-    expect(tableHeaders[1].properties.innerText).toEqual(
+    expect(tableHeaders[1].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.type'
     );
-    expect(tableHeaders[2].properties.innerText).toEqual(
+    expect(tableHeaders[2].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.date'
     );
-    expect(tableHeaders[3].properties.innerText).toEqual(
+    expect(tableHeaders[3].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.dueDate'
     );
-    expect(tableHeaders[4].properties.innerText).toEqual(
+    expect(tableHeaders[4].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.originalAmount'
     );
-    expect(tableHeaders[5].properties.innerText).toEqual(
+    expect(tableHeaders[5].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.openAmount'
     );
-    expect(tableHeaders[6].properties.innerText).toEqual(
+    expect(tableHeaders[6].nativeElement.textContent?.trim()).toEqual(
       'orgAccountSummary.document.status'
     );
     expect(tableHeaders[7].children[0].attributes.title).toEqual(
@@ -323,31 +323,31 @@ describe('AccountSummaryDocumentComponent', () => {
 
       expect(tableCells?.length).toEqual(8);
 
-      expect(tableCells[0].nativeElement.innerText).toEqual(
+      expect(tableCells[0].nativeElement.textContent?.trim()).toEqual(
         mockAccountSummaryList.orgDocuments?.[rowNumber]?.id
       );
 
-      expect(tableCells[1].nativeElement.innerText).toEqual(
+      expect(tableCells[1].nativeElement.textContent?.trim()).toEqual(
         mockAccountSummaryList.orgDocuments?.[rowNumber]?.orgDocumentType?.name
       );
 
-      expect(isDate(tableCells[2].nativeElement.innerText)).toEqual(
+      expect(isDate(tableCells[2].nativeElement.textContent?.trim())).toEqual(
         !!mockAccountSummaryList.orgDocuments?.[rowNumber]?.createdAtDate
       );
 
-      expect(isDate(tableCells[3].nativeElement.innerText)).toEqual(
+      expect(isDate(tableCells[3].nativeElement.textContent?.trim())).toEqual(
         !!mockAccountSummaryList.orgDocuments?.[rowNumber]?.dueAtDate
       );
 
-      expect(tableCells[4].nativeElement.innerText).toEqual(
+      expect(tableCells[4].nativeElement.textContent?.trim()).toEqual(
         mockAccountSummaryList.orgDocuments?.[rowNumber]?.formattedAmount
       );
 
-      expect(tableCells[5].nativeElement.innerText).toEqual(
+      expect(tableCells[5].nativeElement.textContent?.trim()).toEqual(
         mockAccountSummaryList.orgDocuments?.[rowNumber]?.formattedOpenAmount
       );
 
-      expect(tableCells[6].nativeElement.innerText).toEqual(
+      expect(tableCells[6].nativeElement.textContent?.trim()).toEqual(
         `orgAccountSummary.statuses.${mockAccountSummaryList.orgDocuments?.[rowNumber]?.status}`
       );
 
@@ -363,12 +363,12 @@ describe('AccountSummaryDocumentComponent', () => {
         (doc) => doc?.attachments?.length && doc?.attachments?.length > 0
       ) || {};
 
-    spyOn(accountSummaryFacade, 'getDocumentAttachment').and.returnValue(
+    vi.spyOn(accountSummaryFacade, 'getDocumentAttachment').mockReturnValue(
       of(blob)
     );
     const fakeUrl =
       'blob:http://localhost:9877/50d43852-5f76-41e0-bb36-599d4b99af07';
-    spyOn(URL, 'createObjectURL').and.returnValue(fakeUrl);
+    vi.spyOn(URL, 'createObjectURL').mockReturnValue(fakeUrl);
 
     component.downloadAttachment(
       documentWithAttachment.id,

feature-libs/organization/account-summary/components/details/document/filter/account-summary-document-filter.component.spec.ts

@@ -1,3 +1,4 @@
+import { vi } from 'vitest';
 import { Component, CUSTOM_ELEMENTS_SCHEMA, Input } from '@angular/core';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { FormGroup, ReactiveFormsModule } from '@angular/forms';
@@ -149,21 +150,21 @@ describe('AccountSummaryDocumentFilterComponent', () => {
   });
 
   it('should test Filter By selector', () => {
-    const eventSpy = spyOn(component.filterListEvent, 'emit');
-    const resetSpy = spyOn<any>(component, 'resetForm').and.callThrough();
+    const eventSpy = vi.spyOn(component.filterListEvent, 'emit');
+    const resetSpy = vi.spyOn<any, any>(component, 'resetForm');
 
     let filterByValue: string | undefined;
     let startRange: string | undefined;
     let endRange: string | undefined;
 
     const pressSearch = () => {
-      eventSpy.calls.reset();
+      eventSpy.mockClear();
       const searchButton = fixture.debugElement.query(By.css('.btn-primary'));
       searchButton?.nativeElement.click();
       fixture.detectChanges();
     };
     const pressClear = () => {
-      eventSpy.calls.reset();
+      eventSpy.mockClear();
       const clearButton = fixture.debugElement.query(By.css('.clear-btn'));
       clearButton?.nativeElement.click();
       fixture.detectChanges();
@@ -428,7 +429,7 @@ describe('AccountSummaryDocumentFilterComponent', () => {
 
     // Press clear button and expect default values and search to be triggered
     expect(resetSpy).toHaveBeenCalledTimes(6);
-    resetSpy.calls.reset();
+    resetSpy.mockClear();
     pressClear();
     expect(resetSpy).toHaveBeenCalledWith(true);
     formItems = filterFormItems();