ci: attempt to use correct platform name for ci runs

appium · Jan 3, 2024 · 1d41a92 · 1d41a92
1 parent f379b6c
commit 1d41a92
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -34,7 +34,7 @@ jobs:
         with:
           pattern: '{.renovaterc.json,renovate/default.json}'
   test:
-    needs: 
+    needs:
     - lint
     - prepare_matrix
     strategy:
@@ -77,9 +77,9 @@ jobs:
         shell: bash
         run: |
           if ["$RUNNER_OS" == "Windows"]; then
-            npm run test:ci
+            TEST_PLATFORM="$RUNNER_OS" npm run test:ci
           else
-            DISPLAY=:99 npm run test:ci
+            DISPLAY=:99 TEST_PLATFORM="$RUNNER_OS" npm run test:ci
           fi
   release:
     if: github.ref == 'refs/heads/main'

diff --git a/test/e2e/driver.spec.js b/test/e2e/driver.spec.js
@@ -2,7 +2,11 @@ import {main as startAppium} from 'appium';
 import path from 'path';
 import {remote} from 'webdriverio';
 
-const PLATFORM = process.env.TEST_PLATFORM || 'mac';
+const PLATFORM_ENV = process.env.TEST_PLATFORM || '';
+
+const PLATFORM = PLATFORM_ENV.toLowerCase() === 'macos' ? 'mac' :
+  PLATFORM_ENV.toLowerCase() ||
+  'mac';
 const PORT = process.env.TEST_PORT || 4780;
 const HOST = '127.0.0.1';
 const CHROME_BIN = process.env.TEST_CHROME;