Merge pull request #36 from Financial-Times/am/CON-3084-update-mocha

[CON-3084] update `mocha`
Financial-Times · Dec 18, 2023 · 92f60d8 · 92f60d8
2 parents 00c904c + 9806c07
commit 92f60d8
Show file tree

Hide file tree

Showing 5 changed files with 1,256 additions and 282 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,4 @@
 /.idea
 /coverage
 /.nyc_output/
+test-results.xml
diff --git a/.mocharc.json b/.mocharc.json
@@ -0,0 +1,10 @@
+{
+  "reporter": "mocha-multi-reporters",
+  "recursive": true,
+  "exit": true,
+  "reporterEnabled": "mocha-junit-reporter, spec",
+  "reporterOptions": "configFile=.mocharc.json",
+  "mochaJunitReporterReporterOptions": {
+    "mochaFile": "./test-results.xml"
+  }
+}
diff --git a/Makefile b/Makefile
@@ -7,7 +7,7 @@ node_modules/@financial-times/n-gage/index.mk:
 -include node_modules/@financial-times/n-gage/index.mk
 
 unit-test:
-	export NODE_ENV=test; mocha 'test/spec/**/*.spec.js'
+	export NODE_ENV=test; mocha test
 
 # unit-test-coverage:
 	# nyc --reporter=$(if $(CIRCLECI),lcovonly,lcov) make unit-test