Skip to content

test: raise temp file manager coverage to 92%#5

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

test: raise temp file manager coverage to 92%#5
Patrik652 merged 1 commit intomainfrom
002-claude-code-vision

Conversation

@Patrik652
Copy link
Owner

Summary

  • add tests/unit/test_temp_file_manager.py
  • cover create/cleanup lifecycle, disabled-cleanup behavior, tracked+orphan cleanup, aggregated error handling, and destructor cleanup path
  • improve confidence in temp file lifecycle semantics across services

Test Plan

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

Result

  • 360 passed, 0 skipped, 2 warnings
  • total coverage: 85%
  • src/services/temp_file_manager.py: 92%

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