Skip to content

Commit 608a914

Browse files
committed
fix: build the binary from the test script
cargo test does not produce the bin target, so the suites had nothing to run against on a clean checkout.
1 parent 5f91db5 commit 608a914

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,12 @@ jobs:
4141
- name: Start the dev container
4242
run: docker compose -f docker/compose.yml up -d dev
4343

44-
- name: Unit tests
45-
run: docker compose -f docker/compose.yml exec -T dev cargo test --release --locked
46-
4744
- name: Clippy
4845
run: |
4946
docker compose -f docker/compose.yml exec -T dev \
5047
cargo clippy --release --locked -- -D warnings
5148
52-
- name: Conformance and fault injection
49+
- name: Build and test
5350
run: docker compose -f docker/compose.yml exec -T dev sh -c 'cd /work && sh tests/run-all.sh'
5451

5552
- name: Server logs on failure

tests/run-all.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
set -e
44
cd "$(dirname "$0")/.."
55

6-
cargo test --release
6+
cargo build --release --locked
7+
cargo test --release --locked
78
sh bench/setup-ref.sh
89
cp php/sse.php php/stream-large.php /srv-php/
910

0 commit comments

Comments
 (0)