Add prettier to mvntest

Author: felipecrs

CONTRIBUTING.md

@@ -98,9 +98,9 @@ mvn package -P quick-build
 
 Code style will be enforced by GitHub pull request checks.
 
-For frontend code we use [eslint](https://eslint.org) and [prettier](https://prettier.io/).
+For frontend code we use [prettier](https://prettier.io/) and [eslint](https://eslint.org).
 
-You can automatically fix issues with `npm run lint:fix`
+You can automatically fix issues with `npm run format && npm run eslint:fix`
 
 For java code we use [spotless](https://github.com/diffplug/spotless).
 

package.json

@@ -5,16 +5,20 @@
   "private": true,
   "type": "module",
   "scripts": {
-    "mvnbuild": "npm run build",
-    "mvntest": "eslint --format checkstyle --output-file target/eslint-warnings.xml; vitest run --reporter junit",
     "build": "vite build --mode production",
     "build:dev": "vite build --mode production --watch",
-    "format:check": "prettier --check .",
     "format": "prettier --write .",
-    "lint": "tsc && eslint && npm run format:check",
-    "lint:fix": "eslint --fix && npm run format",
+    "format:check": "prettier --check .",
+    "tsc:check": "tsc",
+    "eslint": "eslint",
+    "eslint:fix": "eslint --fix",
+    "lint": "npm run tsc:check && npm run eslint",
     "test": "vitest run",
-    "test:dev": "vitest watch"
+    "test:dev": "vitest watch",
+    "eslint:mvn": "eslint --format checkstyle --output-file target/eslint-warnings.xml || ([ -n \"$JENKINS_URL\" ] && [ -f target/eslint-warnings.xml ])",
+    "vitest:mvn": "vitest run --reporter default --reporter junit --outputFile \"$PWD/target/surefire-reports/vitest-junit.xml\" || ([ -n \"$JENKINS_URL\" ] && [ -f target/surefire-reports/vitest-junit.xml ])",
+    "mvnbuild": "npm run build",
+    "mvntest": "npm run format:check && npm run tsc:check && npm run eslint:mvn && npm run vitest:mvn"
   },
   "repository": {
     "type": "git",

pom.xml

@@ -38,10 +38,6 @@
     <node.version>22.15.0</node.version>
     <npm.version>11.3.0</npm.version>
     <spotless.check.skip>false</spotless.check.skip>
-
-    <!-- Opt in to Jenkins handling eslint results -->
-    <maven.test.failure.ignore>false</maven.test.failure.ignore>
-    <frontend.testFailureIgnore>${maven.test.failure.ignore}</frontend.testFailureIgnore>
   </properties>
 
   <dependencyManagement>

vitest.config.ts

@@ -1,15 +1,10 @@
 import { defineConfig } from "vitest/config";
-import path from "node:path";
 
 export default defineConfig({
   test: {
     root: "src/main/frontend",
     globals: true,
     environment: "jsdom",
     setupFiles: ["setupTests.ts"],
-    outputFile: path.resolve(
-      import.meta.dirname,
-      "target/surefire-reports/vitest-junit.xml",
-    ),
   },
 });