Skip to content

Commit

Permalink
ci: add ech-test coverage to main CI
Browse files Browse the repository at this point in the history
  • Loading branch information
cpu committed Dec 21, 2024
1 parent 202a0b3 commit 5003f40
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,13 @@ jobs:
if: runner.os == 'Linux' # For 'nm'
run: |
! nm build/tests/client | grep '__asan_init'
! nm build/tests/server | grep '__asan_init'
! nm build/tests/server | grep '__asan_init'
- name: Run ECH connect test
if: matrix.crypto == 'aws-lc-rs' # No HPKE in ring
run: |
cmake --build build --target ech-test > ech-test.log
grep 'sni=encrypted' ech-test.log
# Our integration tests rely on a built-in provider being enabled.
# Double-check the library/unit tests work without any providers to
Expand Down Expand Up @@ -244,6 +250,12 @@ jobs:
- name: Integration test
run: cmake --build build --config "${{matrix.config}}" --target integration-test

- name: Run ECH connect test
if: matrix.crypto == 'aws-lc-rs' # No HPKE in ring
run: |
cmake --build build --target ech-test > ech-test.log
grep 'sni=encrypted' ech-test.log
ensure-header-updated:
runs-on: ubuntu-latest
defaults:
Expand Down

0 comments on commit 5003f40

Please sign in to comment.