Alan-Jowett / CoPilot-For-Consensus / 20213945061
78%

DEFAULT BRANCH: main
Ran
Jobs 6
Files 16
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 83.469% (-8.9%) from 92.336%
20213945061

push

github

web-flow 
Refactor DocumentStore to use exceptions instead of bool return values (#168)

* Initial plan

* Refactor DocumentStore to raise exceptions instead of returning bool/None for errors (Phase 1: Adapter Storage)

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Update embedding service test mock to match new DocumentStore API

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix redundant exception handlers in MongoDocumentStore

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Re-raise original exception in parsing main instead of wrapping in ConnectionError

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback: add exc_info=True to logger.error calls, handle specific exceptions, remove unused import

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix CI failure: add skip markers for tests requiring optional copilot_schema_validation dependency

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix integration tests: import DocumentNotFoundError at module level instead of inline

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* fix(storage): handle InvalidId exception in mongo_document_store

Catch bson.errors.InvalidId when converting document IDs to ObjectId in
get_document, update_document, and delete_document methods. This ensures
proper behavior when invalid ObjectId strings are passed:
- get_document returns None
- update_document raises DocumentNotFoundError
- delete_document raises DocumentNotFoundError

Fixes 3 failing MongoDB integration tests:
- test_update_nonexistent_document
- test_delete_nonexistent_document
- test_invalid_document_id

* fix(services): connect document store in reporting and chunking services

Both reporting and chunking services were creating document stores but
never calling connect() on ... (continued)

1025 of 1228 relevant lines covered (83.47%)

0.83 hits per line

Jobs
ID Job ID Ran Files Coverage
1 error-reporting - 20213945061.1 2
43.4
 GitHub Action Run
2 parsing - 20213945061.2 6
81.9
 GitHub Action Run
3 ingestion - 20213945061.3 2
80.99
 GitHub Action Run
4 orchestrator - 20213945061.4 2
84.52
 GitHub Action Run
5 summarization - 20213945061.5 2
92.9
 GitHub Action Run
6 embedding - 20213945061.6 2
94.97
 GitHub Action Run
Source Files on build 20213945061