Skip to content
This repository was archived by the owner on Mar 24, 2022. It is now read-only.

Commit 809ba7a

Browse files
committed
add make test-full target; make test more focused
The benchmark and fuzz targets in particular take quite a long time to run, and are usually not the focus of development. They're still worth running in CI, though, to catch bitrot.
1 parent 9239c97 commit 809ba7a

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
# This action builds the container and executes the test suite inside it.
1515
- uses: ./.github/actions/test
1616
with:
17-
target: test
17+
target: test-full
1818

1919
- name: Ensure testing did not change sources
2020
run: git diff --exit-code

Makefile

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,10 @@ install-dev: build-dev
2121
@helpers/install.sh --unoptimized
2222

2323
.PHONY: test
24-
test: indent-check test-except-fuzz test-fuzz
24+
test: indent-check test-packages
2525

26-
.PHONY: test-except-fuzz
27-
test-except-fuzz:
28-
cargo build -p lucet-spectest # build but *not* run spectests to mitigate bitrot while spectests don't pass
26+
.PHONY: test-packages
27+
test-packages:
2928
cargo test --no-fail-fast \
3029
-p lucet-runtime-internals \
3130
-p lucet-runtime \
@@ -35,6 +34,13 @@ test-except-fuzz:
3534
-p lucet-wasi \
3635
-p lucet-wasi-fuzz \
3736
-p lucet-validate
37+
38+
.PHONY: test-full
39+
test-full: indent-check test-except-fuzz test-fuzz
40+
41+
.PHONY: test-except-fuzz
42+
test-except-fuzz: test-packages
43+
cargo build -p lucet-spectest # build but *not* run spectests to mitigate bitrot while spectests don't pass
3844
cargo test --benches -p lucet-benchmarks -- --test # run the benchmarks in debug mode
3945
helpers/lucet-toolchain-tests/signature.sh
4046

0 commit comments

Comments
 (0)