Fix tests

.eslintrc.yml

@@ -1,11 +1,12 @@
 env:
   browser: true
   es6: true
-  jest/globals: true
+  vitest-globals/env: true
 extends:
   - 'eslint:recommended'
   - 'plugin:react/recommended'
   - 'plugin:react/jsx-runtime'
+  - 'plugin:vitest-globals/recommended'
 globals:
   global: false
   module: false
@@ -265,15 +266,15 @@ rules:
     - error
     - never
   react/jsx-uses-react: error
-  jest/no-disabled-tests: warn
-  jest/no-focused-tests: error
-  jest/no-identical-title: error
-  jest/prefer-to-have-length: warn
-  jest/valid-expect: error
+  vitest/no-disabled-tests: warn
+  vitest/no-focused-tests: error
+  vitest/no-identical-title: error
+  vitest/prefer-to-have-length: warn
+  vitest/valid-expect: error
 plugins:
   - react
+  - vitest
   - simple-import-sort
-  - jest
 settings:
   react:
     version: detect

.github/workflows/ci.yml

@@ -64,7 +64,7 @@ jobs:
         with:
           name: Frontend
           token: ${{ secrets.CODECOV_TOKEN }}
-          files: .jest-coverage/clover.xml
+          files: coverage/clover.xml
           flags: frontend
           fail_ci_if_error: true
           verbose: true

Makefile

@@ -46,7 +46,7 @@ node_modules/.timestamp: package.json
 
 oereb_client/static/build/.timestamp: node_modules/.timestamp vite.config.js $(SRC_JS) $(SRC_SCSS)
 	rm -rf oereb_client/static/build/
-	NODE_ENV=production ./node_modules/.bin/vite build
+	./node_modules/.bin/vite build
 	touch $@
 
 
@@ -100,7 +100,7 @@ lint-js-fix: node_modules/.timestamp .eslintrc.yml $(SRC_JS) $(TEST_JS)
 
 .PHONY: test-js
 test-js: $(SRC_JS) $(TEST_JS)
-	NODE_ENV=test ./node_modules/.bin/vitest --coverage --run
+	./node_modules/.bin/vitest --coverage --run
 
 .PHONY: check-js
 check-js: git-attributes lint-js test-js
@@ -123,7 +123,7 @@ serve: build app.ini
 
 .PHONY: serve-dev
 serve-dev: build app.ini node_modules/.timestamp .venv/.requirements.timestamp
-	NODE_ENV=production ./node_modules/.bin/vite build --watch &
+	./node_modules/.bin/vite build --watch &
 	.venv/bin/pserve app.ini --reload
 
 .PHONY: serve-devwin

package.json

@@ -52,6 +52,8 @@
     "eslint-plugin-react": "^7.33.2",
     "eslint-plugin-simple-import-sort": "^10.0.0",
     "eslint-plugin-storybook": "^0.6.15",
+    "eslint-plugin-vitest": "^0.3.10",
+    "eslint-plugin-vitest-globals": "^1.4.0",
     "file-loader": "^6.2.0",
     "file-saver": "^2.0.5",
     "glob-parent": "^6.0.2",

test/js/component/topic/__snapshots__/topic.test.jsx.snap

@@ -333,8 +333,8 @@ exports[`topic component > should render topic 2`] = `
       </button>
     </h2>
     <div
-      class="accordion-collapse collapsing"
-      style="height: 0px;"
+      class="accordion-collapse collapse show"
+      style=""
     >
       <div
         class="accordion-body"

test/js/setupTests.js

@@ -1,9 +1,9 @@
 import '@testing-library/jest-dom/vitest';
 
-import createFetchMock from "vitest-fetch-mock";
 import {initReactI18next} from "react-i18next";
 import ResizeObserver from "resize-observer-polyfill";
-import { vi } from "vitest";
+import {vi} from "vitest";
+import createFetchMock from "vitest-fetch-mock";
 
 import i18n from '../../oereb_client/static/src/i18n';
 

vite.config.js

@@ -21,6 +21,10 @@ export default defineConfig({
     react(),
     htmlPurge()
   ],
+  define: {
+    // 'process.env.NODE_ENV': '"production"'
+    'process.env': {}
+  },
   test: {
     environment: 'jsdom',
     globals: true,