From 3627acd81e39d64b2a7df777cac4f130a9b88743 Mon Sep 17 00:00:00 2001 From: Daniel Chalef <131175+danielchalef@users.noreply.github.com> Date: Wed, 5 Nov 2025 10:32:10 -0800 Subject: [PATCH] Enable FalkorDB fulltext search tests (#1050) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Testing revealed that all FalkorDB fulltext search tests pass successfully. The skip was added in PR #872 without explanation and appears to be unnecessary. Removed FalkorDB from the skip conditions, keeping only Kuzu skipped (which legitimately doesn't support dynamic fulltext index creation). Tests verified passing locally with FalkorDB 1.2.3. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude --- tests/test_graphiti_mock.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/test_graphiti_mock.py b/tests/test_graphiti_mock.py index 62a8ffd3..a30c9392 100644 --- a/tests/test_graphiti_mock.py +++ b/tests/test_graphiti_mock.py @@ -918,8 +918,8 @@ async def test_get_communities_by_nodes(graph_driver, mock_embedder): async def test_edge_fulltext_search( graph_driver, mock_embedder, mock_llm_client, mock_cross_encoder_client ): - if graph_driver.provider in [GraphProvider.FALKORDB, GraphProvider.KUZU]: - pytest.skip('Skipping as fulltext indexing not supported for FalkorDB and Kuzu') + if graph_driver.provider == GraphProvider.KUZU: + pytest.skip('Skipping as fulltext indexing not supported for Kuzu') graphiti = Graphiti( graph_driver=graph_driver, @@ -1307,8 +1307,8 @@ async def test_edge_bfs_search(graph_driver, mock_embedder): async def test_node_fulltext_search( graph_driver, mock_embedder, mock_llm_client, mock_cross_encoder_client ): - if graph_driver.provider in [GraphProvider.FALKORDB, GraphProvider.KUZU]: - pytest.skip('Skipping as fulltext indexing not supported for FalkorDB and Kuzu') + if graph_driver.provider == GraphProvider.KUZU: + pytest.skip('Skipping as fulltext indexing not supported for Kuzu') graphiti = Graphiti( graph_driver=graph_driver, @@ -1516,8 +1516,8 @@ async def test_node_bfs_search(graph_driver, mock_embedder): async def test_episode_fulltext_search( graph_driver, mock_embedder, mock_llm_client, mock_cross_encoder_client ): - if graph_driver.provider in [GraphProvider.FALKORDB, GraphProvider.KUZU]: - pytest.skip('Skipping as fulltext indexing not supported for FalkorDB and Kuzu') + if graph_driver.provider == GraphProvider.KUZU: + pytest.skip('Skipping as fulltext indexing not supported for Kuzu') graphiti = Graphiti( graph_driver=graph_driver, @@ -1567,8 +1567,8 @@ async def test_episode_fulltext_search( async def test_community_fulltext_search( graph_driver, mock_embedder, mock_llm_client, mock_cross_encoder_client ): - if graph_driver.provider in [GraphProvider.FALKORDB, GraphProvider.KUZU]: - pytest.skip('Skipping as fulltext indexing not supported for FalkorDB and Kuzu') + if graph_driver.provider == GraphProvider.KUZU: + pytest.skip('Skipping as fulltext indexing not supported for Kuzu') graphiti = Graphiti( graph_driver=graph_driver,