Skip to content

test: add Docker e2e for profile sync and doctor check#76

Merged
Keith-CY merged 1 commit intodevelopfrom
test/e2e-docker-profile-sync
Mar 5, 2026
Merged

test: add Docker e2e for profile sync and doctor check#76
Keith-CY merged 1 commit intodevelopfrom
test/e2e-docker-profile-sync

Conversation

@Keith-CY
Copy link
Copy Markdown
Collaborator

@Keith-CY Keith-CY commented Mar 5, 2026

Summary

  • Adds docker_profile_sync_e2e.rs — a Docker-based e2e test that spins up an Ubuntu container with SSH + fake openclaw CLI + seeded configs
  • Tests the full ClawPal remote workflow: SSH password auth connect → SFTP config read → model profile extraction → auth store credential resolution → openclaw doctor --json health check → agents listing → disconnect
  • Guarded by CLAWPAL_RUN_DOCKER_SYNC_E2E=1 env var (skipped by default)

Test plan

  • Run locally: CLAWPAL_RUN_DOCKER_SYNC_E2E=1 cargo test -p clawpal --test docker_profile_sync_e2e -- --nocapture
  • Verify test passes with Docker Desktop running
  • Verify test is skipped gracefully when env var is not set
  • Verify test is skipped gracefully when Docker is not available

🤖 Generated with Claude Code

@Keith-CY Keith-CY changed the base branch from main to develop March 5, 2026 02:57
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

📊 Test Coverage Report

Metric Base (develop) PR (test/e2e-docker-profile-sync) Delta
Lines 73.52% (5545/7542) 73.61% (5552/7542) 🟢 +0.09%
Functions 66.98% (641/957) 66.98% (641/957) ⚪ ±0.00%
Regions 74.92% (9329/12452) 75.02% (9342/12452) 🟢 +0.10%

Coverage measured by cargo llvm-cov (clawpal-core + clawpal-cli).

Adds docker_profile_sync_e2e.rs that spins up an Ubuntu container with SSH
and a fake openclaw CLI, then verifies the full ClawPal workflow: SSH connect
with password auth, config reading via SFTP, model profile extraction, auth
store credential resolution, doctor health check, and agents listing.

Guarded by CLAWPAL_RUN_DOCKER_SYNC_E2E=1 env var.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Keith-CY Keith-CY force-pushed the test/e2e-docker-profile-sync branch from c75edc4 to 87a0a02 Compare March 5, 2026 02:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

📦 PR Build Artifacts

⚠️ No build artifacts were produced. Check the workflow run for details.


🔨 Built from 87a0a02 · View workflow run
⚠️ Unsigned development builds — for testing only

@Keith-CY Keith-CY merged commit b742730 into develop Mar 5, 2026
9 checks passed
@Keith-CY Keith-CY deleted the test/e2e-docker-profile-sync branch March 5, 2026 04:59
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