From 22ed0ec2987965071c4ad7909b4bedcc38586f2c Mon Sep 17 00:00:00 2001 From: Daniel Chalef <131175+danielchalef@users.noreply.github.com> Date: Wed, 5 Nov 2025 09:59:12 -0800 Subject: [PATCH] Enable FalkorDB fulltext search tests 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,