From 3223e09452dd4836257ea95170b3fa9953ee8152 Mon Sep 17 00:00:00 2001 From: Daniel Chalef <131175+danielchalef@users.noreply.github.com> Date: Sat, 11 Oct 2025 19:23:16 -0700 Subject: [PATCH] Move pure unit tests from database integration to unit test job MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- .github/workflows/unit_tests.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/unit_tests.yml b/.github/workflows/unit_tests.yml index 8c70ee74..35b811a3 100644 --- a/.github/workflows/unit_tests.yml +++ b/.github/workflows/unit_tests.yml @@ -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