awslabs / awslabs/graphrag-toolkit

[FEATURE] Improve test coverage: storage error handling and multi-tenant isolation

Open
#426 0 comments 0 reactions 1 assignee Claimed by @noel-improv View on GitHub
good first issue
Dominant language
Python
Stars
442
Forks
106
Avg merge
2d 3h
Merged PRs (30d)
41

Description

### Package

lexical-graph

### Problem statement

OpenSearch error handling has minimal test coverage (only NotFoundError retry tested). Neptune error tests already exist in `test_neptune_graph_stores.py`. Checkpoint, cascading deletion, and multi-tenant vector store have no error-path tests.

Related: #408

### Proposed solution

- **OpenSearch**: connection timeout, bulk insert partial failure, auth failure (403), index_exists 404 handling
- **Checkpoint**: missing directory, unreadable file (PermissionError), read-only filesystem
- **delete_sources**: error mid-cascade propagation, empty source_ids list
- **Multi-tenant**: query/write with wrong tenant scoped correctly, no cross-tenant leakage

### Alternatives considered

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.