Skip to content

Make the Safari CI workflow use the local test server #2481

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

Merged
soulgalore merged 3 commits into from
May 23, 2026
Merged

Make the Safari CI workflow use the local test server #2481

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
35037c1
Make the Safari CI workflow use the local test server
soulgalore May 23, 2026
e6bee7a
fix
soulgalore May 23, 2026
2dd8fd4
test
soulgalore May 23, 2026
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
17 changes: 13 additions & 4 deletions .github/workflows/mac-m1.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:
- main
jobs:
build:
runs-on: macos-latest
runs-on: macos-14
timeout-minutes: 20
steps:
- name: Harden Runner
Expand Down Expand Up @@ -40,10 +40,19 @@ jobs:
# run: system_profiler SPDisplaysDataType |grep Resolution
- name: Enable Safaridriver
run: sudo safaridriver --enable
#- name: Start local HTTP server
# run: (npm run start-server&)
- name: Start local HTTP server
run: (npm run start-server&)
- name: Safari environment diagnostics
run: |
sw_vers
echo "---"
/usr/bin/safaridriver --version
echo "---"
mdls -name kMDItemVersion /Applications/Safari.app || true
echo "---"
pgrep -lf safaridriver || echo "no safaridriver process"
- name: Test Safari without video
run: ./bin/browsertime.js -b safari -n 1 https://www.sitespeed.io/
run: ./bin/browsertime.js -b safari -n 1 http://127.0.0.1:3000/simple/
#- name: Test Safari with video and Visual Metrics
# run: ./bin/browsertime.js -b safari -n 1 --viewPort 800x600 --video --visualMetrics https://www.sitespeed.io/
#- name: Test Safari iOS simulator
Expand Down
19 changes: 14 additions & 5 deletions .github/workflows/safari.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:
- main
jobs:
build:
runs-on: macos-latest
runs-on: macos-14
timeout-minutes: 20
steps:
- name: Harden Runner
Expand Down Expand Up @@ -44,12 +44,21 @@ jobs:
run: ffmpeg -version
- name: List all simulators
run: xcrun xctrace list devices
#- name: Start local HTTP server
# run: (npm run start-server&)
- name: Start local HTTP server
run: (npm run start-server&)
- name: Safari environment diagnostics
run: |
sw_vers
echo "---"
/usr/bin/safaridriver --version
echo "---"
mdls -name kMDItemVersion /Applications/Safari.app || true
echo "---"
pgrep -lf safaridriver || echo "no safaridriver process"
- name: Test Safari without video
run: ./bin/browsertime.js -b safari -n 1 https://www.sitespeed.io/
run: ./bin/browsertime.js -b safari -n 1 http://127.0.0.1:3000/simple/
- name: Test Safari with video and Visual Metrics
run: ./bin/browsertime.js -b safari -n 1 --viewPort 800x600 --video --visualMetrics https://www.sitespeed.io/
run: ./bin/browsertime.js -b safari -n 1 --viewPort 800x600 --video --visualMetrics http://127.0.0.1:3000/simple/
#- name: Test Safari iOS simulator
# run: |
# IPHONE14=$(xcrun xctrace list devices 2>&1 | grep -m 1 "iPhone 14 Simulator" | awk -F'[()]' '{print $4}')
Expand Down
Loading