Skip to content

Fix Ubuntu test prerequisites - #264

Merged
fliingelephant merged 2 commits into
QuantumBFS:mainfrom
Joe-Nor:agent/fix-ubuntu-ci
Aug 6, 2026
Merged

Fix Ubuntu test prerequisites#264
fliingelephant merged 2 commits into
QuantumBFS:mainfrom
Joe-Nor:agent/fix-ubuntu-ci

Conversation

@Joe-Nor

@Joe-Nor Joe-Nor commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Install uv in the Ubuntu test job, matching the phase-diagram CLI test's declared runtime.
  • Validate the Julia shim destination before installing DaemonMode, so a request to overwrite the daemon runner itself is rejected before any Julia lookup or installation side effect.

Why

The current main branch and newly synchronized pull requests fail only on Ubuntu for two unrelated infrastructure reasons:

  1. scripts/tests/test_phase_diagram.py invokes uv, but the Ubuntu workflow did not install it.
  2. scripts/tests/test_julia_daemon.py expects the self-overwrite safety error to take precedence, while the install-shim dispatcher attempted to install DaemonMode first and therefore reported a missing Julia executable.

This infrastructure fix is intentionally separate from challenge submission #196.

Validation

  • Targeted regressions: 2 passed.
  • Full suite: 235 passed, 9 skipped; 95% reported coverage.
  • bash -n scripts/julia-daemon.sh
  • git diff --check

@hz-xiaxz

Copy link
Copy Markdown
Contributor

Thanks for fixing CI for julia-daemon!

hz-xiaxz

This comment was marked as spam.

hz-xiaxz added a commit to hz-xiaxz/quantum.harness that referenced this pull request Jul 30, 2026
@Joe-Nor
Joe-Nor marked this pull request as ready for review July 30, 2026 11:15
Co-authored-by: LeoXia <114814844+hz-xiaxz@users.noreply.github.com>
@fliingelephant
fliingelephant merged commit 711e5e0 into QuantumBFS:main Aug 6, 2026
2 checks passed
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.

3 participants