Skip to content

test: increase configuration manager coverage to 91%#3

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

test: increase configuration manager coverage to 91%#3
Patrik652 merged 1 commit intomainfrom
002-claude-code-vision

Conversation

@Patrik652
Copy link
Owner

Summary

  • add tests/unit/test_config_manager.py with comprehensive unit coverage for ConfigurationManager
  • cover load behavior (missing/empty/invalid YAML, partial merges)
  • cover save/round-trip and validation error surfaces

Test Plan

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

Result

  • 339 passed, 0 skipped, 2 warnings
  • total coverage: 81%
  • src/services/config_manager.py: 91%

@Patrik652 Patrik652 merged commit 9fd7352 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