Skip to content

Fix test.yml: build + stage sidecar before Tauri shell tests - #91

Closed
CodeGhost21 wants to merge 2 commits into
tinyhumansai:mainfrom
CodeGhost21:ci/fix-test-sidecar-v2
Closed

CodeGhost21 wants to merge 2 commits into
tinyhumansai:mainfrom
CodeGhost21:ci/fix-test-sidecar-v2

Conversation

@CodeGhost21

Copy link
Copy Markdown
Contributor

Summary

  • Build and stage the openhuman sidecar binary before running cargo test on the Tauri shell
  • The Tauri build script validates sidecar resource paths even during tests, so the binary must exist at app/src-tauri/binaries/openhuman-x86_64-unknown-linux-gnu
  • This catches real shell/sidecar integration issues in CI rather than skipping the Tauri shell tests entirely

Test plan

  • Verify Rust Tests + Quality job passes (including Tauri shell tests)

🤖 Generated with Claude Code

The Tauri build script validates sidecar resource paths even during
cargo test. Build and stage the openhuman binary so Tauri shell tests
can run, catching real integration issues early.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@CodeGhost21
CodeGhost21 marked this pull request as ready for review March 30, 2026 20:11
@CodeGhost21
CodeGhost21 marked this pull request as draft March 30, 2026 20:13
Pass the JSON config directly to avoid shell variable expansion issues
with yarn. Disables updater plugin so TAURI_SIGNING_PRIVATE_KEY is
not required for CI builds.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@senamakel senamakel closed this Apr 1, 2026
AusAgentSmith pushed a commit to AusAgentSmith/openhuman that referenced this pull request May 23, 2026
chore: merge develop into main (v0.38.0)
senamakel added a commit to nocstah/openhuman that referenced this pull request Sep 11, 2026
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.

2 participants