Skip to content

Commit 5a6463b

Browse files
committed
CI: Test multiple ACVP versions
Previously, the acvp_client.py would test multiple versions of ACVP. This behaviour was changed when switching to on-the-fly downloading of test vectors to only use the most recent ACVP version. This commit adds testing older versions of ACVP in CI as well restoring the previous test coverage. Signed-off-by: Matthias J. Kannwischer <matthias@kannwischer.eu>
1 parent da6540b commit 5a6463b

File tree

1 file changed

+25
-0
lines changed

1 file changed

+25
-0
lines changed

.github/workflows/base.yml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,31 @@ jobs:
6161
- name: tests func
6262
run: |
6363
./scripts/tests func --check-namespace
64+
quickcheck-acvp:
65+
strategy:
66+
fail-fast: false
67+
matrix:
68+
external:
69+
- ${{ github.repository_owner != 'pq-code-package' }}
70+
target:
71+
- runner: pqcp-arm64
72+
name: 'aarch64'
73+
- runner: ubuntu-latest
74+
name: 'x86_64'
75+
acvp-version: [v1.1.0.38, v1.1.0.39, v1.1.0.40]
76+
exclude:
77+
- {external: true,
78+
target: {
79+
runner: pqcp-arm64,
80+
name: 'aarch64'
81+
}}
82+
name: Quickcheck ACVP (${{ matrix.target.name }}, ${{ matrix.acvp-version }})
83+
runs-on: ${{ matrix.target.runner }}
84+
steps:
85+
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
86+
- name: Run ACVP test
87+
run: |
88+
./scripts/tests acvp --version ${{ matrix.acvp-version }}
6489
quickcheck_bench:
6590
strategy:
6691
fail-fast: false

0 commit comments

Comments
 (0)