test(core): support vitest

mumenthalers · mumenthalers · commit 42fa7b29138b · 2026-01-22T17:06:35.000+01:00
diff --git a/angular.json b/angular.json
@@ -12,12 +12,14 @@
         "build": {
           "builder": "@angular/build:ng-packagr",
           "options": {
-            "tsConfig": "libs/core/tsconfig.lib.json",
             "project": "libs/core/ng-package.json"
           },
           "configurations": {
             "production": {
               "tsConfig": "libs/core/tsconfig.lib.prod.json"
+            },
+            "development": {
+              "tsConfig": "libs/core/tsconfig.lib.json"
             }
           },
           "defaultConfiguration": "production"
@@ -26,7 +28,7 @@
           "builder": "@angular/build:unit-test",
           "options": {
             "tsConfig": "tsconfig.spec.json",
-            "polyfills": ["zone.js", "zone.js/testing"]
+            "runner": "vitest"
           }
         }
       }
@@ -40,12 +42,14 @@
         "build": {
           "builder": "@angular/build:ng-packagr",
           "options": {
-            "project": "libs/components/ng-package.json",
-            "tsConfig": "libs/components/tsconfig.lib.json"
+            "project": "libs/components/ng-package.json"
           },
           "configurations": {
             "production": {
               "tsConfig": "libs/components/tsconfig.lib.prod.json"
+            },
+            "development": {
+              "tsConfig": "libs/components/tsconfig.lib.json"
             }
           },
           "defaultConfiguration": "production"
@@ -54,7 +58,7 @@
           "builder": "@angular/build:unit-test",
           "options": {
             "tsConfig": "tsconfig.spec.json",
-            "polyfills": ["zone.js", "zone.js/testing"]
+            "runner": "vitest"
           }
         }
       }
diff --git a/libs/components/eslint.config.mjs b/libs/components/eslint.config.mjs
@@ -28,4 +28,12 @@ export default defineScAngularConfig(
       'no-undef': 'off',
     },
   },
+  {
+    files: ['**/*.spec.ts', '**/*.test.ts'],
+    rules: {
+      '@typescript-eslint/no-unsafe-member-access': 'off',
+      '@typescript-eslint/no-unsafe-call': 'off',
+      '@typescript-eslint/no-unsafe-return': 'off',
+    }
+  }
 )
diff --git a/libs/core/eslint.config.mjs b/libs/core/eslint.config.mjs
@@ -36,4 +36,12 @@ export default defineScAngularConfig(
       'no-undef': 'off',
     },
   },
+  {
+    files: ['**/*.spec.ts', '**/*.test.ts'],
+    rules: {
+      '@typescript-eslint/no-unsafe-member-access': 'off',
+      '@typescript-eslint/no-unsafe-call': 'off',
+      '@typescript-eslint/no-unsafe-return': 'off',
+    }
+  }
 )
diff --git a/libs/core/src/lib/http/http-date.interceptor.spec.ts b/libs/core/src/lib/http/http-date.interceptor.spec.ts
@@ -9,7 +9,7 @@ import { HttpTestingController, provideHttpClientTesting } from '@angular/common
 import { ClassProvider } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
 import { firstValueFrom } from 'rxjs'
