Merge pull request #4388 from serlo/refactor/migrate-to-vitest

refactor(tests): migrate to vitest, move all tests to editor

.github/workflows/web_checks.yaml

@@ -34,16 +34,6 @@ jobs:
       - run: yarn
       - run: yarn lint:tsc
 
-  test:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: 20
-      - run: yarn
-      - run: yarn test
-
   editor-test:
     runs-on: ubuntu-latest
     steps:
@@ -52,4 +42,4 @@ jobs:
         with:
           node-version: 20
       - run: yarn
-      - run: yarn editor:test
+      - run: yarn test

README.md

@@ -108,7 +108,7 @@ Creates a build of the frontend, shows summary of build artefacts and creates in
 yarn test
 ```
 
-Runs jest tests.
+Runs editor unit tests (vitest).
 
 ```
 yarn codegen

apps/web/package.json

@@ -24,8 +24,7 @@
     "lint:prettier": "yarn _prettier --check",
     "lint:tsc": "tsc --noEmit",
     "show-unused-exports": "ts-unused-exports tsconfig.json --excludePathsFromReport=src/pages/ --excludePathsFromReport=src/fetcher/graphql-types/operations.ts --allowUnusedTypes",
-    "start": "next start",
-    "test": "jest"
+    "start": "next start"
   },
   "resolutions": {
     "@serlo/katex-styles": "1.0.1"
@@ -100,7 +99,6 @@
     "@tailwindcss/typography": "^0.5.15",
     "@types/canvas-confetti": "^1.6.4",
     "@types/iframe-resizer": "^3.5.13",
-    "@types/jest": "^29.5.14",
     "@types/js-cookie": "^3.0.6",
     "@types/json-diff": "^1.0.3",
     "@types/katex": "^0.16.7",
@@ -123,7 +121,6 @@
     "eslint-plugin-react": "^7.37.2",
     "eslint-plugin-react-hooks": "^5.0.0",
     "graphql-tag": "2.12.6",
-    "jest": "^29.7.0",
     "micromark": "^4.0.1",
     "npm-run-all": "^4.1.5",
     "postcss-flexbugs-fixes": "^5.0.2",

package.json

@@ -35,7 +35,6 @@
     "editor:build": "yarn workspace @serlo/editor build",
     "editor:publish": "yarn workspace @serlo/editor run publish:package",
     "editor:publish-local": "yarn workspace @serlo/editor yalc:publish",
-    "editor:test": "yarn workspace @serlo/editor test",
     "format": "yarn workspace @serlo/frontend format",
     "format:eslint": "yarn workspace @serlo/frontend format:eslint",
     "format:prettier": "yarn workspaces foreach -Ap run format:prettier",
@@ -44,7 +43,7 @@
     "lint:prettier": "cross-env FORCE_COLOR=true yarn workspaces foreach -Aip run lint:prettier",
     "lint:tsc": "yarn workspace @serlo/frontend lint:tsc",
     "start": "yarn workspace @serlo/frontend start",
-    "test": "yarn workspace @serlo/frontend test"
+    "test": "yarn workspace @serlo/editor test"
   },
   "devDependencies": {
     "@eslint/eslintrc": "^3.1.0",

apps/web/__tests__/editor/input-exercise-plugin/expression-equal-evaluation.ts → packages/editor/__tests__/input-exercise-plugin/expression-equal-evaluation.test.ts

@@ -1,6 +1,7 @@
 import { getMatchingAnswer } from '@editor/plugins/input-exercise/helper/get-matching-answer'
 import { InputExerciseType } from '@editor/plugins/input-exercise/input-exercise-type'
 import { evaluate } from 'mathjs'
+import { expect, test, describe } from 'vitest'
 
 const answers = [
   {

apps/web/__tests__/editor/input-exercise-plugin/number-exact-evaluation.ts → packages/editor/__tests__/input-exercise-plugin/number-exact-evaluation.test.ts

@@ -1,6 +1,7 @@
 import { getMatchingAnswer } from '@editor/plugins/input-exercise/helper/get-matching-answer'
 import { InputExerciseType } from '@editor/plugins/input-exercise/input-exercise-type'
 import { evaluate } from 'mathjs'
+import { expect, test, describe } from 'vitest'
 
 const answers = [
   {

apps/web/__tests__/editor/input-exercise-plugin/string-match-evaluation.ts → packages/editor/__tests__/input-exercise-plugin/string-match-evaluation.test.ts

@@ -1,6 +1,7 @@
 import { getMatchingAnswer } from '@editor/plugins/input-exercise/helper/get-matching-answer'
 import { InputExerciseType } from '@editor/plugins/input-exercise/input-exercise-type'
 import { evaluate } from 'mathjs'
+import { expect, test, describe } from 'vitest'
 
 const answers = [
   {

packages/editor/__tests__/migration-state.ts → packages/editor/__tests__/migration-state.test.ts

@@ -1,4 +1,5 @@
 import { migrate } from '@editor/package/storage-format'
+import { expect, test } from 'vitest'
 
 test('A document not being in the storage format can be migrated', () => {
   const { stateChanged, migratedState } = migrate({ plugin: 'rows' }, 'unknown')

apps/web/__tests__/editor/text-static-is-empty.ts → packages/editor/__tests__/text-static-is-empty.test.ts

@@ -1,4 +1,5 @@
 import { isEmptyCustomText, isEmptyDescendant } from '@editor/package'
+import { expect, test, describe } from 'vitest'
 
 const text = { text: 'abc123' }
 const emptyText = { text: '' }

packages/editor/package.json

@@ -39,7 +39,7 @@
     "lint:eslint": "yarn _eslint",
     "lint:prettier": "yarn _prettier --check",
     "lint:tsc": "tsc --noEmit",
-    "test": "jest",
+    "test": "vitest run",
     "publish:package": "npm publish",
     "yalc:publish": "yarn build && yalc publish --push --sig"
   },
@@ -95,8 +95,6 @@
     "fp-ts": "^2.16.9",
     "io-ts": "^2.2.21",
     "is-hotkey": "0.2.0",
-    "jest": "^29.7.0",
-    "jest-environment-jsdom": "^29.7.0",
     "katex": "^0.16.11",
     "mathjs": "^13.2.2",
     "npm-run-all": "^4.1.5",
@@ -124,13 +122,13 @@
     "slate-react": "0.111.0",
     "tailwindcss": "^3.4.15",
     "ts-debounce": "^4.0.0",
-    "ts-jest": "^29.2.5",
     "ts-key-enum": "^2.0.13",
     "typescript": "^5.6.3",
     "uuid": "^11.0.3",
     "vite": "^5.4.11",
     "vite-plugin-dts": "4.3.0",
-    "vite-plugin-svgr": "^4.3.0"
+    "vite-plugin-svgr": "^4.3.0",
+    "vitest": "^2.1.8"
   },
   "packageManager": "[email protected]",
   "engines": {

packages/eslint-config/next.js

@@ -17,7 +17,6 @@ module.exports = {
   },
   env: {
     node: true,
-    jest: true, //!
   },
   plugins: ['@typescript-eslint', 'import', 'react-hooks'],
   settings: {
@@ -40,7 +39,6 @@ module.exports = {
     'node_modules/',
     'tailwind.config.js',
     'next.config.js',
-    'jest.config.js',
     'prettier.config.js',
   ],
   overrides: [