Skip to content

Commit a89517c

Browse files
committed
conformance testing: switch to pytest + cwltest
1 parent 955616b commit a89517c

File tree

3 files changed

+169
-140
lines changed

3 files changed

+169
-140
lines changed

.github/workflows/ci-tests.yml

Lines changed: 31 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Continuous integration tests
1+
name: CI Tests
22

33
on:
44
push:
@@ -11,16 +11,11 @@ concurrency:
1111
group: build-${{ github.event.pull_request.number || github.ref }}
1212
cancel-in-progress: true
1313

14-
env:
15-
singularity_version: 3.8.3
16-
1714
jobs:
1815

1916
tox:
20-
name: CI tests via Tox
21-
17+
name: Tox
2218
runs-on: ubuntu-20.04 # 22.04 doesn't support Python 3.6
23-
2419
strategy:
2520
matrix:
2621
py-ver-major: [3]
@@ -42,9 +37,9 @@ jobs:
4237

4338
- name: Set up Singularity
4439
if: ${{ matrix.step == 'unit' || matrix.step == 'mypy' }}
45-
uses: eWaterCycle/setup-singularity@v7
46-
with:
47-
singularity-version: ${{ env.singularity_version }}
40+
run: |
41+
wget --no-verbose https://github.com/sylabs/singularity/releases/download/v3.10.4/singularity-ce_3.10.4-focal_amd64.deb
42+
sudo apt-get install -y ./singularity-ce_3.10.4-focal_amd64.deb
4843
4944
- name: Give the test runner user a name to make provenance happy.
5045
if: ${{ matrix.step == 'unit' || matrix.step == 'mypy' }}
@@ -82,8 +77,7 @@ jobs:
8277
token: ${{ secrets.CODECOV_TOKEN }}
8378

8479
tox-style:
85-
name: CI linters via Tox
86-
80+
name: Linters
8781
runs-on: ubuntu-20.04
8882

8983
strategy:
@@ -118,9 +112,8 @@ jobs:
118112
run: tox
119113

120114
conformance_tests:
121-
name: CWL spec conformance tests
122-
123-
runs-on: ubuntu-20.04
115+
name: CWL conformance
116+
runs-on: ubuntu-22.04
124117

125118
strategy:
126119
matrix:
@@ -137,9 +130,16 @@ jobs:
137130

138131
- name: Set up Singularity
139132
if: ${{ matrix.container == 'singularity' }}
140-
uses: eWaterCycle/setup-singularity@v7
133+
run: |
134+
wget --no-verbose https://github.com/sylabs/singularity/releases/download/v3.10.4/singularity-ce_3.10.4-jammy_amd64.deb
135+
sudo apt-get install -y ./singularity-ce_3.10.4-jammy_amd64.deb
136+
137+
- name: Singularity cache
138+
if: ${{ matrix.container == 'singularity' }}
139+
uses: actions/cache@v3
141140
with:
142-
singularity-version: ${{ env.singularity_version }}
141+
path: sifcache
142+
key: singularity
143143

144144
- name: Set up Podman
145145
if: ${{ matrix.container == 'podman' }}
@@ -148,29 +148,32 @@ jobs:
148148
- name: Set up Python
149149
uses: actions/setup-python@v4
150150
with:
151-
python-version: 3.9
151+
python-version: 3.11
152152
cache: pip
153153

154-
- name: Run CWL conformance tests ${{ matrix.cwl-version }}
154+
- name: "Test CWL ${{ matrix.version }} conformance"
155155
env:
156-
version: ${{ matrix.cwl-version }}
157-
container: ${{ matrix.container }}
158-
spec_branch: main
156+
VERSION: ${{ matrix.cwl-version }}
157+
CONTAINER: ${{ matrix.container }}
158+
GIT_TARGET: main
159159
CWLTOOL_OPTIONS: ${{ matrix.extras }}
160160
run: ./conformance-test.sh
161-
161+
- name: Upload coverage to Codecov
162+
uses: codecov/codecov-action@v3
163+
with:
164+
fail_ci_if_error: true
165+
token: ${{ secrets.CODECOV_TOKEN }}
162166
release_test:
163167
name: cwltool release test
164-
165168
runs-on: ubuntu-22.04
166169

167170
steps:
168171
- uses: actions/checkout@v3
169172

170173
- name: Set up Singularity
171-
uses: eWaterCycle/setup-singularity@v7
172-
with:
173-
singularity-version: ${{ env.singularity_version }}
174+
run: |
175+
wget --no-verbose https://github.com/sylabs/singularity/releases/download/v3.10.4/singularity-ce_3.10.4-jammy_amd64.deb
176+
sudo apt-get install -y ./singularity-ce_3.10.4-jammy_amd64.deb
174177
175178
- name: Set up Python
176179
uses: actions/setup-python@v4
@@ -202,7 +205,7 @@ jobs:
202205
run: ./build-cwltool-docker.sh
203206

204207
macos:
205-
name: CI test on macos-latest
208+
name: Test on macos-latest
206209
runs-on: macos-latest
207210
env:
208211
TOXENV: py310-unit

0 commit comments

Comments
 (0)