test: skip version export testing in ci

.github/workflows/unit-tests.yml

@@ -18,4 +18,4 @@ jobs:
         run: npm run lint
 
       - name: Run Test
-        run: npm run test
+        run: npm run test:ci

package.json

@@ -47,9 +47,10 @@
     "doc": "npm run types && rm -rf docs && jsdoc -c .jsdoc --verbose && cp -R misc/assets docs/",
     "lint": "npx eslint src",
     "test": "npx mocha tests/*/*.spec.js",
+    "test:ci": "npx mocha tests/*/*.spec.js --ci",
     "test:only": "npx mocha",
     "types": "rm -rf types && tsc",
-    "preversion": "node ./misc/scripts/generate-version-file.js $npm_new_version && npm run types",
+    "preversion": "npm run lint && npm run test && node ./misc/scripts/generate-version-file.js $npm_new_version && npm run types",
     "postversion": "node ./misc/scripts/check-changelog.js && git commit -am \"$npm_new_version\" --allow-empty"
   },
   "dependencies": {

tests/misc/version-test.spec.js

@@ -8,6 +8,7 @@ import { assert } from 'chai';
 
 import config from '../utils/config.js';
 const { version } = JSON.parse(fs.readFileSync('package.json').toString());
+const CI = process.argv.includes('--ci');
 
 describe(`# Client / Server test versions discrepancies`, () => {
   it(`client and server should check soundworks version used`, async () => {
@@ -26,7 +27,13 @@ describe(`# Client / Server test versions discrepancies`, () => {
     await server.stop();
   });
 
-  it('should properly export version', () => {
+  it('should properly export version', function() {
+    // Skip in CI - this crashes in CI when we make a new version because we have
+    // 2 different commits, not fundamentally critical...
+    if (CI) {
+      this.skip();
+    }
+
     assert.equal(serverVersion, version);
     assert.equal(clientVersion, version);
   });