Add test coverage for recent PRs (#56, #57, #58)

Tinker

Home Epics Add test coverage for recent PRs (#56, #57, #58)Edit

Cancel

Title *

Project *

Description

## Overview

Several recent PRs were merged without full Rails test coverage verification. This ticket adds missing tests to ensure code quality and prevent regressions.

## Missing Tests by PR

### PR #56 (Remove Artifact/CodeDiff systems)
- Verify no broken references to removed models
- Check model, controller, view specs for stale references
- Run full `bundle exec rspec` and fix failures

### PR #57 (assign_ticket & list_members MCP tools)
- Add specs for `app/controllers/api/v1/mcp_controller.rb`:
  - `handle_list_members` - test filtering by role, active status
  - `handle_assign_ticket` - test:
    - Self-assignment (member_id omitted)
    - Assignment to specific member
    - Assignment + status transition
    - Validation: same project, active agent, claimable statuses
    - Error cases

### PR #58 (Data model simplification)
- Verify no broken references to Artifact/CodeDiff models
- Check `auditor_service_spec.rb` for removed dependencies
- Run full test suite

## Acceptance Criteria

1. All new MCP handler code has spec coverage
2. Full test suite passes: `bundle exec rspec`
3. No references to removed Artifact/CodeDiff models in tests
4. At least 80% code coverage for `mcp_controller.rb`

## Files to Modify

- `spec/controllers/api/v1/mcp_controller_spec.rb` (add tests)
- Any test files with broken references to removed models
- Consider adding `spec/services/auditor_service_spec.rb` if missing