Alan-Jowett / CoPilot-For-Consensus / 21423794427

78%

push

github

Add per-type container routing for Cosmos DB document store (#1029)

* Initial plan

* Implement container routing in AzureCosmosDocumentStore

- Add container_routing_mode parameter (legacy/per_type)
- Update schema and regenerate typed configuration
- Implement helper methods for container routing
- Update all CRUD operations to use routing logic
- Support both legacy single-container and per-type multi-container modes

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

* Add comprehensive unit tests for container routing

- Test configuration and initialization
- Test container config helpers for all routing modes
- Test connect() behavior in both modes
- Test CRUD operations in both modes
- Test that legacy mode is backward compatible
- All new tests passing (21/21)
- All existing tests still passing (62/63, 1 unrelated failure)

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

* Add documentation for container routing feature

Added comprehensive documentation covering:
- Feature overview and motivation
- Configuration options
- Container mapping strategy
- Benefits and use cases
- Implementation details
- Testing instructions
- Migration path

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

* Address code review feedback

- Remove unused _get_partition_key_value() method
- Fix documentation placeholder for issue number
- All tests still passing (21/21)

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

* Address PR review feedback

- Add SPDX/MIT header to cosmos-container-routing.md
- Remove hard-coded test counts from documentation
- Remove unused doc_id variables in tests (lines 218, 242)
- Add AzureError exception handling in _get_container_for_collection()
- Add missing import in documentation code example
- Fix CLI example to use SQL API instead of MongoDB API
- Remove unused imports from test file
- Add per_type mode tests for get... (continued)

66 of 81 new or added lines in 1 file covered. (81.48%)

8705 of 11186 relevant lines covered (77.82%)

0.87 hits per line