Fix ACM paper parsing & Tests in CI (#333)

* 🐛 fix: Declare `message`

* ♻  refactor: Handle (and warn) non-string keys in `bibtexToString`

* 🐛 fix: Fix ACM parsing

* 🏗  build: Dev bundle

* 🔄 ci: Fix build CI

* 🧪 test: Default to headless

* 🏗  build: Cleanup available package.json scripts &+ `test-ci`

* 🧪 test: Add test action on push

* 🧪 test: rename action

* 🔄 ci: No sandbox for CI

* 🏗  build: Add `--exit` flag for test-ci

Author: vict0rsch

.github/workflows/build.yml

@@ -10,9 +10,7 @@ jobs:
               with:
                   node-version: "22.x"
 
-            - name: Install Yarn
-              run: npm install --global yarn
             - name: Install dependencies
-              run: yarn install
+              run: npm install
             - name: Build
-              run: gulp build
+              run: npm run build

.github/workflows/test.yml

@@ -0,0 +1,16 @@
+name: test-ci
+on: push
+
+jobs:
+    test:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v4
+            - uses: actions/setup-node@v4
+              with:
+                  node-version: "22.x"
+
+            - name: Install dependencies
+              run: npm install
+            - name: Build
+              run: npm run test-ci

package.json

@@ -11,26 +11,10 @@
     "scripts": {
         "pretest": "echo 'Make sure to be in dev mode (npm run dev)'",
         "test": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-*.js",
+        "pretest-ci": "npm run dev",
+        "test-ci": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-*.js --exclude *test-storage.js --exclude test/test-storage.js --exclude test/test-duplicates.js --exclude test/test-sync.js --exit",
         "pretest-cov": "echo 'Make sure to be in dev mode (npm run dev)'",
         "test-cov": "node --import ./register.mjs ./node_modules/.bin/nyc --reporter=text ./node_modules/.bin/mocha test/test-*.js",
-        "pretest-storage": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-storage": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-storage.js",
-        "pretest-duplicates": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-duplicates": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-duplicates.js",
-        "pretest-sync": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-sync": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-sync.js",
-        "pretest-extension-loading": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-extension-loading": "node --import ./register.mjs ./node_modules/.bin/mocha test/test-extension-loading.js",
-        "pretest-no-browser": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-no-browser": "node --import ./register.mjs ./node_modules/.bin/mocha 'test/test-!(storage|duplicates|sync|extension-loading)*.js'",
-        "pretest-no-browser-cov": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-no-browser-cov": "node --import ./register.mjs ./node_modules/.bin/nyc --reporter=text ./node_modules/.bin/mocha 'test/test-!(storage|duplicates|sync|extension-loading)*.js'",
-        "pretest-no-sync": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-no-sync": "node --import ./register.mjs ./node_modules/.bin/mocha 'test/test-!(sync)*.js'",
-        "pretest-no-sync-cov": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "test-no-sync-cov": "node --import ./register.mjs ./node_modules/.bin/nyc --reporter=text ./node_modules/.bin/mocha 'test/test-!(sync)*.js'",
-        "prescreenshots": "echo 'Make sure to be in dev mode (npm run dev)'",
-        "screenshots": "node --import ./register.mjs ./node_modules/.bin/mocha test/screenshots.js",
         "dev": "rollup -c",
         "dev:watch": "rollup -c -w",
         "build": "NODE_ENV=production rollup -c"