Alan-Jowett / CoPilot-For-Consensus / 20763756154 / 17
81%
main: 81%

DEFAULT BRANCH: main
Ran
Files 9
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: 67.49%. Remained the same
20763756154.17

push

github

web-flow 
Fix source deletion: Cosmos DB document ID resolution and logger signature (#776)

* Initial plan

* Fix logger calls and document store operations for source management

- Fixed all logger.error calls in api.py to use keyword arguments instead of printf-style formatting
- Fixed delete_source() to query for document ID before deleting (Cosmos SDK compatibility)
- Fixed update_source() to query for document ID before updating (Cosmos SDK compatibility)

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

* Remove xfail markers from source update/delete tests

Tests should now pass with the fixed delete_source() and update_source() implementations

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

* Remove redundant queries in delete_source and update_source

Removed duplicate get_source() calls that were querying the database unnecessarily before the actual query to get the document ID

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

* Use explicit None checks for document ID fallback

Changed from 'or' operator to explicit None check to avoid issues with falsy values like empty strings or 0

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

* Extract document ID lookup into helper method

Created _get_source_doc_id() helper to eliminate code duplication and avoid duplicate dictionary lookups in update_source() and delete_source()

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

* Simplify ID extraction and fix error handling

- Simplified doc_id extraction to avoid duplicate get() calls
- Added explicit ValueError re-raise to prevent double-wrapping error messages

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

* Fix ID extraction and remove redundant str() conversions

- Use explicit None checks in ID extraction to preserve falsy but valid IDs (e.g., 0, "")
- Remove redundant str() conv... (continued)

164 of 243 relevant lines covered (67.49%)

0.67 hits per line

Source Files on job copilot_archive_fetcher - 20763756154.17