-
Notifications
You must be signed in to change notification settings - Fork 107
Staging #282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Staging #282
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
f6f200e
update deps
gkorland f5c400a
fix lint issues
gkorland d7a9c60
Merge pull request #281 from FalkorDB/deps-update
gkorland 1d310e9
Bump psycopg2-binary from 2.9.10 to 2.9.11
dependabot[bot] 21161ef
Merge pull request #285 from FalkorDB/dependabot/pip/staging/psycopg2…
gkorland 4646143
Migrate to react
Naseem77 3dd4dc6
Merge staging into migrate-to-react - resolved conflicts by keeping R…
Naseem77 4fa975b
Add mobile responsiveness
Naseem77 0220c57
fix lint
Naseem77 aa597a0
Update CI, fix uint tests
Naseem77 d691341
Bump esbuild from 0.25.10 to 0.27.0 in /app
dependabot[bot] 0ebcad0
Fix: Auto-quote SQL identifiers with special characters (dashes, spac…
shahar-biron fd08200
Fix CI errors: Add __init__.py, fix pylint issues, add spellcheck words
shahar-biron 7add51e
Fix remaining pylint errors: export classes and add disable comments
shahar-biron f7f588d
Fix pylint import error: use package-level imports
shahar-biron e0dd33b
Fix module conflict: rename api/utils to api/sql_utils
shahar-biron f192dcb
Merge branch 'staging' into fix/sql-identifier-quoting
gkorland 599f7ce
Merge pull request #305 from FalkorDB/dependabot/npm_and_yarn/app/sta…
gkorland 682c885
fix ai comments
Naseem77 b56492b
fix ai comments
Naseem77 d977e4a
Merge branch 'staging' into migrate-to-react
Naseem77 152996a
fix package json
Naseem77 7c625f7
fix ai comments
Naseem77 b567d40
fix logging
Naseem77 b13bb8a
fix generic error message
Naseem77 1d63bbf
fix uint tests
Naseem77 40b4777
Bump fastmcp from 2.12.4 to 2.13.0.1
dependabot[bot] 9c6c305
Merge pull request #301 from FalkorDB/migrate-to-react
galshubeli 28e7511
Resolve security vulnerability and code quality issues
Naseem77 a77cc23
check vulnerabilities and spellcheck on PR's targeting staging
Naseem77 9ee4cb6
Merge pull request #309 from FalkorDB/fix-resolve-security-vulnerabil…
galshubeli c04d495
Merge branch 'staging' into dependabot/pip/staging/fastmcp-2.13.0.1
gkorland 935a231
Merge pull request #294 from FalkorDB/dependabot/pip/staging/fastmcp-…
gkorland 1ece860
Bump litellm from 1.77.7 to 1.79.0
dependabot[bot] 64706d8
Merge pull request #297 from FalkorDB/dependabot/pip/staging/litellm-…
gkorland c81b12e
update deps
gkorland 7152314
Merge pull request #312 from FalkorDB/deps_update
gkorland a3a5a11
UI improvements
Naseem77 7d4c4e4
Merge branch 'staging' into ui-improvements
Naseem77 d8183ea
Add missing hyperlink and loading state for refresh schema
Naseem77 1a1981d
add fonts change sizes, colors for ui
Naseem77 3606aca
fix ai comments
Naseem77 62e71d0
disabling UI controls while processing to prevent concurrent actions
Naseem77 38f1afb
Merge pull request #313 from FalkorDB/ui-improvements
galshubeli b93cc04
Merge branch 'main' into staging
galshubeli 9075242
Merge branch 'staging' into fix/sql-identifier-quoting
galshubeli 05c93f2
Merge pull request #306 from FalkorDB/fix/sql-identifier-quoting
galshubeli 6ad5b08
Bump playwright from 1.55.0 to 1.56.0
dependabot[bot] 6b8477c
Merge pull request #323 from FalkorDB/dependabot/pip/staging/playwrig…
gkorland cdec8f6
Bump fastapi from 0.121.3 to 0.123.0
dependabot[bot] c004c4f
Merge pull request #326 from FalkorDB/dependabot/pip/staging/fastapi-…
gkorland 1e4eb97
Bump litellm from 1.80.5 to 1.80.7
dependabot[bot] c718a01
Merge pull request #327 from FalkorDB/dependabot/pip/staging/litellm-…
gkorland 4906b4c
Add e2e Tests
Naseem77 1238be6
Add api calls/responses for logic layer
Naseem77 d96ac6b
adding pom for homePage
Naseem77 7b40625
adding auth setup
Naseem77 783d9f4
add docker compose file for database connections
Naseem77 862a87b
add sidebar tests
Naseem77 371e1fb
Bump falkordb from 1.2.0 to 1.2.2
dependabot[bot] 692efed
add user profile tests
Naseem77 4d34598
update user profile tests
Naseem77 a966b03
update auth setup - adding another user
Naseem77 1b880a4
add database tests
Naseem77 f9e369c
update playwright CI
Naseem77 c124c2e
update ci
Naseem77 f6c751f
update ci
Naseem77 2ddeb39
Update playwright.yml
Naseem77 13be2d3
Merge pull request #336 from FalkorDB/dependabot/pip/staging/falkordb…
galshubeli 2e65792
update ci
Naseem77 d7233b7
update ci
Naseem77 bfa04c1
Update playwright.yml
Naseem77 610c3f5
update CI and tests
Naseem77 fb43ce5
add chat tests
Naseem77 21e2fa0
enable firefox project
Naseem77 054461d
Merge branch 'staging' into add-e2e-tests
Naseem77 dea7f58
fix logout test
Naseem77 2c1c716
Merge branch 'add-e2e-tests' of https://github.com/FalkorDB/QueryWeav…
Naseem77 f30011d
fix ai comments
Naseem77 c78add6
fix apiRequests
Naseem77 54b50d4
Fix the high severity vulnerability in the mcp package in stage
Naseem77 60db6ef
Merge pull request #331 from FalkorDB/add-e2e-tests
Naseem77 0f3f696
Add inline confirmation for destructive database operations
Naseem77 5fde342
fix ai comments
Naseem77 63e772f
add tests for confirmation for destructive database operations
Naseem77 f97d39b
Merge pull request #339 from FalkorDB/add-inline-conformation-ui-for-…
Naseem77 b635ba7
fix ai comments
Naseem77 48ff79c
Merge pull request #341 from FalkorDB/fix-ai-comments
galshubeli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,168 @@ | ||
| name: Playwright Tests | ||
| on: | ||
| push: | ||
| branches: [ main, staging ] | ||
| pull_request: | ||
| branches: [ main, staging ] | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| env: | ||
| PYTHON_VERSION: '3.12' | ||
| NODE_VERSION: 'lts/*' | ||
|
|
||
|
|
||
| jobs: | ||
| test: | ||
| timeout-minutes: 60 | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| # Setup Python | ||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: ${{ env.PYTHON_VERSION }} | ||
|
|
||
| # Setup Node.js | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: ${{ env.NODE_VERSION }} | ||
|
|
||
| # Install pipenv | ||
| - name: Install pipenv | ||
| run: pip install pipenv | ||
|
|
||
| # Install Python dependencies | ||
| - name: Install Python dependencies | ||
| run: pipenv sync --dev | ||
|
|
||
| # Install Node dependencies (root - for Playwright) | ||
| - name: Install root dependencies | ||
| run: npm ci | ||
|
|
||
| # Install Node dependencies (frontend) | ||
| - name: Install frontend dependencies | ||
| run: npm ci | ||
| working-directory: ./app | ||
|
|
||
| # Build frontend | ||
| - name: Build frontend | ||
| run: npm run build | ||
| working-directory: ./app | ||
|
|
||
| # Start Docker Compose services (test databases) | ||
| - name: Start test databases | ||
| run: | | ||
| docker compose -f e2e/docker-compose.test.yml up -d | ||
| # Wait for databases to be healthy | ||
| echo "Waiting for databases to be ready..." | ||
| sleep 20 | ||
| docker ps -a | ||
| # Verify all containers are running | ||
| docker compose -f e2e/docker-compose.test.yml ps | ||
|
|
||
| # Start FalkorDB (Redis graph database) | ||
| - name: Start FalkorDB | ||
| run: | | ||
| docker run -d --name falkordb-test -p 6379:6379 falkordb/falkordb:latest | ||
| echo "Waiting for FalkorDB to be ready..." | ||
| # Wait for FalkorDB to accept connections and be fully initialized | ||
| for i in {1..30}; do | ||
| if docker exec falkordb-test redis-cli PING > /dev/null 2>&1; then | ||
| # Test that FalkorDB graph commands work | ||
| if docker exec falkordb-test redis-cli GRAPH.LIST > /dev/null 2>&1; then | ||
| echo "FalkorDB graph module loaded, testing vector index creation..." | ||
| # Test creating a graph with vector index (the actual operation that fails) | ||
| if docker exec falkordb-test redis-cli GRAPH.QUERY test_graph "CREATE VECTOR INDEX FOR (n:Test) ON (n.embedding)" > /dev/null 2>&1; then | ||
| # Clean up test graph | ||
| docker exec falkordb-test redis-cli GRAPH.DELETE test_graph > /dev/null 2>&1 || true | ||
| echo "FalkorDB is fully ready with vector index support!" | ||
| # Extra wait to ensure complete initialization | ||
| sleep 5 | ||
| break | ||
| fi | ||
| fi | ||
| fi | ||
| echo "Waiting for FalkorDB initialization... attempt $i/30" | ||
| sleep 1 | ||
| done | ||
| # Verify FalkorDB is accessible from host | ||
| echo "Testing FalkorDB connectivity from host..." | ||
| docker ps -a | ||
|
|
||
| # Start the FastAPI application | ||
| - name: Start FastAPI application | ||
| run: | | ||
| # Start server in background directly with pipenv | ||
| pipenv run uvicorn api.index:app --host localhost --port 5000 > /tmp/uvicorn.log 2>&1 & | ||
| UVICORN_PID=$! | ||
| echo "Started server with PID: $UVICORN_PID" | ||
| # Wait for app to start | ||
| echo "Waiting for application to start..." | ||
| for i in {1..30}; do | ||
| if curl -f http://localhost:5000/ 2>/dev/null; then | ||
| echo "Application started successfully!" | ||
| break | ||
| fi | ||
| echo "Waiting... attempt $i/30" | ||
| sleep 1 | ||
| if [ $i -eq 30 ]; then | ||
| echo "Failed to start application after 30 seconds" | ||
| echo "=== Server logs ===" | ||
| cat /tmp/uvicorn.log | ||
| exit 1 | ||
| fi | ||
| done | ||
| env: | ||
| PYTHONUNBUFFERED: 1 | ||
| FASTAPI_SECRET_KEY: test-secret-key-for-ci | ||
| APP_ENV: development | ||
| FASTAPI_DEBUG: False | ||
| FALKORDB_URL: redis://localhost:6379 | ||
| DISABLE_MCP: true | ||
| # Azure OpenAI API keys - required for database schema analysis | ||
| AZURE_API_KEY: ${{ secrets.AZURE_API_KEY }} | ||
| AZURE_API_BASE: ${{ secrets.AZURE_API_BASE }} | ||
| AZURE_API_VERSION: ${{ secrets.AZURE_API_VERSION }} | ||
|
|
||
| # Install Playwright browsers | ||
| - name: Install Playwright Browsers | ||
| run: npx playwright install --with-deps chromium firefox | ||
|
|
||
| # Create auth directory for storage state | ||
| - name: Create auth directory | ||
| run: mkdir -p e2e/.auth | ||
|
|
||
| # Run Playwright tests | ||
| - name: Run Playwright tests | ||
| run: npx playwright test --reporter=list | ||
| env: | ||
| CI: true | ||
|
|
||
| # Upload test results on failure | ||
| - uses: actions/upload-artifact@v4 | ||
| if: failure() | ||
| with: | ||
| name: playwright-report | ||
| path: playwright-report/ | ||
| retention-days: 30 | ||
|
|
||
| # Upload test screenshots on failure | ||
| - uses: actions/upload-artifact@v4 | ||
| if: failure() | ||
| with: | ||
| name: test-results | ||
| path: test-results/ | ||
| retention-days: 30 | ||
|
|
||
| # Cleanup - Stop all containers | ||
| - name: Cleanup Docker containers | ||
| if: always() | ||
| run: | | ||
| docker compose -f e2e/docker-compose.test.yml down -v || true | ||
| docker stop falkordb-test || true | ||
| docker rm falkordb-test || true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.