Skip to content

Commit 96382e8

Browse files
committed
Fix CI: run correct test files, remove hardcoded paths
- Remove hardcoded /Users/nico/Code/sqlmesh path from test files - Unit test job: test_console.py, test_datasets.py, test_integration.py - Integration test job: test_marquez_integration.py (actual Marquez) - Enable web UI in docker-compose for local development
1 parent 30381bb commit 96382e8

File tree

4 files changed

+14
-23
lines changed

4 files changed

+14
-23
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
run: uv sync --all-extras --dev
2525

2626
- name: Run unit tests
27-
run: uv run pytest tests/test_console.py tests/test_datasets.py -v
27+
run: uv run pytest tests/test_console.py tests/test_datasets.py tests/test_integration.py -v
2828

2929
integration-test:
3030
runs-on: ubuntu-latest
@@ -83,5 +83,5 @@ jobs:
8383
sleep 2
8484
done
8585
86-
- name: Run integration tests
87-
run: uv run pytest tests/test_integration.py tests/test_marquez_integration.py -v -s
86+
- name: Run Marquez integration tests
87+
run: uv run pytest tests/test_marquez_integration.py -v -s

docker-compose.yml

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,17 +17,17 @@ services:
1717
- POSTGRES_PASSWORD=marquez
1818

1919
# Uncomment to run the Marquez web UI locally
20-
# web:
21-
# image: marquezproject/marquez-web:latest
22-
# platform: linux/amd64
23-
# ports:
24-
# - "3000:3000"
25-
# depends_on:
26-
# - marquez
27-
# environment:
28-
# - MARQUEZ_HOST=marquez
29-
# - MARQUEZ_PORT=5000
30-
# - WEB_PORT=3000
20+
web:
21+
image: marquezproject/marquez-web:latest
22+
platform: linux/amd64
23+
ports:
24+
- "3000:3000"
25+
depends_on:
26+
- marquez
27+
environment:
28+
- MARQUEZ_HOST=marquez
29+
- MARQUEZ_PORT=5000
30+
- WEB_PORT=3000
3131

3232
db:
3333
image: postgres:14

tests/test_integration.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,11 @@
22
from __future__ import annotations
33

44
import os
5-
import sys
65
import tempfile
76
import shutil
87
import pytest
98
from unittest.mock import MagicMock, patch
109

11-
# Add sqlmesh to path for tests
12-
sys.path.insert(0, "/Users/nico/Code/sqlmesh")
13-
1410

1511
class TestOpenLineageIntegration:
1612
"""Integration tests for OpenLineage event emission."""

tests/test_marquez_integration.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,12 @@
22
from __future__ import annotations
33

44
import os
5-
import sys
65
import time
76
import tempfile
87
import shutil
9-
import subprocess
108
import pytest
119
import requests
1210

13-
# Add sqlmesh to path for tests
14-
sys.path.insert(0, "/Users/nico/Code/sqlmesh")
15-
1611
MARQUEZ_API_URL = "http://localhost:5001"
1712
MARQUEZ_LINEAGE_URL = "http://localhost:5001"
1813
NAMESPACE = "sqlmesh_test"

0 commit comments

Comments
 (0)