Skip to content

Conversation

@PPeitsch
Copy link
Owner

Description

Increases test coverage from 75% to 98% by adding comprehensive unit tests for all modules.

Changes

New Test Files

  • est_init_db.py: 15 tests for env parsing and db info extraction
  • est_init_data.py: 8 tests for data seeding functions
  • est_import_log.py: 18 tests for import routes
  • est_pdf_importer.py: 19 tests for PDF parsing

Expanded Tests

  • est_importers.py: +15 Excel importer edge cases
  • est_routes.py: +7 time summary edge cases
  • est_utils.py: +2 validator tests

Configuration

  • Added .coveragerc to exclude init_db.py (interactive CLI script)
  • Updated Codecov integration with test-results-action
  • README: Dynamic Codecov badge

Type of Change

  • Test coverage

Testing

  • 156 tests passing
  • 98% coverage (excluding interactive init_db.py)
  • black, isort, mypy all pass

Checklist

  • I have followed the project's code style
  • My code generates no new warnings
  • I have added tests for new functionality
  • All tests pass locally

- Add test_init_db.py: 15 tests for env parsing, db info extraction
- Add test_init_data.py: 8 tests for data seeding functions
- Add test_import_log.py: 18 tests for import routes
- Add test_pdf_importer.py: 19 tests for PDF parsing
- Expand test_importers.py: +15 Excel importer edge cases
- Expand test_routes.py: +7 time summary edge cases
- Expand test_utils.py: +2 validator tests
- Update README coverage badge

Total: 156 tests passing, 86% coverage
- Add .coveragerc to exclude init_db.py from coverage
- Update README with dynamic Codecov badge
- Add junitxml output to pytest for test results
- Add codecov/test-results-action for better CI integration

Coverage now at 98% for app modules
@codecov
Copy link

codecov bot commented Dec 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@PPeitsch PPeitsch merged commit a62d63b into main Dec 16, 2025
2 checks passed
@PPeitsch PPeitsch deleted the feature/issue-29-test-coverage branch December 16, 2025 01:44
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.

2 participants