CI TESTS: Update Github workflow to pass integration tests using Xvfb

`npm run test-integration` uses Electron and therefore needs a display to work properly.

Other changes:
- Add a build step to make sure `npm run build` is always ran (dist/ could change without a package-lock.json change, I think)
- Cache: add ~/.npm and fix /tmp/.__tmp_int (test-meteor-desktop path)
- Trigger action for all pull requests, not only those against master
- Update actions/checkout to v4
- Renaming

Author: dd137

.github/workflows/test.yml

@@ -1,69 +1,80 @@
-
 on:
-  push:
-    branches:
-      - master
   pull_request:
+  push:
     branches:
       - master
 
+env:
+  METEOR_VERSION: 2.6.1
+
 jobs:
   build:
     name: Build and Install
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - name: Cache dependencies
+      - uses: actions/checkout@v4
+      - name: Get Cache dependencies
         uses: actions/cache@v3
         id: files-cache
         with:
-         path: |
-           node_modules
-           dist/
-           ~/.meteor
-           /tmp/tests/.__tmp_int
-         key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
+          path: |
+            dist
+            node_modules
+            /tmp/.__tmp_int
+            ~/.meteor
+            ~/.npm
+          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
       - name: Install/Retrieve Meteor
         if: steps.files-cache.outputs.cache-hit != 'true'
-        run: curl https://install.meteor.com/?release=2.6.1 | /bin/sh
+        run: curl https://install.meteor.com/?release=${{ env.METEOR_VERSION }} | /bin/sh
       - name: Install NPM Dependencies
+        # Creates dist/ through postinstall->build script
         if: steps.files-cache.outputs.cache-hit != 'true'
-        run: meteor npm ci
+        run: ~/.meteor/meteor npm ci
+      - name: Run build script
+        # Make sure dist/ is up-to-date when package-lock.json hasn't changed
+        if: steps.files-cache.outputs.cache-hit == 'true'
+        run: ~/.meteor/meteor npm run build
 
-  test:
+  unit-tests:
     needs: build
     name: Unit Tests
     runs-on: ubuntu-latest
     steps:
-     - uses: actions/checkout@v3
-     - name: Get Cache dependencies
-       uses: actions/cache@v3
-       id: files-cache
-       with:
-        path: |
-           node_modules
-           dist/
-           ~/.meteor
-           /tmp/tests/.__tmp_int
-        key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-     - name: Unit Tests
-       run: ~/.meteor/meteor npm run test
+      - uses: actions/checkout@v4
+      - name: Get Cache dependencies
+        uses: actions/cache@v3
+        id: files-cache
+        with:
+          path: |
+            dist
+            node_modules
+            /tmp/.__tmp_int
+            ~/.meteor
+            ~/.npm
+          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
+      - name: Run test script
+        run: ~/.meteor/meteor npm run test
 
-  test-integration:
+  integration-tests:
     needs: build
     name: Integration Tests
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Get Cache dependencies
         uses: actions/cache@v3
         id: files-cache
         with:
-         path: |
-           node_modules
-           dist/
-           ~/.meteor
-           /tmp/tests/.__tmp_int
-         key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
+          path: |
+            dist
+            node_modules
+            /tmp/.__tmp_int
+            ~/.meteor
+            ~/.npm
+          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
       - run: ~/.meteor/meteor npm run prepare-integration-tests
-      - run: ~/.meteor/meteor npm run test-integration
+      - name: Run test-integration script (using Xvfb)
+        uses: coactions/setup-xvfb@v1
+        with: # needs full path to meteor
+          run: /home/runner/.meteor/meteor npm run test-integration