* Prepare code * Fix tests * As -> AS, remove trailing spaces * Enable more tests for FalkorDB * Fix more cypher queries * Return all created nodes and edges * Add Neo4j service to unit tests workflow - Introduced Neo4j as a service in the GitHub Actions workflow for unit tests. - Configured Neo4j with appropriate ports, authentication, and health checks. - Updated test steps to include waiting for Neo4j and running integration tests against it. - Set environment variables for Neo4j connection in both non-integration and integration test steps. * Update Neo4j authentication in unit tests workflow - Changed Neo4j authentication password from 'test' to 'testpass' in the GitHub Actions workflow. - Updated health check command to reflect the new password. - Ensured consistency across all test steps that utilize Neo4j credentials. * fix health check * Fix Neo4j integration tests in CI workflow Remove reference to non-existent test_neo4j_driver.py file from test command. Integration tests now run via parametrized tests using the drivers list. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Add OPENAI_API_KEY to Neo4j integration tests Neo4j integration tests require OpenAI API access for LLM functionality. Add the secret environment variable to enable these tests to run properly. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Fix Neo4j Cypher syntax error in BFS search queries Replace parameter substitution in relationship pattern ranges (*1..$depth) with direct string interpolation (*1..{bfs_max_depth}). Neo4j doesn't allow parameter maps in MATCH pattern ranges - they must be literal values. Fixed in both node_bfs_search and edge_bfs_search functions. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Fix variable name mismatch in edge_bfs_search query Change relationship variable from 'r' to 'e' to match ENTITY_EDGE_RETURN constant expectations. The ENTITY_EDGE_RETURN constant references variable 'e' for relationships, but the query was using 'r', causing "Variable e not defined" errors. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Isolate database tests in CI workflow - FalkorDB tests: Add DISABLE_NEO4J=1 and remove Neo4j env vars - Neo4j tests: Keep current setup without DISABLE_NEO4J flag This ensures proper test isolation where each test suite only runs against its intended database backend. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Siddhartha Sahu <sid@kuzudb.com> Co-authored-by: Claude <noreply@anthropic.com>
78 lines
2.4 KiB
YAML
78 lines
2.4 KiB
YAML
name: Unit Tests
|
|
|
|
on:
|
|
push:
|
|
branches: [main]
|
|
pull_request:
|
|
branches: [main]
|
|
|
|
permissions:
|
|
contents: read
|
|
|
|
jobs:
|
|
test:
|
|
runs-on: depot-ubuntu-22.04
|
|
environment:
|
|
name: development
|
|
services:
|
|
falkordb:
|
|
image: falkordb/falkordb:latest
|
|
ports:
|
|
- 6379:6379
|
|
options: --health-cmd "redis-cli ping" --health-interval 10s --health-timeout 5s --health-retries 5
|
|
neo4j:
|
|
image: neo4j:5.26-community
|
|
ports:
|
|
- 7687:7687
|
|
- 7474:7474
|
|
env:
|
|
NEO4J_AUTH: neo4j/testpass
|
|
NEO4J_PLUGINS: '["apoc"]'
|
|
options: --health-cmd "cypher-shell -u neo4j -p testpass 'RETURN 1'" --health-interval 10s --health-timeout 5s --health-retries 10
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
- name: Set up Python
|
|
uses: actions/setup-python@v5
|
|
with:
|
|
python-version: "3.10"
|
|
- name: Install uv
|
|
uses: astral-sh/setup-uv@v3
|
|
with:
|
|
version: "latest"
|
|
- name: Install redis-cli for FalkorDB health check
|
|
run: sudo apt-get update && sudo apt-get install -y redis-tools
|
|
- name: Install dependencies
|
|
run: uv sync --all-extras
|
|
- name: Run non-integration tests
|
|
env:
|
|
PYTHONPATH: ${{ github.workspace }}
|
|
NEO4J_URI: bolt://localhost:7687
|
|
NEO4J_USER: neo4j
|
|
NEO4J_PASSWORD: testpass
|
|
run: |
|
|
uv run pytest -m "not integration"
|
|
- name: Wait for FalkorDB
|
|
run: |
|
|
timeout 60 bash -c 'until redis-cli -h localhost -p 6379 ping; do sleep 1; done'
|
|
- name: Wait for Neo4j
|
|
run: |
|
|
timeout 60 bash -c 'until wget -O /dev/null http://localhost:7474 >/dev/null 2>&1; do sleep 1; done'
|
|
- name: Run FalkorDB integration tests
|
|
env:
|
|
PYTHONPATH: ${{ github.workspace }}
|
|
FALKORDB_HOST: localhost
|
|
FALKORDB_PORT: 6379
|
|
DISABLE_NEO4J: 1
|
|
run: |
|
|
uv run pytest tests/driver/test_falkordb_driver.py
|
|
- name: Run Neo4j integration tests
|
|
env:
|
|
PYTHONPATH: ${{ github.workspace }}
|
|
NEO4J_URI: bolt://localhost:7687
|
|
NEO4J_USER: neo4j
|
|
NEO4J_PASSWORD: testpass
|
|
FALKORDB_HOST: localhost
|
|
FALKORDB_PORT: 6379
|
|
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
|
|
run: |
|
|
uv run pytest tests/test_*_int.py -k "neo4j"
|