Move pure unit tests from database integration to unit test job
The reviewer correctly identified that test_bulk_utils.py, test_edge_operations.py, and test_node_operations.py are pure unit tests using only mocks - they don't require database connections. Changes: - Removed tests/utils/maintenance/ from ignore list (too broad) - Added specific ignore for test_temporal_operations_int.py (true integration test) - Moved test_bulk_utils.py, test_edge_operations.py, test_node_operations.py to unit tests - Kept test_graphiti_mock.py in database integration (uses real graph_driver fixture) This reduces database integration test time and properly categorizes tests. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
a7814d52ad
commit
3223e09452
1 changed files with 2 additions and 2 deletions
4
.github/workflows/unit_tests.yml
vendored
4
.github/workflows/unit_tests.yml
vendored
|
|
@ -40,7 +40,7 @@ jobs:
|
|||
--ignore=tests/test_entity_exclusion_int.py \
|
||||
--ignore=tests/driver/ \
|
||||
--ignore=tests/llm_client/test_anthropic_client_int.py \
|
||||
--ignore=tests/utils/maintenance/ \
|
||||
--ignore=tests/utils/maintenance/test_temporal_operations_int.py \
|
||||
--ignore=tests/cross_encoder/test_bge_reranker_client_int.py \
|
||||
--ignore=tests/evals/
|
||||
|
||||
|
|
@ -100,7 +100,7 @@ jobs:
|
|||
FALKORDB_PORT: 6379
|
||||
DISABLE_NEPTUNE: 1
|
||||
run: |
|
||||
uv run pytest tests/test_graphiti_mock.py tests/utils/maintenance/test_bulk_utils.py tests/utils/maintenance/test_edge_operations.py tests/utils/maintenance/test_node_operations.py
|
||||
uv run pytest tests/test_graphiti_mock.py
|
||||
|
||||
api-integration-tests:
|
||||
runs-on: depot-ubuntu-22.04
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue