Explicitly import Vitest APIs instead of relying on globals

Kocal · Kocal · commit ca7ca82415a9 · 2025-07-20T12:48:08.000+02:00
diff --git a/assets/test/Backend/RequestBuilder.test.ts b/assets/test/Backend/RequestBuilder.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import RequestBuilder from '../../src/Backend/RequestBuilder';
 
 describe('buildRequest', () => {
diff --git a/assets/test/Component/index.test.ts b/assets/test/Component/index.test.ts
@@ -1,5 +1,6 @@
 import { waitFor } from '@testing-library/dom';
 import { Response } from 'node-fetch';
+import { describe, expect, it } from 'vitest';
 import type { BackendAction, BackendInterface } from '../../src/Backend/Backend';
 import BackendRequest from '../../src/Backend/BackendRequest';
 import type BackendResponse from '../../src/Backend/BackendResponse';
@@ -13,7 +14,7 @@ interface MockBackend extends BackendInterface {
 const makeTestComponent = (): { component: Component; backend: MockBackend } => {
     const backend: MockBackend = {
         actions: [],
-        makeRequest(data: any, actions: BackendAction[]): BackendRequest {
+        makeRequest(_data: any, actions: BackendAction[]): BackendRequest {
             this.actions = actions;
 
             return new BackendRequest(
diff --git a/assets/test/ComponentRegistry.test.ts b/assets/test/ComponentRegistry.test.ts
@@ -1,4 +1,5 @@
 import { Response } from 'node-fetch';
+import { beforeEach, describe, expect, it } from 'vitest';
 import type { BackendInterface } from '../src/Backend/Backend';
 import BackendRequest from '../src/Backend/BackendRequest';
 import Component from '../src/Component';
diff --git a/assets/test/Directive/directives_parser.test.ts b/assets/test/Directive/directives_parser.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { type Directive, parseDirectives } from '../../src/Directive/directives_parser';
 
 const assertDirectiveEquals = (actual: Directive, expected: any) => {
diff --git a/assets/test/Directive/get_model_binding.test.ts b/assets/test/Directive/get_model_binding.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { parseDirectives } from '../../src/Directive/directives_parser';
 import getModelBinding from '../../src/Directive/get_model_binding';
 
diff --git a/assets/test/Rendering/ChangingItemsTracker.test.ts b/assets/test/Rendering/ChangingItemsTracker.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import ChangingItemsTracker from '../../src/Rendering/ChangingItemsTracker';
 
 describe('ChangingItemsTracker', () => {
diff --git a/assets/test/Rendering/ElementChanges.test.ts b/assets/test/Rendering/ElementChanges.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import ElementChanges from '../../src/Rendering/ElementChanges';
 
 describe('ElementChanges', () => {
diff --git a/assets/test/Rendering/ExternalMutationTracker.test.ts b/assets/test/Rendering/ExternalMutationTracker.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { htmlToElement } from '../../src/dom_utils';
 import ExternalMutationTracker from '../../src/Rendering/ExternalMutationTracker';
 
diff --git a/assets/test/UnsyncedInputContainer.test.ts b/assets/test/UnsyncedInputContainer.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { UnsyncedInputContainer } from '../src/Component/UnsyncedInputsTracker';
 import { htmlToElement } from '../src/dom_utils';
 
diff --git a/assets/test/Util/getElementAsTagText.test.ts b/assets/test/Util/getElementAsTagText.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { htmlToElement } from '../../src/dom_utils';
 import getElementAsTagText from '../../src/Util/getElementAsTagText';
 
diff --git a/assets/test/ValueStore.test.ts b/assets/test/ValueStore.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import ValueStore from '../src/Component/ValueStore';
 
 describe('ValueStore', () => {
diff --git a/assets/test/controller/action.test.ts b/assets/test/controller/action.test.ts
@@ -9,6 +9,7 @@
 
 import { getByText, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController Action Tests', () => {
diff --git a/assets/test/controller/basic.test.ts b/assets/test/controller/basic.test.ts
@@ -7,6 +7,7 @@
  * file that was distributed with this source code.
  */
 
+import { afterEach, describe, expect, it } from 'vitest';
 import Component from '../../src/Component';
 import { findComponents } from '../../src/ComponentRegistry';
 import { htmlToElement } from '../../src/dom_utils';
diff --git a/assets/test/controller/child-model.test.ts b/assets/test/controller/child-model.test.ts
@@ -9,6 +9,7 @@
 
 import { getByTestId, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('Component parent -> child data-model binding tests', () => {
diff --git a/assets/test/controller/child.test.ts b/assets/test/controller/child.test.ts
@@ -10,6 +10,7 @@
 import { Controller } from '@hotwired/stimulus';
 import { getByTestId, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { findChildren } from '../../src/ComponentRegistry';
 import {
     createTest,
diff --git a/assets/test/controller/dispatch-event.test.ts b/assets/test/controller/dispatch-event.test.ts
@@ -7,6 +7,7 @@
  * file that was distributed with this source code.
  */
 
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController Event Dispatching Tests', () => {
diff --git a/assets/test/controller/emit.test.ts b/assets/test/controller/emit.test.ts
@@ -8,6 +8,7 @@
  */
 
 import { getByText, waitFor } from '@testing-library/dom';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController Emit Tests', () => {
diff --git a/assets/test/controller/error.test.ts b/assets/test/controller/error.test.ts
@@ -8,6 +8,7 @@
  */
 
 import { getByText, waitFor } from '@testing-library/dom';
+import { afterEach, describe, expect, it } from 'vitest';
 import type BackendResponse from '../../src/Backend/BackendResponse';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
@@ -108,7 +109,7 @@ describe('LiveController Error Handling', () => {
             .expectActionCalled('save');
 
         let isHookCalled = false;
-        test.component.on('response:error', (backendResponse: BackendResponse, controls) => {
+        test.component.on('response:error', (_backendResponse: BackendResponse, controls) => {
             isHookCalled = true;
             controls.displayError = false;
         });
diff --git a/assets/test/controller/loading.test.ts b/assets/test/controller/loading.test.ts
@@ -9,6 +9,7 @@
 
 import { getByTestId, getByText, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController data-loading Tests', () => {
@@ -21,7 +22,7 @@ describe('LiveController data-loading Tests', () => {
             { food: 'pizza' },
             (data: any) => `
             <div ${initComponent(data)}>
-                <span>I like: ${data.food}</span> 
+                <span>I like: ${data.food}</span>
                 <span data-loading="show" data-testid="loading-element">Loading...</span>
             </div>
         `
@@ -53,7 +54,7 @@ describe('LiveController data-loading Tests', () => {
             { food: 'pizza' },
             (data: any) => `
             <div ${initComponent(data)} data-loading="addClass(opacity-20)">
-                <span>I like: ${data.food}</span> 
+                <span>I like: ${data.food}</span>
                 <button data-action="live#$render">Re-Render</button>
             </div>
         `
@@ -84,7 +85,7 @@ describe('LiveController data-loading Tests', () => {
         const test = await createTest(
             {},
             (data: any) => `
-            <div ${initComponent(data)}> 
+            <div ${initComponent(data)}>
                 <span data-loading="action(save)|show" data-testid="loading-element">Loading...</span>
 
                 <button data-action="live#action" data-live-action-param="save">Save</button>
@@ -131,7 +132,7 @@ describe('LiveController data-loading Tests', () => {
         const test = await createTest(
             { comments: '', user: { email: '' } },
             (data: any) => `
-            <div ${initComponent(data)}> 
+            <div ${initComponent(data)}>
                 <textarea data-model="comments"></textarea>
                 <span data-loading="model(comments)|show" data-testid="comments-loading">Comments change loading...</span>
 
@@ -185,7 +186,7 @@ describe('LiveController data-loading Tests', () => {
         const test = await createTest(
             {},
             (data: any) => `
-            <div ${initComponent(data)}> 
+            <div ${initComponent(data)}>
                 <span data-loading="action(otherAction)|show" data-testid="loading-element">Loading...</span>
 
                 <button data-action="live#action" data-live-action-param="debounce(50)|save">Save</button>
@@ -215,7 +216,7 @@ describe('LiveController data-loading Tests', () => {
         const test = await createTest(
             {},
             (data: any) => `
-           <div ${initComponent(data)}> 
+           <div ${initComponent(data)}>
                <span data-loading="action(save)|delay(50)|show" data-testid="loading-element">Loading...</span>
 
                <button data-action="live#action" data-live-action-param="save">Save</button>
diff --git a/assets/test/controller/model.test.ts b/assets/test/controller/model.test.ts
@@ -9,6 +9,7 @@
 
 import { getByLabelText, getByTestId, getByText, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController data-model Tests', () => {
diff --git a/assets/test/controller/poll.test.ts b/assets/test/controller/poll.test.ts
@@ -9,6 +9,7 @@
 
 import { waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
 describe('LiveController polling Tests', () => {
diff --git a/assets/test/controller/query-binding.test.ts b/assets/test/controller/query-binding.test.ts
@@ -8,6 +8,7 @@
  */
 
 import { getByText, waitFor } from '@testing-library/dom';
+import { afterEach, describe, expect, it } from 'vitest';
 import { createTest, expectCurrentSearch, initComponent, setCurrentSearch, shutdownTests } from '../tools';
 
 describe('LiveController query string binding', () => {
diff --git a/assets/test/controller/render-with-external-changes.test.ts b/assets/test/controller/render-with-external-changes.test.ts
@@ -8,6 +8,7 @@
  */
 
 import { getByTestId } from '@testing-library/dom';
+import { afterEach, describe, expect, it } from 'vitest';
 import { htmlToElement } from '../../src/dom_utils';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
diff --git a/assets/test/controller/render.test.ts b/assets/test/controller/render.test.ts
@@ -9,6 +9,7 @@
 
 import { getByTestId, getByText, waitFor } from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
+import { afterEach, describe, expect, it } from 'vitest';
 import { htmlToElement } from '../../src/dom_utils';
 import { createTest, initComponent, shutdownTests } from '../tools';
 
@@ -51,7 +52,7 @@ describe('LiveController rendering Tests', () => {
                     We want the first to finish, to see if it overwrites our value
                 -->
                 <textarea data-model="norender|comment">${data.comment}</textarea>
-                
+
                 Title: "${data.title}"
                 Comment: "${data.comment}"
 
@@ -213,7 +214,7 @@ describe('LiveController rendering Tests', () => {
             (data: any) => `
             <div ${initComponent(data)}>
                 <div data-live-ignore>Inside Ignore Name: <span>${data.firstName}</span></div>
-                
+
                 Outside Ignore Name: ${data.firstName}
 
                 <button data-action="live#$render">Reload</button>
@@ -249,7 +250,7 @@ describe('LiveController rendering Tests', () => {
                 <div id="${data.containerId}">
                     <div data-live-ignore>Inside Ignore Name: <span>${data.firstName}</span></div>
                 </div>
-                
+
                 Outside Ignore Name: ${data.firstName}
 
                 <button data-action="live#$render">Reload</button>
@@ -491,8 +492,8 @@ describe('LiveController rendering Tests', () => {
             // correctly delayed, the "first render finish" and "second render
             // start" times could be the same, because no time has passed.
             const sleep = (milliseconds: number) => {
-                const startTime = new Date().getTime();
-                while (new Date().getTime() < startTime + milliseconds);
+                const startTime = Date.now();
+                while (Date.now() < startTime + milliseconds);
             };
             sleep(10);
         });
@@ -585,7 +586,7 @@ describe('LiveController rendering Tests', () => {
                         <option value="2">Two</option>
                         <option value="3">Three</option>
                     </select>
-                    
+
                     <select id="select_option_2">
                         <option value="">Choose option 2</option>
                         <option value="1_1">One - One</option>
@@ -610,7 +611,7 @@ describe('LiveController rendering Tests', () => {
                             <option value="2" selected>Two</option>
                             <option value="3">Three</option>
                         </select>
-                        
+
                         <select id="select_option_2">
                             <option value="">Choose option 2</option>
                             <option value="2_1">Two - One</option>
diff --git a/assets/test/dom_utils.test.ts b/assets/test/dom_utils.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import Backend from '../src/Backend/Backend';
 import Component from '../src/Component';
 import ValueStore from '../src/Component/ValueStore';
diff --git a/assets/test/normalize_attributes_for_comparison.test.ts b/assets/test/normalize_attributes_for_comparison.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { htmlToElement } from '../src/dom_utils';
 import { normalizeAttributesForComparison } from '../src/normalize_attributes_for_comparison';
 
diff --git a/assets/test/string_utils.test.ts b/assets/test/string_utils.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { combineSpacedArray, normalizeModelName } from '../src/string_utils';
 
 describe('combinedSpacedArray', () => {
diff --git a/assets/test/tools.ts b/assets/test/tools.ts
@@ -1,6 +1,7 @@
 import { Application } from '@hotwired/stimulus';
 import { waitFor } from '@testing-library/dom';
 import { Response } from 'node-fetch';
+import { expect } from 'vitest';
 import type { BackendAction, BackendInterface, ChildrenFingerprints } from '../src/Backend/Backend';
 import BackendRequest from '../src/Backend/BackendRequest';
 import Component from '../src/Component';
diff --git a/assets/test/url_utils.test.ts b/assets/test/url_utils.test.ts
@@ -7,6 +7,7 @@
  * file that was distributed with this source code.
  */
 
+import { afterEach, beforeAll, beforeEach, describe, expect, it } from 'vitest';
 import { HistoryStrategy, UrlUtils } from '../src/url_utils';
 
 describe('url_utils', () => {

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import RequestBuilder from '../../src/Backend/RequestBuilder';`
`2`	`3`
`3`	`4`	`describe('buildRequest', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`import { Response } from 'node-fetch';`
	`2`	`+import { beforeEach, describe, expect, it } from 'vitest';`
`2`	`3`	`import type { BackendInterface } from '../src/Backend/Backend';`
`3`	`4`	`import BackendRequest from '../src/Backend/BackendRequest';`
`4`	`5`	`import Component from '../src/Component';`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import { type Directive, parseDirectives } from '../../src/Directive/directives_parser';`
`2`	`3`
`3`	`4`	`const assertDirectiveEquals = (actual: Directive, expected: any) => {`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import { parseDirectives } from '../../src/Directive/directives_parser';`
`2`	`3`	`import getModelBinding from '../../src/Directive/get_model_binding';`
`3`	`4`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import ChangingItemsTracker from '../../src/Rendering/ChangingItemsTracker';`
`2`	`3`
`3`	`4`	`describe('ChangingItemsTracker', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import ElementChanges from '../../src/Rendering/ElementChanges';`
`2`	`3`
`3`	`4`	`describe('ElementChanges', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import { htmlToElement } from '../../src/dom_utils';`
`2`	`3`	`import ExternalMutationTracker from '../../src/Rendering/ExternalMutationTracker';`
`3`	`4`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from 'vitest';`
`1`	`2`	`import { UnsyncedInputContainer } from '../src/Component/UnsyncedInputsTracker';`
`2`	`3`	`import { htmlToElement } from '../src/dom_utils';`
`3`	`4`