Skip to content

Commit 0fa3c6f

Browse files
committed
[python] migrate platform tests from rust->python
Signed-off-by: Gerd Zellweger <mail@gerdzellweger.com>
1 parent 87193d7 commit 0fa3c6f

21 files changed

+3322
-3961
lines changed

.github/workflows/test-integration-platform.yml

Lines changed: 14 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -60,17 +60,11 @@ jobs:
6060
name: Make sure manager runs with HTTPS
6161
runs-on: ubuntu-latest-amd64
6262
steps:
63-
- name: Download Test Binaries
64-
uses: actions/download-artifact@v4
65-
with:
66-
name: feldera-test-binaries-x86_64-unknown-linux-gnu
67-
path: build
68-
run-id: ${{ github.event.inputs.run_id || github.run_id }}
69-
github-token: ${{ secrets.GITHUB_TOKEN }}
63+
- name: Checkout repository
64+
uses: actions/checkout@v4
7065

71-
# Remove if https://github.com/actions/upload-artifact/issues/38 ever gets fixed
72-
- name: Make binaries executable
73-
run: chmod +x ./build/*
66+
- name: Install the latest version of uv
67+
uses: astral-sh/setup-uv@v6
7468

7569
- name: Login to GHCR with GITHUB_TOKEN
7670
run: echo "${{ secrets.GITHUB_TOKEN }}" | docker login ghcr.io -u ${{ github.actor }} --password-stdin
@@ -111,13 +105,16 @@ jobs:
111105
done
112106
echo "Timed out waiting for pipeline-manager to become healthy"
113107
exit 1
114-
- name: Run manager integration tests
115-
run: |
116-
./build/integration_test-*
108+
- name: Run platform tests
109+
if: ${{ vars.CI_DRY_RUN != 'true' }}
110+
run: uv run --locked pytest -n ${{ vars.PYTEST_WORKERS }} tests/platform/test_pipeline_crud.py --timeout=600
111+
working-directory: python
117112
env:
118113
TEST_FELDERA_HTTPS_TLS_CERT_PATH: test-tls/tls_test.crt
119-
TEST_FELDERA_URL: https://localhost:8080
120-
RUST_BACKTRACE: 1
114+
FELDERA_TLS_INSECURE: 1
115+
FELDERA_HOST: https://localhost:8080
116+
PYTHONPATH: ${{ github.workspace }}/python
117+
IN_CI: 1 # We use this flag to skip some kafka tests in the python code base
121118
- name: Logs & Cleanup
122119
if: always()
123120
run: |
@@ -155,18 +152,6 @@ jobs:
155152
- name: Checkout repository
156153
uses: actions/checkout@v4
157154

158-
- name: Download Test Binaries
159-
uses: actions/download-artifact@v4
160-
with:
161-
name: feldera-test-binaries-${{ matrix.target }}
162-
path: build
163-
run-id: ${{ github.event.inputs.run_id || github.run_id }}
164-
github-token: ${{ secrets.GITHUB_TOKEN }}
165-
166-
# Remove if https://github.com/actions/upload-artifact/issues/38 ever gets fixed
167-
- name: Make binaries executable
168-
run: chmod +x ./build/*
169-
170155
- name: Validate and run packaged demos
171156
if: ${{ vars.CI_DRY_RUN != 'true' }}
172157
run: |
@@ -175,16 +160,9 @@ jobs:
175160
env:
176161
PYTHONPATH: ${{ github.workspace }}/python
177162

178-
- name: Run manager tests
179-
if: ${{ vars.CI_DRY_RUN != 'true' }}
180-
run: ./build/integration_test-*
181-
env:
182-
TEST_FELDERA_URL: http://localhost:8080
183-
RUST_BACKTRACE: 1
184-
185-
- name: Run python tests
163+
- name: Run platform tests
186164
if: ${{ vars.CI_DRY_RUN != 'true' }}
187-
run: uv run --locked pytest -n ${{ vars.PYTEST_WORKERS }} tests/platform --timeout=600
165+
run: uv run --locked pytest -n ${{ vars.PYTEST_WORKERS }} tests/platform --timeout=1500
188166
working-directory: python
189167
env:
190168
FELDERA_HOST: http://localhost:8080

0 commit comments

Comments
 (0)