test: add ParseUInt16() unit test and fuzz coverage #21488

pull jonatack wants to merge 1 commits into bitcoin:master from jonatack:ParseUInt16-test-coverage changing 2 files +39 −0

jonatack commented at 7:45 PM on March 20, 2021: member

ParseUInt16() was just added in #21328.
test: add ParseUInt16() test coverage 3d086f42ab
DrahtBot added the label Tests on Mar 20, 2021
practicalswift commented at 9:22 PM on March 20, 2021: contributor

cr ACK 3d086f42ab58f0f5984101d5285d5a707e3dc8e7: patch looks correct & more coverage is better than less coverage
MarcoFalke merged this on Mar 21, 2021
MarcoFalke closed this on Mar 21, 2021

in src/test/util_tests.cpp:1552 in 3d086f42ab

1547 | +    BOOST_CHECK(!ParseUInt16("-1", &n));
1548 | +    BOOST_CHECK(!ParseUInt16("1 ", &n));
1549 | +    BOOST_CHECK(!ParseUInt16("1a", &n));
1550 | +    BOOST_CHECK(!ParseUInt16("aap", &n));
1551 | +    BOOST_CHECK(!ParseUInt16("0x1", &n)); // no hex
1552 | +    BOOST_CHECK(!ParseUInt16("0x1", &n)); // no hex

kiminuo commented at 9:16 AM on March 21, 2021:

These two lines look the same to me.

jonatack commented at 10:18 AM on March 21, 2021:

Thanks! Fixed in #21491, co-crediting you

jonatack deleted the branch on Mar 21, 2021
sidhujag referenced this in commit fc65f9795d on Mar 21, 2021
fanquake referenced this in commit 80cb51cc6b on Mar 22, 2021
sidhujag referenced this in commit 495322941e on Mar 22, 2021
Fabcien referenced this in commit c38178957a on Feb 21, 2022
DrahtBot locked this on Aug 16, 2022

Contributors

Labels

Linked (view graph)

#21328 net, refactor: pass uint16 CService::port as uint16 #21491 test: remove duplicate assertions in util_tests