build: upgraded from Vitest v3 to v4

Author: thorsten

CHANGELOG.md

@@ -40,7 +40,7 @@ This is a log of major user-visible changes in each phpMyFAQ release.
 - migrated from WYSIWYG editor from TinyMCE to Jodit Editor (Thorsten)
 - migrated from JavaScript to TypeScript (Thorsten)
 - migrated from Webpack to Vite v7 (Thorsten)
-- migrated from Jest to Vitest v3 (Thorsten)
+- migrated from Jest to Vitest v4 (Thorsten)
 
 ### phpMyFAQ v4.0.14 - 2025-11-15
 

SECURITY.md

@@ -45,7 +45,7 @@ We’ll provide periodic updates while triage and remediation are in progress.
 ## Coordinated Disclosure
 
 - Please do not publish or share details publicly until a fix is available and users have had a reasonable time to update.
-- We coordinate on a release date with you; our default disclosure window is up to 90 days from report, adjusted by severity and complexity.
+- We coordinate on a release date with you; our default disclosure window is up to 90 days from a report, adjusted by severity and complexity.
 - We will credit reporters in release notes and/or advisories unless you prefer to remain anonymous.
 
 ## Scope Guidance

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thorsten/phpmyfaq",
-  "version": "4.1.0-RC",
+  "version": "4.1.0-beta.2",
   "description": "phpMyFAQ",
   "repository": "git://github.com/thorsten/phpMyFAQ.git",
   "author": "Thorsten Rinne",
@@ -57,8 +57,8 @@
     "@types/masonry-layout": "^4.2.8",
     "@types/node": "^24.10.3",
     "@types/sortablejs": "^1.15.9",
-    "@vitest/coverage-istanbul": "^3.2.4",
-    "@vitest/coverage-v8": "^3.2.4",
+    "@vitest/coverage-istanbul": "4.0.15",
+    "@vitest/coverage-v8": "4.0.15",
     "autoprefixer": "^10.4.22",
     "husky": "^9.1.7",
     "jsdom": "^27.3.0",
@@ -73,7 +73,7 @@
     "vite-plugin-html": "^3.2.2",
     "vite-plugin-minify": "^2.1.0",
     "vite-plugin-static-copy": "^3.1.4",
-    "vitest": "^3.2.4",
+    "vitest": "^4.0.15",
     "vitest-fetch-mock": "^0.4.5"
   },
   "husky": {

phpmyfaq/admin/assets/src/configuration/instance.test.ts

@@ -19,11 +19,11 @@ import * as api from '../api/instance';
 
 // Mock Bootstrap Modal globally
 vi.mock('bootstrap', () => ({
-  Modal: vi.fn().mockImplementation((element: HTMLElement) => ({
-    element,
-    hide: vi.fn(),
-    show: vi.fn(),
-  })),
+  Modal: vi.fn(function (this: any, element: HTMLElement) {
+    this.element = element;
+    this.hide = vi.fn();
+    this.show = vi.fn();
+  }),
 }));
 
 describe('handleInstances - Initialization', () => {

phpmyfaq/admin/assets/src/utils/session.test.ts

@@ -2,10 +2,10 @@ import { describe, it, expect, vi, afterEach } from 'vitest';
 import { handleSessionTimeout } from './session';
 
 vi.mock('bootstrap', () => ({
-  Modal: vi.fn().mockImplementation(() => ({
-    show: vi.fn(),
-    hide: vi.fn(),
-  })),
+  Modal: class {
+    show = vi.fn();
+    hide = vi.fn();
+  },
 }));
 
 describe('Session Utils', () => {
@@ -26,7 +26,9 @@ describe('Session Utils', () => {
         disconnect: vi.fn(),
         takeRecords: vi.fn(),
       };
-      global.MutationObserver = vi.fn(() => mockObserver) as unknown as typeof MutationObserver;
+      global.MutationObserver = function () {
+        return mockObserver;
+      } as unknown as typeof MutationObserver;
 
       const mockReload = vi.fn();
       Object.defineProperty(global, 'location', {

phpmyfaq/assets/src/search/category.test.ts

@@ -4,24 +4,18 @@ import { TranslationService } from '../utils';
 import { handleCategorySelection } from './category';
 
 // Mock Choices.js
-vi.mock('choices.js', () => {
-  return {
-    default: vi.fn(),
-  };
-});
+vi.mock('choices.js', () => ({
+  default: vi.fn(),
+}));
 
 // Mock TranslationService
-vi.mock('../utils', () => {
-  const mockTranslationService = {
-    translations: new Map(),
-    loadTranslations: vi.fn().mockResolvedValue(undefined),
-    translate: vi.fn().mockImplementation((key: string) => `translated_${key}`),
-  };
-
-  return {
-    TranslationService: vi.fn(() => mockTranslationService),
-  };
-});
+vi.mock('../utils', () => ({
+  TranslationService: vi.fn(function () {
+    this.translations = new Map();
+    this.loadTranslations = vi.fn().mockResolvedValue(undefined);
+    this.translate = vi.fn().mockImplementation((key: string) => `translated_${key}`);
+  }),
+}));
 
 describe('handleCategorySelection', () => {
   const mockChoices = vi.mocked(Choices);
@@ -72,7 +66,11 @@ describe('handleCategorySelection', () => {
       loadTranslations: vi.fn().mockResolvedValue(undefined),
       translate: vi.fn().mockReturnValue('translated'),
     };
-    mockTranslationService.mockReturnValue(mockInstance);
+    mockTranslationService.mockImplementation(function () {
+      this.translations = mockInstance.translations;
+      this.loadTranslations = mockInstance.loadTranslations;
+      this.translate = mockInstance.translate;
+    });
 
     await handleCategorySelection();