Skip to content

Commit 70d807c

Browse files
committed
refactor: integrate no_nul into univalue unitester
1 parent 98a0ae6 commit 70d807c

File tree

7 files changed

+11
-20
lines changed

7 files changed

+11
-20
lines changed

ci/test/wrap-qemu.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
export LC_ALL=C.UTF-8
88

9-
for b_name in {"${BASE_OUTDIR}/bin"/*,src/secp256k1/*tests,src/minisketch/test{,-verify},src/univalue/{no_nul,test_json,unitester,object}}; do
9+
for b_name in {"${BASE_OUTDIR}/bin"/*,src/secp256k1/*tests,src/minisketch/test{,-verify},src/univalue/{test_json,unitester,object}}; do
1010
# shellcheck disable=SC2044
1111
for b in $(find "${BASE_ROOT_DIR}" -executable -type f -name "$(basename "$b_name")"); do
1212
echo "Wrap $b ..."

ci/test/wrap-wine.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
export LC_ALL=C.UTF-8
88

9-
for b_name in {"${BASE_OUTDIR}/bin"/*,src/secp256k1/*tests,src/minisketch/test{,-verify},src/univalue/{no_nul,test_json,unitester,object}}.exe; do
9+
for b_name in {"${BASE_OUTDIR}/bin"/*,src/secp256k1/*tests,src/minisketch/test{,-verify},src/univalue/{test_json,unitester,object}}.exe; do
1010
# shellcheck disable=SC2044
1111
for b in $(find "${BASE_ROOT_DIR}" -executable -type f -name "$(basename "$b_name")"); do
1212
if (file "$b" | grep "Windows"); then

src/Makefile.test.include

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ endif
373373
$(AM_V_at)$(MAKE) $(AM_MAKEFLAGS) -C secp256k1 check
374374

375375
if ENABLE_TESTS
376-
UNIVALUE_TESTS = univalue/test/object univalue/test/unitester univalue/test/no_nul
376+
UNIVALUE_TESTS = univalue/test/object univalue/test/unitester
377377
noinst_PROGRAMS += $(UNIVALUE_TESTS)
378378
TESTS += $(UNIVALUE_TESTS)
379379

@@ -382,11 +382,6 @@ univalue_test_unitester_LDADD = $(LIBUNIVALUE)
382382
univalue_test_unitester_CPPFLAGS = -I$(srcdir)/$(UNIVALUE_INCLUDE_DIR_INT) -DJSON_TEST_SRC=\"$(srcdir)/$(UNIVALUE_TEST_DATA_DIR_INT)\"
383383
univalue_test_unitester_LDFLAGS = -static $(LIBTOOL_APP_LDFLAGS)
384384

385-
univalue_test_no_nul_SOURCES = $(UNIVALUE_TEST_NO_NUL_INT)
386-
univalue_test_no_nul_LDADD = $(LIBUNIVALUE)
387-
univalue_test_no_nul_CPPFLAGS = -I$(srcdir)/$(UNIVALUE_INCLUDE_DIR_INT)
388-
univalue_test_no_nul_LDFLAGS = -static $(LIBTOOL_APP_LDFLAGS)
389-
390385
univalue_test_object_SOURCES = $(UNIVALUE_TEST_OBJECT_INT)
391386
univalue_test_object_LDADD = $(LIBUNIVALUE)
392387
univalue_test_object_CPPFLAGS = -I$(srcdir)/$(UNIVALUE_INCLUDE_DIR_INT)

src/univalue/sources.mk

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ UNIVALUE_TEST_UNITESTER_INT += %reldir%/test/unitester.cpp
2525
UNIVALUE_TEST_JSON_INT =
2626
UNIVALUE_TEST_JSON_INT += %reldir%/test/test_json.cpp
2727

28-
UNIVALUE_TEST_NO_NUL_INT =
29-
UNIVALUE_TEST_NO_NUL_INT += %reldir%/test/no_nul.cpp
30-
3128
UNIVALUE_TEST_OBJECT_INT =
3229
UNIVALUE_TEST_OBJECT_INT += %reldir%/test/object.cpp
3330

src/univalue/test/.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
object
33
unitester
44
test_json
5-
no_nul
65

76
*.trs
87
*.log

src/univalue/test/no_nul.cpp

Lines changed: 0 additions & 8 deletions
This file was deleted.

src/univalue/test/unitester.cpp

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,13 +149,21 @@ void unescape_unicode_test()
149149
assert(val[0].get_str() == "\xf0\x9d\x85\xa1");
150150
}
151151

152+
void no_nul_test()
153+
{
154+
char buf[] = "___[1,2,3]___";
155+
UniValue val;
156+
assert(val.read(buf + 3, 7));
157+
}
158+
152159
int main (int argc, char *argv[])
153160
{
154161
for (const auto& f: filenames) {
155162
runtest_file(f);
156163
}
157164

158165
unescape_unicode_test();
166+
no_nul_test();
159167

160168
return 0;
161169
}

0 commit comments

Comments
 (0)