Skip to content

test: replace remaining contract placeholders with executable suites#1

Merged
Patrik652 merged 2 commits intomainfrom
002-claude-code-vision
Feb 13, 2026
Merged

test: replace remaining contract placeholders with executable suites#1
Patrik652 merged 2 commits intomainfrom
002-claude-code-vision

Conversation

@Patrik652
Copy link
Owner

Summary

  • replace tests/contract/test_vision_service.py with executable contract tests using deterministic doubles
  • replace tests/contract/test_claude_api_client.py with executable Anthropic client contract tests using mocked API responses
  • remove the last placeholder-driven contract skips and keep behavior covered by runnable tests

Test Plan

  • PYTHONPATH=. .venv/bin/ruff check src/services src/cli tests/contract/test_vision_service.py tests/contract/test_claude_api_client.py
  • PYTHONPATH=. .venv/bin/mypy src/services src/cli
  • PYTHONPATH=. .venv/bin/pytest -q

Result

  • 317 passed, 0 skipped, 2 warnings

@Patrik652 Patrik652 merged commit f958f44 into main Feb 13, 2026
2 of 7 checks passed
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