test/fuzz: Don't gather coverage under short-fuzz job

That was the main intention behind the following patch:

https://review.spdk.io/gerrit/c/spdk/spdk/+/15356

however, the introduced logic was faulty. Fix it.

Signed-off-by: Michal Berger <michal.berger@intel.com>
Change-Id: I84ec7ba79d477756c9064e592e93140e68038bc3
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16348
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
This commit is contained in:
Michal Berger 2023-01-19 12:50:56 +01:00 committed by Tomasz Zawadzki
parent cf0cf4790b
commit 14671b8f4e

View File

@ -55,7 +55,7 @@ function lcov_stop() {
# Collect coverage data when run fuzzers for longer period of time
# this allow to check coverage progression between runs, and grow of corpus files
if [[ $SPDK_TEST_FUZZER -eq 1 || $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then
if [[ $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then
lcov_start
fi
@ -66,7 +66,7 @@ for fuzzer in "${fuzzers[@]}"; do
esac
done
if [[ $SPDK_TEST_FUZZER -eq 1 || $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then
if [[ $SPDK_TEST_FUZZER_SHORT -eq 0 ]]; then
lcov_stop
genhtml $llvm_out/cov_total.info --output-directory $llvm_out
fi