llm-multiverse/llm-multiverse
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: deduplicate make_params test helper (#224) #227

Merged
shahondin1624 merged 1 commits from refactor/issue-224-dedup-make-params into main 2026-03-11 13:30:19 +01:00
Conversation 0 Commits 1 Files Changed 7 +18 -42
shahondin1624 commented 2026-03-11 13:30:13 +01:00
Owner
Copy Link

Summary

  • Extract make_params() test helper into test_util module in tool-broker crate root
  • Replace 6 identical copies across dispatch, fs_read, fs_write, run_code, run_shell, and package_install

Test plan

  • cargo test -p tool-broker — all 313 tests pass

🤖 Generated with Claude Code

## Summary - Extract `make_params()` test helper into `test_util` module in tool-broker crate root - Replace 6 identical copies across dispatch, fs_read, fs_write, run_code, run_shell, and package_install ## Test plan - [x] `cargo test -p tool-broker` — all 313 tests pass 🤖 Generated with [Claude Code](https://claude.com/claude-code)
shahondin1624 added 1 commit 2026-03-11 13:30:14 +01:00
refactor: deduplicate make_params test helper in tool-broker (issue #224) 7e3b182109 
Extract shared make_params() into crate-level test_util module and replace
6 identical copies across dispatch, fs_read, fs_write, run_code, run_shell,
and package_install test modules.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
shahondin1624 merged commit d90361b21b into main 2026-03-11 13:30:19 +01:00
shahondin1624 deleted branch refactor/issue-224-dedup-make-params 2026-03-11 13:30:19 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
cat:agent
Agent related
 cat:cross-cutting
Cross-cutting concern
 cat:deployment
Deployment related
 cat:devops
DevOps related
 cat:performance
Performance related
 cat:security
Security related
 cat:tech-debt
Technical debt from code reviews
 lang:protobuf
Protocol Buffers
 lang:python
Python language
 lang:rust
Rust language
 priority:critical
Critical priority
 priority:high
High priority
 priority:low
Low priority
 priority:medium
Medium priority
 service:audit
Audit service
 service:memory
Memory service
 service:model-gateway
Model Gateway service
 service:orchestrator
Orchestrator service
 service:search
Search service
 service:secrets
Secrets service
 service:tool-broker
Tool Broker service
 type:bug
Bug fix
 type:documentation
Docs
 type:feature
New functionality
 type:infrastructure
Build tooling, project setup
 type:refactor
Restructuring
 type:research
Investigation, benchmarking
 type:testing
Tests
 workflow:blocked
Blocked after max retries
 workflow:manual
Requires manual intervention — blocked from auto-dev
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
shahondin1624
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: llm-multiverse/llm-multiverse#227
Delete Branch "refactor/issue-224-dedup-make-params"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?