Page MenuHomePhabricator

fuzz: Move deserialize tests to test/fuzz/deserialize.cpp
ClosedPublic

Authored by Fabien on Dec 4 2019, 12:25.

Details

Summary

This will avoid confusion when adding other tests, as test_bitcoin_fuzzy
is not a meaningful name.

Partial backport of core PR15399 (commit fabcfa5):
https://github.com/bitcoin/bitcoin/pull/15399/commits/fabcfa5f0cfe2484fe5cf5ced7cbae34a35c47c9

Depends on D4627.

Test Plan
./configure --disable-ccache --disable-wallet --disable-bench \
  --with-utils=no --with-daemon=no --with-libs=no --with-gui=no \
  --with-seeder=no --enable-fuzz --with-sanitizers=fuzzer,address \
  CC=clang CXX=clang++
make
mkdir -p test/fuzz
cp ../test/fuzz/test_runner.py test/fuzz/
./test/fuzz/test_runner.py -l DEBUG <path_to_corpus>

cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer"
ninja bitcoin-fuzzers link-fuzz-test_runner.py
./test/fuzz/test_runner.py -l DEBUG <path_to_corpus>

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

This revision is now accepted and ready to land.Dec 9 2019, 00:41