GitHub Actions: npm test is failing Windows tests on M1 Macs (#3011)

cclauss · StefanStojanovic · web-flow · commit 0bab6a071bbc · 2024-04-11T17:34:49.000+02:00
* GitHub Actions: npm test is failing Windows tests on M1 Macs

`npm test` is running and failing Windows `find-visualstudio` tests on an M1 Mac!!!

These tests are not being run or not failing on Intel Macs.

* Only run "Find Visual Studio" tests on Windows

* Update test/test-find-visualstudio.js

Co-authored-by: Christian Clauss &lt;cclauss@me.com&gt;

---------

Co-authored-by: Stefan Stojanovic &lt;StefanStojanovic@users.noreply.github.com&gt;
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -93,11 +93,15 @@ jobs:
       fail-fast: false
       max-parallel: 15
       matrix:
-        os: [macos, ubuntu, windows]
+        os: [macos-latest, ubuntu-latest, windows-latest]
         python: ["3.8", "3.10", "3.12"]
         node: [16.x, 18.x, 20.x]
+        include:  # `npm test` is running Windows find-visualstudio tests on an M1 Mac!!!
+          - os: macos-14
+            python: "3.12"
+            node: 20.x
     name: ${{ matrix.os }} - ${{ matrix.python }} - ${{ matrix.node }}
-    runs-on: ${{ matrix.os }}-latest
+    runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout Repository
         uses: actions/checkout@v4
@@ -116,20 +120,20 @@ jobs:
           npm install
           pip install pytest
       - name: Set Windows Env
-        if: runner.os == 'Windows'
+        if: startsWith(matrix.os, 'windows')
         run: |
           echo 'GYP_MSVS_VERSION=2015' >> $Env:GITHUB_ENV
           echo 'GYP_MSVS_OVERRIDE_PATH=C:\\Dummy' >> $Env:GITHUB_ENV
       - name: Run Python Tests
         run: python -m pytest
       - name: Run Tests (macOS or Linux)
-        if: runner.os != 'Windows'
+        if: "!startsWith(matrix.os, 'windows')"
         shell: bash
         run: npm test --python="${pythonLocation}/python"
         env:
           FULL_TEST: ${{ (matrix.node == '20.x' && matrix.python == '3.12') && '1' || '0' }}
       - name: Run Tests (Windows)
-        if: runner.os == 'Windows'
+        if: startsWith(matrix.os, 'windows')
         shell: pwsh
         run: npm run test --python="${env:pythonLocation}\\python.exe"
         env:
diff --git a/test/test-find-visualstudio.js b/test/test-find-visualstudio.js
@@ -21,7 +21,13 @@ class TestVisualStudioFinder extends VisualStudioFinder {
   }
 }
 
+const shouldSkip = process.platform !== 'win32'
+
 describe('find-visualstudio', function () {
+  if (shouldSkip) {
+    return
+  }
+
   this.beforeAll(function () {
     // Condition to skip the test suite
     if (process.env.SystemRoot === undefined) {

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,13 @@ class TestVisualStudioFinder extends VisualStudioFinder {`
`21`	`21`	`}`
`22`	`22`	`}`
`23`	`23`
	`24`	`+const shouldSkip = process.platform !== 'win32'`
	`25`	`+`
`24`	`26`	`describe('find-visualstudio', function () {`
	`27`	`+ if (shouldSkip) {`
	`28`	`+ return`
	`29`	`+ }`
	`30`	`+`
`25`	`31`	`this.beforeAll(function () {`
`26`	`32`	`// Condition to skip the test suite`
`27`	`33`	`if (process.env.SystemRoot === undefined) {`