Skip to content

Commit 0cd8ce2

Browse files
committed
Apply code rabbit suggestions
1 parent 7b04902 commit 0cd8ce2

File tree

3 files changed

+5
-20
lines changed

3 files changed

+5
-20
lines changed

.github/workflows/_runner-siracusa.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ name: _runner-siracusa
1414
docker-image:
1515
required: true
1616
type: string
17-
test-type:
17+
pytest-marker:
1818
required: true
1919
type: string
20-
description: "Type of tests to run: kernels or models"
20+
description: "Pytest marker for test selection (e.g., 'kernels', 'models')"
2121

2222
jobs:
2323
test-runner-siracusa:
@@ -44,5 +44,5 @@ jobs:
4444
export CCACHE_DIR=/app/.ccache
4545
4646
# Run tests using pytest markers
47-
pytest test_platforms.py::test_siracusa_${{ inputs.test-type }} -v -s
47+
pytest test_platforms.py -m -v -n 4 "siracusa and ${{ inputs.pytest-marker }}"
4848
shell: bash

.github/workflows/ci-platform-siracusa.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,12 @@ jobs:
3535
with:
3636
runner: ${{ needs.select-env.outputs.runner }}
3737
docker-image: ${{ needs.select-env.outputs.image }}
38-
test-type: kernels
38+
pytest-marker: kernels
3939

4040
siracusa-models:
4141
needs: select-env
4242
uses: ./.github/workflows/_runner-siracusa.yml
4343
with:
4444
runner: ${{ needs.select-env.outputs.runner }}
4545
docker-image: ${{ needs.select-env.outputs.image }}
46-
test-type: models
46+
pytest-marker: models

DeeployTest/test_platforms.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -304,21 +304,6 @@ def test_siracusa_tiled_kernels_l2_singlebuffer(test_params, deeploy_test_dir, t
304304
double_buffer = False,
305305
)
306306
run_and_assert_test(test_name, config, skipgen, skipsim)
307-
config = create_test_config(
308-
test_name = test_name,
309-
platform = "Siracusa",
310-
simulator = "gvsoc",
311-
deeploy_test_dir = deeploy_test_dir,
312-
toolchain = toolchain,
313-
toolchain_dir = toolchain_dir,
314-
cmake_args = cmake_args,
315-
tiling = True,
316-
cores = SIRACUSA_DEFAULT_CORES,
317-
l1 = l1,
318-
default_mem_level = "L2",
319-
double_buffer = False,
320-
)
321-
run_and_assert_test(test_name, config, skipgen, skipsim)
322307

323308

324309
@pytest.mark.siracusa_tiled

0 commit comments

Comments
 (0)