Skip to content

ci: two specs existed in no tier — wire them (DEV-2203) #242

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
danielzytohoc merged 2 commits into from
Aug 20, 2026
+7 −2
Merged

ci: two specs existed in no tier — wire them (DEV-2203) #242

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4100b7e
ci: two specs existed in no tier — wire them (DEV-2203 final sweep)
danielzytohoc Aug 20, 2026
5fe9942
Merge origin/master into chore/DEV-2203-wire-orphaned-specs
danielzytohoc Aug 20, 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
9 changes: 7 additions & 2 deletions .github/workflows/e2e-live.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ jobs:
e2e/version-pinning.spec.ts
e2e/share-view.spec.ts
e2e/style-panel.spec.ts
e2e/preview-scheme.spec.ts
e2e/preview-head-assets.spec.ts

# Container suites, one worker: each Tier-2 case holds a live-preview
Expand Down Expand Up @@ -175,10 +176,14 @@ jobs:
if: ${{ !inputs.smoke && env.BASE_URL != '' }}
env:
E2E_BASE_URL: ${{ env.BASE_URL }}
# The session-leak spec (DEV-2567) drives the API directly and reads
# the auth-gated /api/admin/sessions listing, so it rides this
# token-gated step rather than the anonymous suites.
E2E_API_BASE: ${{ env.BASE_URL }}
E2E_BROKER_TOKEN: ${{ secrets.E2E_BROKER_TOKEN }}
run: |
if [ -z "$E2E_BROKER_TOKEN" ]; then
echo "::notice::E2E_BROKER_TOKEN is not set — the authed share round-trip was skipped."
echo "::notice::E2E_BROKER_TOKEN is not set — the authed share round-trip and the session-leak spec were skipped."
exit 0
fi
code=$(curl -s -o /dev/null -w "%{http_code}" -H "Authorization: Bearer $E2E_BROKER_TOKEN" \
Expand All @@ -193,7 +198,7 @@ jobs:
# and this repo is public. No traces, no retries, and every artifact
# of this step is scrubbed before the on-failure upload can see it.
status=0
pnpm e2e e2e/share-create-live.spec.ts --workers=1 --retries=0 --trace off || status=$?
pnpm e2e e2e/share-create-live.spec.ts e2e/session-abandoned-create.spec.ts --workers=1 --retries=0 --trace off || status=$?
rm -rf test-results playwright-report
exit $status

Expand Down
Loading