Page MenuHomePhabricator

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

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

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
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
Branch
PR15399_part1
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 8401
Build 14819: Bitcoin ABC Buildbot
Build 14818: arc lint + arc unit

Event Timeline

Fabien created this revision.Dec 4 2019, 12:25
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 4 2019, 12:25
Fabien updated this revision to Diff 14609.Dec 4 2019, 13:00

Rebase

Fabien updated this revision to Diff 14611.Dec 4 2019, 13:02

Rebase

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