Alan-Jowett / CoPilot-For-Consensus / 21308345625 / 5
78%
main: 78%

DEFAULT BRANCH: main
Ran
Files 8
Run time 0s
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: 74.085%. First build
21308345625.5

push

github

web-flow 
Standardize duplicate document handling across all document store adapters (#1016)

* Initial plan

* Add DocumentAlreadyExistsError exception and update handlers

- Add DocumentAlreadyExistsError exception to document_store module
- Update Azure Cosmos DB adapter to raise DocumentAlreadyExistsError instead of generic DocumentStoreError
- Update parsing service to catch DocumentAlreadyExistsError and handle as duplicate
- Add tests for new exception handling in messages and threads

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

* Update Azure Cosmos DB test to expect DocumentAlreadyExistsError

- Update test_insert_document_resource_exists to expect DocumentAlreadyExistsError
- Import DocumentAlreadyExistsError in test file
- Update test docstring to reflect the new specific exception type

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

* Update MongoDB adapter to raise DocumentAlreadyExistsError

- Import DocumentAlreadyExistsError in MongoDocumentStore
- Catch pymongo DuplicateKeyError and raise DocumentAlreadyExistsError
- Update docstring to document new exception
- Maintains backward compatibility by handling pymongo not being installed

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

* Remove redundant duplicate key error fallback logic

- Remove _is_duplicate_key_error helper function (no longer needed)
- Remove fallback handling for raw DuplicateKeyError exceptions
- All duplicate detection now uses standardized DocumentAlreadyExistsError
- Update tests to use DocumentAlreadyExistsError instead of DuplicateKeyError
- Remove pymongo.errors.DuplicateKeyError import from tests

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

* Refactor MongoDB DuplicateKeyError handling to avoid nested try-catch

- Move pymongo.errors.DuplicateKeyError import to module level with try-except
- Replace nested try-catch in insert_document with simpl... (continued)

486 of 656 relevant lines covered (74.09%)

0.74 hits per line

Source Files on job copilot_vectorstore - 21308345625.5