Skip to content

Fix all stale test fixtures: cargo test --workspace fully green#73

Merged
YASSERRMD merged 3 commits into
mainfrom
fix_workspace_tests
Jun 9, 2026
Merged

Fix all stale test fixtures: cargo test --workspace fully green#73
YASSERRMD merged 3 commits into
mainfrom
fix_workspace_tests

Conversation

@YASSERRMD

Copy link
Copy Markdown
Owner

Make the whole workspace test suite pass

Repairs the pre-existing stale #[cfg(test)] fixtures and example code that failed to compile under cargo test --workspace (libraries already built; only test/example targets were broken, and it predated the provider work).

Fixes

  • clawforge-memory (store.rs tests): VectorEntry now needs created_at + session_id; MemoryQuery gained fields - test literals updated (..Default::default() where the type derives Default).
  • clawforge-scheduler (scheduler.rs test fixture): added allowed_skills to the test AgentSpec.
  • clawforge-cli examples (coding_agent, github_pr_reviewer, research_agent): added allowed_skills to AgentSpec, ..Default::default() to Capabilities literals, fixed get_recent_runs(n) -> get_recent_runs(n, offset), and moved VectorEntry/MemoryQuery imports to clawforge_memory::types.
  • .gitignore: anchored the local tooling-dir rules (/memory/, /.claude/) to the repo root - the unanchored memory/ pattern was shadowing the tracked backend/memory crate.

Verification

cargo test --workspace  ->  185 passed; 0 failed

cargo build --workspace also succeeds.

@YASSERRMD YASSERRMD merged commit e9dcbe7 into main Jun 9, 2026
0 of 4 checks passed
@YASSERRMD YASSERRMD deleted the fix_workspace_tests branch June 9, 2026 17:27
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