Remove vitest globals (#21505)

Explicitly import them instead which is cleaner and enables better
editor integration.

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>

Author: 3 people

.eslintrc.yaml

@@ -35,9 +35,6 @@ overrides:
     rules:
       import/no-unresolved: [0]
       import/no-extraneous-dependencies: [0]
-  - files: ["*.test.js"]
-    env:
-      jest: true
   - files: ["*.config.js"]
     rules:
       import/no-unused-modules: [0]

vitest.config.js

@@ -24,7 +24,6 @@ export default defineConfig({
     open: false,
     allowOnly: true,
     passWithNoTests: true,
-    globals: true,
     watch: false,
   },
   plugins: [

web_src/js/features/repo-findfile.test.js

@@ -1,3 +1,4 @@
+import {describe, expect, test} from 'vitest';
 import {strSubMatch, calcMatchedWeight, filterRepoFilesWeighted} from './repo-findfile.js';
 
 describe('Repo Find Files', () => {

web_src/js/svg.test.js

@@ -1,3 +1,4 @@
+import {expect, test} from 'vitest';
 import {svg} from './svg.js';
 
 test('svg', () => {

web_src/js/utils.test.js

@@ -1,3 +1,4 @@
+import {expect, test} from 'vitest';
 import {
   basename, extname, isObject, uniq, stripTags, joinPaths, parseIssueHref,
   prettyNumber, parseUrl,