Skip to content

ci: run the full test suite on macOS instead of two hand-picked files #483

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vinhnguyenthanhdn wants to merge 1 commit into
base: main
Choose a base branch
from
+22 −6
Open

ci: run the full test suite on macOS instead of two hand-picked files #483

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 22 additions & 6 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,14 +119,30 @@ jobs:
with:
node-version: 24
cache: npm
cache-dependency-path: |
package-lock.json
dashboard/package-lock.json

- name: Test OpenClaw parser on macOS
run: |
npm ci
node --test test/openclaw-parser.test.js
- name: Install root dependencies
run: npm ci

- name: Install dashboard dependencies
run: npm ci --prefix dashboard

# Same reason as the Linux job: the local-runtime install test asserts the
# bundled dashboard/dist/index.html exists, and a fresh checkout has none.
- name: Build dashboard
env:
VITE_INSFORGE_BASE_URL: https://srctyff5.us-east.insforge.app
VITE_INSFORGE_ANON_KEY: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3OC0xMjM0LTU2NzgtOTBhYi1jZGVmMTIzNDU2NzgiLCJlbWFpbCI6ImFub25AaW5zZm9yZ2UuY29tIiwicm9sZSI6ImFub24iLCJpYXQiOjE3ODExNDU5NDd9.T0auta_IrVIh0uXW1bob5QSnzvsnJmN28r5XkSGEuQY
run: npm run dashboard:build

- name: Test macOS native pet limit reset interpolation
run: node --test test/native-pet-limit-reset.test.js
# The whole suite, not a hand-picked pair of files: host assumptions that
# only hold on GNU userland (coreutils, GNU sed, ...) are invisible to the
# Linux job by construction, and `npm test` is what CONTRIBUTING asks
# contributors to run before opening a PR.
- name: Test suite on macOS
run: npm test

- name: Install xcodegen
run: brew install xcodegen
Expand Down
Loading