Remove set-env from actions (#8063)

* Remove set-env from CI

* Eanble macOs

* Another try on macOs, update npmpublish

* No xvfb-run on macOs

* Remove macos

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 55f5f4490b4043665d6db6691593a5a546bfd03e..27b4ad0223eed146d7bae04dc2b8fbb8952cfe56 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -23,27 +23,18 @@ jobs:
     - uses: actions/checkout@v2
     - name: Use Node.js
       uses: actions/setup-node@v1
-    - name: Setup xvfb
-      run: |
-        Xvfb :99 -screen 0 1024x768x24 &
-        echo "::set-env name=DISPLAY:::99.0"
-      if: runner.os == 'Linux'
-    - name: Install chrome
-      run: |
-        brew update
-        brew cask install google-chrome
-      if: runner.os == 'macOS'
-    - name: Select browsers
-      run: |
-        if [ "${{ runner.os }}" == "macOS" ]; then
-          echo "::set-env name=BROWSERS::--browsers chrome"
-        fi
-      shell: bash
     - name: Build and Test
       run: |
         npm ci
         npm run build
-        npm test
+        if [ "${{ runner.os }}" == "Windows" ]; then
+          npm test
+        elif [ "${{ runner.os }}" == "macOS" ]; then
+          npm test --browsers chrome,safari
+        else
+          xvfb-run --auto-servernum npm test
+        fi
+      shell: bash
     - name: Package
       run: |
         npm run docs

diff --git a/.github/workflows/npmpublish.yml b/.github/workflows/npmpublish.yml
index bbe9d84904145ba45eb6c53b738dfeb316c9dec6..40506f7a9c624c91041ad147a48dbcf2472a22a0 100644 (file)
--- a/.github/workflows/npmpublish.yml
+++ b/.github/workflows/npmpublish.yml
@@ -12,10 +12,10 @@ jobs:
     runs-on: ubuntu-latest
     outputs:
       version: ${{ steps.trim.outputs.version }}
-    steps: 
+    steps:
       - id: trim
         run: echo "::set-output name=version::${TAG:1}"
-        env: 
+        env:
           TAG: ${{ github.event.release.tag_name }}
 
   test:
@@ -24,14 +24,10 @@ jobs:
     - uses: actions/checkout@v2
     - name: Use Node.js
       uses: actions/setup-node@v1
-    - name: Setup xvfb
-      run: |
-        Xvfb :99 -screen 0 1024x768x24 &
-        echo "::set-env name=DISPLAY:::99.0"
     - name: Test
       run: |
         npm ci
-        npm test
+        xvfb-run --auto-servernum npm test
 
   publish-npm:
     needs: [test, setup]