-import { describe, it, expect, beforeEach, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { HttpDateInterceptor } from './http-date.interceptor'
 
diff --git a/libs/core/src/lib/local-storage/local-storage.service.spec.ts b/libs/core/src/lib/local-storage/local-storage.service.spec.ts
@@ -1,7 +1,7 @@
 import { ClassProvider, ValueProvider } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
 import { firstValueFrom, scan, shareReplay } from 'rxjs'
-import { describe, it, expect, beforeEach, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { MockStorage } from '../../../test'
 import { WindowRef } from '../window/window-ref.service'
diff --git a/libs/core/src/lib/logger/helper/leading-zero.function.spec.ts b/libs/core/src/lib/logger/helper/leading-zero.function.spec.ts
@@ -1,4 +1,5 @@
-import { describe, it, expect } from 'vitest'
+import { describe, expect,it } from 'vitest'
+
 import { leadingZero } from './leading-zero.function'
 
 describe('leadingZero', () => {
diff --git a/libs/core/src/lib/logger/logger.service.spec.ts b/libs/core/src/lib/logger/logger.service.spec.ts
@@ -2,8 +2,9 @@ import { Component, Directive, inject, Injectable, InjectionToken, PLATFORM_ID }
 import { TestBed } from '@angular/core/testing'
 import { By } from '@angular/platform-browser'
 import { Logger, LogLevel, LogTransport } from '@shiftcode/logger'
-import { LoggerService } from '@shiftcode/ngx-core'
-import { describe, it, expect, beforeEach, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+
+import { LoggerService } from './logger.service'
 
 interface MockLogTransportConfig {
   logLevel: LogLevel
@@ -115,3 +116,4 @@ describe('MyDirective', () => {
     expect(loggerSpy).toHaveBeenCalledWith('doing something')
   })
 })
+
diff --git a/libs/core/src/lib/logger/remote/remote-log.service.spec.ts b/libs/core/src/lib/logger/remote/remote-log.service.spec.ts
@@ -3,7 +3,7 @@ import { HttpTestingController, provideHttpClientTesting } from '@angular/common
 import { FactoryProvider, ValueProvider } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
 import { createJsonLogObjectData, LogLevel } from '@shiftcode/logger'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { LOG_REQUEST_INFO_FN } from '../log-request-info-fn.token'
 import { LogRequestInfoFn } from '../log-request-info-fn.type'
diff --git a/libs/core/src/lib/origin/ensure-origin.interceptor.spec.ts b/libs/core/src/lib/origin/ensure-origin.interceptor.spec.ts
@@ -1,7 +1,7 @@
 import { HttpClient, HttpRequest, provideHttpClient, withInterceptors } from '@angular/common/http'
 import { HttpTestingController, provideHttpClientTesting } from '@angular/common/http/testing'
 import { TestBed } from '@angular/core/testing'
-import { describe, it, expect, beforeEach, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { ensureOriginInterceptor } from './ensure-origin.interceptor'
 import { ORIGIN } from './origin.token'
diff --git a/libs/core/src/lib/origin/provide-origin-from-env.function.spec.ts b/libs/core/src/lib/origin/provide-origin-from-env.function.spec.ts
@@ -1,6 +1,6 @@
 import { PLATFORM_ID } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
-import { describe, it, expect, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 
 import { ORIGIN } from './origin.token'
 import { provideOriginFromEnv } from './provide-origin-from-env.function'
diff --git a/libs/core/src/lib/script-loader/script-loader.service.spec.ts b/libs/core/src/lib/script-loader/script-loader.service.spec.ts
@@ -2,7 +2,7 @@ import { CommonModule } from '@angular/common'
 import { DOCUMENT, PLATFORM_ID } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
 import { LogLevel } from '@shiftcode/logger'
-import { describe, it, expect, beforeEach } from 'vitest'
+import { beforeEach,describe, expect, it } from 'vitest'
 
 import { LoggerService } from '../logger/logger.service'
 import { MockLoggerService } from '../logger/mock-logger.service'
diff --git a/libs/core/src/lib/static-utils/is-input-element.function.spec.ts b/libs/core/src/lib/static-utils/is-input-element.function.spec.ts
@@ -1,6 +1,6 @@
 import { DOCUMENT } from '@angular/core'
 import { TestBed } from '@angular/core/testing'
-import { describe, it, expect, beforeEach } from 'vitest'
+import { beforeEach,describe, expect, it } from 'vitest'
 
 import { isInputElement } from './is-input-element.function'
 
diff --git a/libs/core/src/lib/static-utils/jwt-helper.spec.ts b/libs/core/src/lib/static-utils/jwt-helper.spec.ts
@@ -1,102 +1,110 @@
-import { describe, it, expect } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
+
+import { mockConsole, restoreConsole } from '../../../test/mock-console'
 import { JwtHelper } from './jwt-helper'
 
 describe('JwtHelper', () => {
   describe('isToken', () => {
-  it('returns false when not a string', () => {
+    beforeEach(() => mockConsole())
+    afterEach(() => restoreConsole())
+
+    it('returns false when not a string', () => {
       expect(JwtHelper.isToken(null)).toBe(false)
       expect(JwtHelper.isToken(undefined)).toBe(false)
       expect(JwtHelper.isToken(7)).toBe(false)
       expect(JwtHelper.isToken({})).toBe(false)
       expect(JwtHelper.isToken([])).toBe(false)
       expect(JwtHelper.isToken(new Date())).toBe(false)
     })
-  it('returns false when not 3 parts', () => {
+    it('returns false when not 3 parts', () => {
       expect(JwtHelper.isToken('abc:def')).toBe(false)
       expect(JwtHelper.isToken('abc:def:ghi:jkl')).toBe(false)
     })
-  it('returns false when not decodable', () => {
+    it('returns false when not decodable', () => {
       expect(JwtHelper.isToken('abc:def:ghi')).toBe(false)
     })
-  it('returns false when decoded & parsed part not object', () => {
+    it('returns false when decoded & parsed part not object', () => {
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode('just a string')}.part3`
       expect(JwtHelper.isToken(tokenString)).toBe(false)
     })
-  it('returns false when decoded & parsed part is null', () => {
+    it('returns false when decoded & parsed part is null', () => {
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode(JSON.stringify(null))}.part3`
       expect(JwtHelper.isToken(tokenString)).toBe(false)
     })
-  it('returns true when actual token', () => {
+    it('returns true when actual token', () => {
       const jwtToken = { iss: 'https://www.shiftcode.ch' }
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode(JSON.stringify(jwtToken))}.part3`
       expect(JwtHelper.isToken(tokenString)).toBe(true)
     })
   })
 
   describe('decodeToken', () => {
-  it('throws when not typeof string', () => {
+    beforeEach(() => mockConsole())
+    afterEach(() => restoreConsole())
+
+    it('throws when not typeof string', () => {
       expect(() => JwtHelper.decodeToken(null)).toThrow(Error)
       expect(() => JwtHelper.decodeToken(undefined)).toThrow(Error)
       expect(() => JwtHelper.decodeToken(7)).toThrow(Error)
       expect(() => JwtHelper.decodeToken({})).toThrow(Error)
       expect(() => JwtHelper.decodeToken([])).toThrow(Error)
       expect(() => JwtHelper.decodeToken(new Date())).toThrow(Error)
     })
-  it('throws when not 3 parts', () => {
+    it('throws when not 3 parts', () => {
       expect(() => JwtHelper.decodeToken('abc.def')).toThrow(Error)
       expect(() => JwtHelper.decodeToken('abc.def.ghi.jkl')).toThrow(Error)
     })
-  it('throws when not decodable', () => {
+    it('throws when not decodable', () => {
       expect(() => JwtHelper.decodeToken('abc:def:ghi')).toThrow(Error)
     })
-  it('throws when decoded & parsed part not an object', () => {
+    it('throws when decoded & parsed part not an object', () => {
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode('just a string')}.part3`
       expect(() => JwtHelper.decodeToken(tokenString)).toThrow(Error)
     })
-  it('throws when decoded & parsed part is null', () => {
+    it('throws when decoded & parsed part is null', () => {
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode(JSON.stringify(null))}.part3`
       expect(() => JwtHelper.decodeToken(tokenString)).toThrow()
     })
-  it('returns decoded token when ok', () => {
+    it('returns decoded token when ok', () => {
       const jwtToken = { iss: 'https://www.shiftcode.ch' }
       const tokenString = `part1.${JwtHelper.b64EncodeUnicode(JSON.stringify(jwtToken))}.part3`
       expect(JwtHelper.decodeToken(tokenString)).toEqual(jwtToken)
     })
   })
 
   describe('getTokenExpirationDate', () => {
-  it('throws when invalid token', () => {
+    it('throws when invalid token', () => {
       const tokenString = `a.b.c`
       expect(() => JwtHelper.getTokenExpirationDate(tokenString)).toThrow(Error)
     })
-  it('returns null when exp not set', () => {
+    it('returns null when exp not set', () => {
       const token = { iss: 'https://www.shiftcode.ch' }
       const tokenString = `a.${JwtHelper.b64EncodeUnicode(JSON.stringify(token))}.c`
       expect(JwtHelper.getTokenExpirationDate(tokenString)).toBe(null)
     })
-  it('returns correct date', () => {
+    it('returns correct date', () => {
       const token = { exp: Math.floor(Date.now() / 1_000) }
       const tokenString = `a.${JwtHelper.b64EncodeUnicode(JSON.stringify(token))}.c`
       expect(JwtHelper.getTokenExpirationDate(tokenString)).toEqual(new Date(token.exp * 1000))
     })
   })
 
   describe('isTokenExpired', () => {
-  it('throws when invalid token', () => {
+    it('throws when invalid token', () => {
       const tokenString = `a.b.c`
       expect(() => JwtHelper.isTokenExpired(tokenString)).toThrow(Error)
     })
-  it('returns false when exp not set', () => {
+    it('returns false when exp not set', () => {
       const token = { iss: 'https://www.shiftcode.ch' }
       const tokenString = `a.${JwtHelper.b64EncodeUnicode(JSON.stringify(token))}.c`
       expect(JwtHelper.isTokenExpired(tokenString)).toBe(false)
     })
-  it('returns true when expired', () => {
+    it('returns true when expired', () => {
       const token = { exp: Math.floor(Date.now() / 1_000) - 10 }
       const tokenString = `a.${JwtHelper.b64EncodeUnicode(JSON.stringify(token))}.c`
       expect(JwtHelper.isTokenExpired(tokenString)).toBe(true)
     })
-  it('works with offsetSeconds', () => {
+    it('works with offsetSeconds', () => {
       const token = { exp: Math.floor(Date.now() / 1_000) + 30 }
       const tokenString = `a.${JwtHelper.b64EncodeUnicode(JSON.stringify(token))}.c`
       expect(JwtHelper.isTokenExpired(tokenString, 20)).toBe(false)
diff --git a/libs/core/src/lib/static-utils/resolve-fn-to-can-activate-fn.function.spec.ts b/libs/core/src/lib/static-utils/resolve-fn-to-can-activate-fn.function.spec.ts
@@ -1,5 +1,5 @@
 import { ResolveFn } from '@angular/router'
-import { describe, it, expect } from 'vitest'
+import { describe, expect,it } from 'vitest'
 
 import { resolveFnToCanActivateFn } from './resolve-fn-to-can-activate-fn.function'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/filter-if-falsy.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/filter-if-falsy.operator.spec.ts
@@ -1,5 +1,5 @@
 import { Subject } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { filterIfFalsy } from './filter-if-falsy.operator'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/filter-if-instance-of.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/filter-if-instance-of.operator.spec.ts
@@ -1,5 +1,5 @@
 import { of, Subject, tap } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { filterIfInstanceOf } from './filter-if-instance-of.operator'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/filter-if-truthy.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/filter-if-truthy.operator.spec.ts
@@ -1,5 +1,5 @@
-import { from, of, scan, tap } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { from, lastValueFrom, of, tap } from 'rxjs'
+import { describe, expect, it, vi } from 'vitest'
 
 import { filterIfTruthy } from './filter-if-truthy.operator'
 
@@ -8,31 +8,24 @@ describe('filterIfTruthy operator', () => {
   const emptyArr: any[] = []
   const emptyObj: any = {}
 
-  it('when truthy', (done) => {
+  it('when truthy', async () => {
     const truthyValues = [true, 1, 'ok', obj, emptyArr, emptyObj]
 
-    from(truthyValues)
-      .pipe(
-        filterIfTruthy(),
-        scan((u, i) => [...u, i], <any[]>[]),
-      )
-      .subscribe({
-        next: (values) => {
-          expect(values).toEqual(truthyValues)
-        },
-        complete: done,
-      })
+    const next = vi.fn()
+
+    await lastValueFrom(from(truthyValues).pipe(filterIfTruthy(), tap(next)))
+
+    expect(next).toHaveBeenCalledTimes(truthyValues.length)
   })
-  it('when false', (done) => {
+
+  it('when false', async () => {
     const falsyValues = [null, undefined, false, 0, '', NaN]
 
     const next = vi.fn()
-    const complete = () => {
-      expect(next).not.toHaveBeenCalled()
-      done()
-    }
 
-    from(falsyValues).pipe(filterIfTruthy()).subscribe({ next, complete })
+    await lastValueFrom(from(falsyValues).pipe(filterIfTruthy(), tap(next)), { defaultValue: undefined })
+
+    expect(next).not.toHaveBeenCalled()
   })
 
   it('restricts types', () => {
diff --git a/libs/core/src/lib/static-utils/rxjs/on-destroy.observable.spec.ts b/libs/core/src/lib/static-utils/rxjs/on-destroy.observable.spec.ts
@@ -1,6 +1,6 @@
 import { DestroyRef, EnvironmentInjector, Injector, runInInjectionContext } from '@angular/core'
 import { BehaviorSubject, takeUntil } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { onDestroy } from './on-destroy.observable'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/select.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/select.operator.spec.ts
@@ -1,5 +1,5 @@
 import { of, Subject, tap } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { select } from './select.operator'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/setup.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/setup.operator.spec.ts
@@ -1,5 +1,5 @@
 import { Subject } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { setup } from './setup.operator'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/tap-last.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/tap-last.operator.spec.ts
@@ -1,5 +1,5 @@
 import { Subject } from 'rxjs'
-import { describe, it, expect, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { tapLast } from './tap-last.operator'
 
diff --git a/libs/core/src/lib/static-utils/rxjs/tap-previous.operator.spec.ts b/libs/core/src/lib/static-utils/rxjs/tap-previous.operator.spec.ts
diff --git a/libs/core/src/lib/static-utils/rxjs/wrap-into-observable.spec.ts b/libs/core/src/lib/static-utils/rxjs/wrap-into-observable.spec.ts
diff --git a/libs/core/src/lib/static-utils/to-promise.function.spec.ts b/libs/core/src/lib/static-utils/to-promise.function.spec.ts
diff --git a/libs/core/src/lib/ui-events/ui-event.service.spec.ts b/libs/core/src/lib/ui-events/ui-event.service.spec.ts
diff --git a/libs/core/test/mock-console.ts b/libs/core/test/mock-console.ts
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
