9df63af4d0
Merge pull request #107 from christianlouis/copilot/fix-missing-model-imports
...
Fix test database initialization and dependency injection
2026-02-07 19:51:06 +01:00
547b9513a5
Merge pull request #108 from christianlouis/copilot/fix-s3-upload-error
...
Fix upload failures: standardize task signatures to accept file_id parameter
2026-02-07 19:45:42 +01:00
dabd9a944c
Final review fixes: consistent parameter order and improved test documentation
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:59:42 +00:00
3d2d8cca1a
Add comprehensive regression tests for all upload tasks
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:56:47 +00:00
4c8ec85bfa
Apply Black formatting to test file
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:54:29 +00:00
388bde2ff0
Fix SQLAlchemy model registration and database session injection in tests
...
- Import all models (DocumentMetadata, FileRecord, ProcessingLog) in conftest.py to register them with SQLAlchemy Base
- Override all three get_db functions used across the app (app.database, app.api.common, app.views.base) to ensure tests use the test database
- Fixes 21 of 24 failing tests (from "no such table" errors to passing)
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:54:05 +00:00
77678f6368
Fix single file deletion frontend to parse JSON response properly
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:53:12 +00:00
7d24b7c668
Apply code formatting and fix linting issues
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:37:50 +00:00
2db58afedf
Add original_filename parameter to preserve user's filename
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:36:36 +00:00
3f1006b035
Code review feedback: improve readability of S3 upload call
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:28:04 +00:00
c0e049ae96
Fix OneDrive and S3 upload signature - add file_id parameter
...
- Updated upload_to_onedrive to accept file_id parameter with bind=True
- Updated upload_to_s3 to accept file_id parameter with bind=True
- Added proper logging with task_id and file_id tracking
- Added comprehensive unit tests for both functions
- All tests passing (8/8)
Fixes #99 and #100
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 17:25:23 +00:00
c9af116ae9
Merge pull request #90 from christianlouis/copilot/fix-worker-file-processing
...
Fix DetachedInstanceError in process_document task
2026-02-07 18:09:45 +01:00
850afd26bd
Fix DetachedInstanceError in process_document task by storing file_id before session closes
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:59:39 +00:00
c162657987
Add tests for bulk operations and status filtering
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:53:34 +00:00
f2a303b6ee
Improve test code quality based on code review feedback
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:40:22 +00:00
f3ce81a6cf
Fix 'min' undefined error in /files view by adding min/max to Jinja2 globals
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:35:54 +00:00
b76fb147cb
Add tests for file listing and pagination features
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:15:30 +00:00
0de31fc775
Refactor tests to use fixture for better maintainability
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 15:03:17 +00:00
8eea420bc5
Add throttling to /processall endpoint to prevent overwhelming downstream APIs
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-07 14:59:19 +00:00
8be5b844c3
Add file processing notifications feature
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-06 22:31:48 +00:00
f0f48b39a9
Add security fixes, testing infrastructure, and CI/CD improvements
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-06 21:55:19 +00:00
b5c74dca4e
Add unit tests for utils.hash_file function
2025-03-26 16:27:36 +00:00
Christian Krakau-Louis