f0dba98903
Merge pull request #324 from christianlouis/copilot/increase-test-coverage-utils-and-tasks
...
test: Increase coverage for imap_tasks.py (48% → 98%) and resolve config_validator.py shadowing
2026-02-14 01:13:10 +01:00
d0e85d0690
style: apply ruff auto-fix
...
- Auto-formatted code with ruff format
- Applied ruff linting fixes with --fix
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
2026-02-14 00:12:30 +00:00
6d61aee464
Merge pull request #322 from christianlouis/copilot/increase-test-coverage-gpt-file-splitter
...
test: increase coverage for extract_metadata_with_gpt.py and file_splitting.py to 90%+
2026-02-14 01:11:54 +01:00
d3011c7063
Merge pull request #326 from christianlouis/copilot/increase-test-coverage-files-process
...
test: Increase coverage for app/api/files.py and app/api/process.py
2026-02-14 01:10:36 +01:00
079b002dfb
Merge branch 'main' into copilot/increase-test-coverage-files-process
2026-02-14 01:10:23 +01:00
f4fbd0e523
Merge pull request #328 from christianlouis/copilot/fix-black-flake8-ci
...
docs: update linting documentation to reflect Ruff migration
2026-02-14 01:07:31 +01:00
6defc8ea7a
test: improve client initialization test assertion based on code review feedback
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-14 00:06:55 +00:00
60a2a59266
Merge pull request #325 from christianlouis/copilot/increase-test-coverage-logging-azure
...
Increase test coverage for logging and Azure API modules to 100%
2026-02-14 01:06:33 +01:00
fc515ebb70
style: apply ruff auto-fix
...
- Auto-formatted code with ruff format
- Applied ruff linting fixes with --fix
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
2026-02-14 00:05:57 +00:00
070ac21c1a
chore(release): update build metadata files [skip ci]
2026-02-14 00:05:28 +00:00
2f400fc2ec
Merge pull request #327 from christianlouis/copilot/increase-test-coverage
...
test: increase coverage for wizard.py and upload_to_nextcloud.py to 98%+
2026-02-14 01:05:09 +01:00
4903260496
style: fix trailing whitespace in test_file_splitting.py
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-14 00:04:26 +00:00
885eaff1bb
test: finalize coverage improvements with code review and security scan
...
- Code review: No issues found
- Security scan (CodeQL): No vulnerabilities detected
- All 75 tests passing
- Ready for merge
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-14 00:03:32 +00:00
f775cf740e
test: increase coverage for file_splitting.py to 100% and maintain extract_metadata_with_gpt.py at 92.31%
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-14 00:02:40 +00:00
d23e0044de
test: significant coverage improvements for app/api/files.py
...
- Add comprehensive exception handling tests for delete operations
- Add tests for bulk reprocess error handling
- Add comprehensive retry pipeline step tests (Azure OCR, GPT metadata, embed)
- Add retry upload task tests for various destinations
- Add additional file operation tests for edge cases
- Coverage increased from 60.76% to 76.39% (+15.63%)
- Combined with process.py at 100%, overall improvement is significant
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:56:48 +00:00
3e7702e39a
chore: normalize line endings in .gitignore
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:53:48 +00:00
f7eff20d8b
chore: add junit.xml to .gitignore
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:51:57 +00:00
7edd6ede3f
docs: verify CI workflow and all checks pass
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:51:30 +00:00
18a80314f2
docs: update AGENTIC_CODING.md to use Ruff instead of Black/Flake8
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:48:04 +00:00
ef66a07e27
fix: remove unused import from test_views_wizard.py
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:46:57 +00:00
37fe5b8068
test: achieve 100% coverage for app/api/process.py
...
- Add comprehensive tests for all process endpoints
- Test success paths for all upload destination endpoints
- Add throttling tests for processall endpoint
- Test boundary conditions and edge cases
- Coverage increased from 58.02% to 100%
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:46:15 +00:00
8de779b1aa
feat(tests): increase test coverage for wizard.py and upload_to_nextcloud.py
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:45:11 +00:00
fc4f5631ee
test: increase coverage for imap_tasks.py and fix config_validator.py
...
- Add comprehensive tests for imap_tasks.py covering:
- Lock acquire/release mechanisms
- pull_all_inboxes task with various scenarios
- pull_inbox for Gmail and non-Gmail with edge cases
- find_all_mail_xlist functionality
- Extended fetch_attachments tests for all MIME types
- Edge cases: invalid JSON, missing Message-ID, already processed, etc.
- Achieve 98.26% coverage for imap_tasks.py (up from 48.78%)
- Fix config_validator.py to include validate_auth_config export
- Update tests to verify all exports including validate_auth_config
- Note: config_validator.py file is shadowed by config_validator/ directory
in Python's module resolution, so it cannot be directly imported or tested.
The package's __init__.py (which has 100% coverage) is what's actually used.
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:42:49 +00:00
e0f3d69485
Initial commit: starting test coverage improvement for files.py and process.py
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:42:30 +00:00
331b3a3a1a
Fix code review issues - use PropertyMock instead of exec
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:28:26 +00:00
bebb26e593
Add comprehensive tests for logging.py - achieved 100% coverage
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:24:21 +00:00
d5d43dfda5
Merge pull request #314 from christianlouis/copilot/increase-test-coverage-app-files
...
test: increase coverage for database.py and upload_to_email.py
2026-02-14 00:19:21 +01:00
025510304a
chore(release): update build metadata files [skip ci]
2026-02-13 23:17:04 +00:00
77bc043a9d
Merge pull request #316 from christianlouis/copilot/increase-test-coverage-utils
...
test: increase coverage for validators.py (95.59%) and filename_utils.py (100%)
2026-02-14 00:16:47 +01:00
031f41ca8d
Merge pull request #317 from christianlouis/copilot/increase-test-coverage-celery-and-google-drive
...
test: increase coverage for celery_app.py and google_drive.py to 100%
2026-02-14 00:15:49 +01:00
6d9649609d
test: address code review feedback
...
- Rename test for clarity (fallback_to_builtin_template_when_custom_template_fails)
- Add MIME type validation for SVG logo test
- Use precise assertion for logo location check count
- Add column type validation in migration test
- All 39 tests pass
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:15:28 +00:00
97520a50a5
docs: add test coverage achievement summary
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:13:19 +00:00
cac568f59f
test: address code review feedback - add assertions and test for empty string folder_id
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:10:07 +00:00
fb73fe0935
test: add comprehensive tests for database.py and upload_to_email.py
...
- Added tests for database.py migration functions
- Added tests for error handling in init_db()
- Added tests for file path columns migration
- Added tests for unique index dropping
- Added tests for idempotent migrations
- Added tests for email template fallback logic
- Added tests for SVG logo attachment
- Added tests for SMTP without TLS and without auth
- Added tests for timeout errors in SMTP
- Added tests for upload_to_email task validation
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:10:04 +00:00
3fd354ecf7
style(tests): fix import ordering per ruff linter
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:09:51 +00:00
b1510f8a8f
feat(tests): increase coverage for validators.py (95.59%) and filename_utils.py (100%)
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:08:39 +00:00
a7f47b1bf0
test: increase coverage for celery_app.py and google_drive.py to 100%
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:07:54 +00:00
51fb85b545
Initial commit: Analyze test coverage gaps
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:02:39 +00:00
b4390c67e8
Initial plan
2026-02-13 23:00:48 +00:00
857e188241
Initial plan
2026-02-13 23:00:39 +00:00
928a86dee5
Initial plan
2026-02-13 23:00:29 +00:00
23ab5b2f41
Initial plan
2026-02-13 22:59:54 +00:00
770891fae5
Initial plan
2026-02-13 22:59:43 +00:00
0bb3c45d0f
Initial plan
2026-02-13 22:59:26 +00:00
18866aaf32
Initial plan
2026-02-13 22:58:33 +00:00
cec93feb04
Initial plan
2026-02-13 22:58:27 +00:00
dac2894c96
Initial plan
2026-02-13 22:58:17 +00:00
a05c5aee45
Merge pull request #299 from christianlouis/copilot/fix-linting-errors
...
style: fix ruff formatting in 6 files
2026-02-13 23:40:59 +01:00
e6a4995a5c
style: run ruff format on 6 files to fix formatting issues
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 22:39:34 +00:00
f8828db835
Initial plan
2026-02-13 22:38:00 +00:00
Christian Krakau-Louis