Skip to content

Keep compiler rollouts out of the deployment smoke test #817

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
thomashoneyman wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Keep compiler rollouts out of the deployment smoke test #817

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c54b9a1
Make the VM smoke test check startup job results
thomashoneyman Oct 23, 2026
cda1da0
Keep the VM smoke test focused on deployment
thomashoneyman Oct 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
14 changes: 12 additions & 2 deletions nix/test/smoke.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,7 @@ else
"curl -s http://localhost:${envVars.SERVER_PORT}/api/v1/jobs"
)

# The server may create matrix jobs on startup for new compilers, so we just verify
# the response is valid JSON (starts with '[')
# Verify the response is valid JSON (starts with '[').
assert result.strip().startswith("["), f"Expected JSON array, got: {result}"

# Verify the database was created and migrations ran
Expand Down Expand Up @@ -107,6 +106,17 @@ else
mkdir -p ${stateDir}
echo 'DHALL_TYPES=/defunct/types' > ${stateDir}/.env
${testConfig.setupGitFixtures}/bin/setup-git-fixtures ${repoFixturesDir}

# Smoke tests cover deployment, not compiler rollouts. Record the test
# compilers in this copy of Prelude without changing integration fixtures.
compilers=$(${(pkgs.extend testConfig.testOverlay).registry-purs-versions}/bin/purs-versions)
prelude=${repoFixturesDir}/purescript/registry/metadata/prelude.json
${pkgs.jq}/bin/jq --arg compilers "$compilers" \
'.published[].compilers = ($compilers | split(" "))' "$prelude" > "$prelude.tmp"
mv "$prelude.tmp" "$prelude"
${pkgs.git}/bin/git -C ${repoFixturesDir}/purescript/registry \
-c user.name=pacchettibotti -c user.email=pacchettibotti@purescript.org \
commit -am "Record test compilers for smoke test"
'';
};

Expand Down
Loading