Skip to content

CI check (fork only) - #2

Closed
isc-klu wants to merge 1 commit into
masterfrom
prepare-ci-check
Closed

CI check (fork only)#2
isc-klu wants to merge 1 commit into
masterfrom
prepare-ci-check

Conversation

@isc-klu

@isc-klu isc-klu commented Sep 10, 2026

Copy link
Copy Markdown
Owner

Throwaway PR so the prepare-* workflow from intersystems-community#1870 runs in GitHub CI.

Replaces the placeholder test with a suite that opens a multi-root workspace whose folders connect to
one iris-community container by each supported mechanism (objectscript.conn host/port, docker-compose
port resolution, intersystems.servers entry, isfs), plus an inactive folder pointing at an unreachable
host. The container's /api/atelier session timeout is 10 s so expired-session recovery is exercised.

Runs from a new prepare-release.yml workflow on PRs from prepare-* branches and on manual dispatch;
the old npm test step is removed from the per-PR workflows.

Also fixes the runner, which could not launch VS Code at all: bump @vscode/test-electron to 3.x
(2.x looks for the renamed Contents/MacOS/Electron), drop the rejected -n arg, unset
ELECTRON_RUN_AS_NODE inherited from extension-spawned terminals, use a fresh user-data-dir per run,
and make the mocha runner's promise settle when the tests finish.

Closes intersystems-community#1863

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@isc-klu

isc-klu commented Sep 10, 2026

Copy link
Copy Markdown
Owner Author

Green on the fork (run 34540092591); closing.

@isc-klu isc-klu closed this Sep 10, 2026
@isc-klu
isc-klu deleted the prepare-ci-check branch September 10, 2026 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant