HomePhabricator
Diffusion Bitcoin ABC staging 6030625631c6

test: Add more thorough test for dbwrapper iterators
6030625631c6Unpublished
Actions

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

test: Add more thorough test for dbwrapper iterators

I made a silly mistake in a database wrapper where keys
were sorted by char instead of uint8_t. As x86 char is signed
the sorting for the block index database was messed up, resulting
in a segfault due to missing records.

Add a test to catch:

  • Wrong sorting
  • Seeking errors
  • Iteration result not complete

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Apr 27 2016, 09:07
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Parents
rSTAGING84c13e759dbb: chain: Add assertion in case of missing records in index db
Branches
Unknown
Tags
Unknown

Event Timeline

Wladimir J. van der Laan <laanwj@gmail.com> committed rSTAGING6030625631c6: test: Add more thorough test for dbwrapper iterators (authored by Wladimir J. van der Laan <laanwj@gmail.com>).Apr 27 2016, 09:11

Changes (1)

PathSize
src/
test/

rSTAGING6030625631c6

View Options

src/test/dbwrapper_tests.cpp

Loading...