Skip to content

Commit 3c31ce7

Browse files
committed
Fix the fuzz bug
We have a bug in the `run_task` script where by in processing the `fuzz` crate we _break_ instead of continuing. This means any crate listed after `fuzz` in the `CRATES` env var will be skipped. However we don't want to bother with the `test_vars.sh` script in `fuzz`, so continue to handle the `fuzz` crate inline in `build_and_test` but call `continue`. While we are at it call `cargo test` for good measure.
1 parent b2ac115 commit 3c31ce7

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

ci/run_task.sh

+6-2
Original file line numberDiff line numberDiff line change
@@ -109,12 +109,16 @@ build_and_test() {
109109
for crate in "${CRATES[@]}"; do
110110
local test_vars_script="$REPO_DIR/$crate/contrib/test_vars.sh"
111111

112-
# Building the fuzz crate is more-or-less just a sanity check.
112+
# Building/testing the fuzz crate is more-or-less just a sanity check.
113+
#
114+
# Do it here instead of calling down to `do_test` so we don't have to
115+
# worry about `contrib/test_vars.sh` in the `fuzz` crates.
113116
if [ "$crate" = "fuzz" ]; then
114117
pushd "$REPO_DIR/$crate" > /dev/null
115118
cargo --locked build
119+
cargo --locked test
116120
popd > /dev/null
117-
break
121+
continue
118122
fi
119123

120124
verbose_say "Sourcing $test_vars_script"

0 commit comments

Comments
 (0)