Skip to content

test: raise gemini API client coverage to 90%#4

Merged
Patrik652 merged 1 commit intomainfrom
002-claude-code-vision
Feb 13, 2026
Merged

test: raise gemini API client coverage to 90%#4
Patrik652 merged 1 commit intomainfrom
002-claude-code-vision

Conversation

@Patrik652
Copy link
Owner

Summary

  • expand tests/unit/test_gemini_api_client.py to cover config-key loading, validation/refresh behavior, model init path, and runtime error mapping
  • harden Gemini error mapping with case-insensitive invalid-key detection
  • keep existing API behavior while improving typed exception mapping coverage

Test Plan

  • PYTHONPATH=. .venv/bin/ruff check src/services src/cli tests/unit/test_gemini_api_client.py
  • PYTHONPATH=. .venv/bin/mypy src/services src/cli
  • PYTHONPATH=. .venv/bin/pytest -q

Result

  • 349 passed, 0 skipped, 2 warnings
  • total coverage: 83%
  • src/services/gemini_api_client.py: 90%

@Patrik652 Patrik652 merged commit 906a30f into main Feb 13, 2026
1 of 6 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