ed50a9c0d1
fix: Remove unused imports from test files
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:40:31 +00:00
882634cbb4
docs: Add comprehensive coverage report
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:37:40 +00:00
089e56f0ab
test: Increase coverage for url_upload.py and files.py to 90%+
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:36:32 +00:00
8c91d7f504
docs: Add comprehensive test coverage improvements documentation
...
Document the significant coverage improvements achieved:
- upload_to_google_drive.py: 77.22% → 98.73% (+21.51%)
- status.py: 77.46% → 89.47% (+12.01%)
Includes detailed analysis of:
- Tests added for each module
- Coverage metrics before/after
- Testing methodology
- Remaining edge cases
- Recommendations for future work
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:35:23 +00:00
14d7d1b4ef
docs: add visual guide for v1.1.0 web clipping feature
...
- Created comprehensive visual guide with ASCII diagrams
- Renamed old visual guide to VISUAL_GUIDE_V1.0.md
- Added UI mockups, data flow diagrams, and use cases
- Documented feature comparison and browser compatibility
- Added security model visualization
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:34:31 +00:00
1b6c503f23
test: Add comprehensive tests for upload_to_google_drive and status views
...
- upload_to_google_drive.py: 98.73% coverage (target: 90%, achieved!)
- status.py: 89.47% coverage (target: 90%, very close!)
New tests added:
- Generic exception handling in OAuth
- Metadata truncation error handling
- Upload without metadata flag
- Docker environment detection edge cases
- Git SHA null/unknown handling
- Complete exception fallback handling
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:33:25 +00:00
2b63593367
docs: add comprehensive implementation summary for v1.1.0
...
- Created IMPLEMENTATION_SUMMARY.md documenting web clipping feature
- Renamed old summary to IMPLEMENTATION_SUMMARY_V1.0.md
- Documented all features, testing, and acceptance criteria
- Added future enhancements section
- Comprehensive documentation of changes and architecture
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:32:59 +00:00
dd2608bcb0
refactor: optimize selection capture and remove dead code
...
- Removed Node.js export from capture.js (browser-only code)
- Simplified selection capture to avoid expensive getComputedStyle() calls
- Added performance notes for selection clipping
- Aligned content.js with capture.js implementation
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:31:06 +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
7fc8dd655a
fix: address code review feedback for browser extension
...
- Removed unused variables in capture.js (clonedDoc, styles array)
- Added explicit return values for message handlers in content.js
- Added detailed comment explaining 500ms render delay before PDF conversion
- Updated PERMISSIONS.md with comprehensive explanation of host_permissions requirement
- Updated test.html with web clipping test scenarios
- Improved code consistency and documentation
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:27:57 +00:00
b2badd0429
style: fix import ordering in test_auth.py
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:26:53 +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
5380c6de6c
docs: update browser extension documentation for web clipping
...
- Updated browser-extension/README.md with web clipping features
- Updated docs/BrowserExtension.md with dual-mode architecture
- Added documentation for clip mode data flow and API endpoints
- Updated permissions explanation for new clipping capabilities
- Added troubleshooting for clip-specific issues
- Documented version 1.1.0 features and changes
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:24:10 +00:00
4221abd991
feat(tests): increase test coverage for dropbox.py and auth.py to >90%
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:23:35 +00:00
f7280be523
feat(browser-extension): add web page clipping functionality
...
- Updated manifest to v1.1.0 with additional permissions (scripting, host_permissions)
- Added web clipping context menu items (Clip Full Page, Clip Selection)
- Enhanced background.js with PDF conversion using Chrome's printToPDF API
- Updated content.js to capture full page HTML and selected content
- Added new capture.js script for page capture utilities
- Enhanced popup UI with mode toggle between "Send URL" and "Clip Page"
- Added support for clipping full pages or selected content to PDF
- Updated CSS for new mode buttons and clip section layout
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:21:08 +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
a2fee4d632
Add tests for google_drive.py - achieved 85.61% coverage (up from 81.55%)
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:18:56 +00: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
1eb3473e52
Add comprehensive tests for settings_service.py - achieved 100% coverage
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:07:49 +00:00
5ccdc79e77
Initial plan for increasing test coverage
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-02-13 23:03:28 +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
8d91b135a0
Initial plan
2026-02-13 23:01:24 +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
3b92bc75a2
Initial plan
2026-02-13 22:59:33 +00:00
0bb3c45d0f
Initial plan
2026-02-13 22:59:26 +00:00
07d4bc456d
Initial plan
2026-02-13 22:59:18 +00:00
f56ecd8040
Initial plan
2026-02-13 22:58:51 +00:00
eb535c9d0f
Initial plan
2026-02-13 22:58:43 +00:00
3a17739f06
Initial plan
2026-02-13 22:58:37 +00:00
18866aaf32
Initial plan
2026-02-13 22:58:33 +00:00
cec93feb04
Initial plan
2026-02-13 22:58:27 +00:00
e96af1e91f
Initial plan
2026-02-13 22:58:21 +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
70bfd3e489
chore(release): update build metadata files [skip ci]
2026-02-13 22:35:05 +00:00
copilot-swe-agent[bot]