Skip to content

fix: address low-priority coverage and config issues #118-#121#136

Merged
longieirl merged 2 commits intomainfrom
fix/low-priority-issues-118-121
Apr 9, 2026
Merged

fix: address low-priority coverage and config issues #118-#121#136
longieirl merged 2 commits intomainfrom
fix/low-priority-issues-118-121

Conversation

@longieirl
Copy link
Copy Markdown
Owner

Summary

Changes

  • `pyproject.toml`: add `if TYPE_CHECKING:` to coverage `exclude_lines`
  • `tests/test_processor_refactored_methods.py`: new `TestCoverageGaps` class with 3 tests
  • `tests/services/test_extraction_orchestrator.py`: 1 new test for template init failure

Type

  • Bug fix

Testing

  • Tests pass (coverage >= 91%)
  • Manually tested
  • `make docker-integration` passed locally

Checklist

  • Code follows project style
  • Self-reviewed
  • Documentation updated (if needed)
  • No new warnings

Downstream impact

  • This PR changes a public interface in `bankstatements_core`

Closes #118, #119, #120, #121

@longieirl longieirl self-assigned this Apr 9, 2026
@github-actions github-actions bot added the bug Something isn't working label Apr 9, 2026
@longieirl longieirl merged commit 962cf9b into main Apr 9, 2026
10 checks passed
@longieirl longieirl deleted the fix/low-priority-issues-118-121 branch April 9, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test coverage: processor.py line 284 (set_duplicate_detection_strategy)

2 participants