From 0d4e7aec9ce0228448c8ad8304bc5daaa4eff3d7 Mon Sep 17 00:00:00 2001 From: Karl-Johan Alm Date: Wed, 21 Dec 2016 13:43:49 +0900 Subject: [PATCH] Added some simple tests for the RAII-style events. --- src/Makefile.test.include | 1 + src/support/events.h | 1 + src/test/CMakeLists.txt | 1 + src/test/raii_event_tests.cpp | 86 +++++++++++++++++++++++++++++++++++ 4 files changed, 89 insertions(+) create mode 100644 src/test/raii_event_tests.cpp diff --git a/src/Makefile.test.include b/src/Makefile.test.include index e904d5a1ea4d8..b90631b6062bd 100644 --- a/src/Makefile.test.include +++ b/src/Makefile.test.include @@ -128,6 +128,7 @@ BITCOIN_TESTS =\ test/pmt_tests.cpp \ test/policyestimator_tests.cpp \ test/prevector_tests.cpp \ + test/raii_event_tests.cpp \ test/random_tests.cpp \ test/reverselock_tests.cpp \ test/rpc_tests.cpp \ diff --git a/src/support/events.h b/src/support/events.h index 16378086fa6ee..4f2f3cf9ef760 100644 --- a/src/support/events.h +++ b/src/support/events.h @@ -6,6 +6,7 @@ #define BITCOIN_SUPPORT_EVENTS_H #include +#include #include #include diff --git a/src/test/CMakeLists.txt b/src/test/CMakeLists.txt index 6345f4981ec27..5d4b726fea4dc 100644 --- a/src/test/CMakeLists.txt +++ b/src/test/CMakeLists.txt @@ -145,6 +145,7 @@ set(BITCOIN_TESTS ${CMAKE_CURRENT_SOURCE_DIR}/pmt_tests.cpp ${CMAKE_CURRENT_SOURCE_DIR}/policyestimator_tests.cpp ${CMAKE_CURRENT_SOURCE_DIR}/prevector_tests.cpp + ${CMAKE_CURRENT_SOURCE_DIR}/raii_event_tests.cpp ${CMAKE_CURRENT_SOURCE_DIR}/random_tests.cpp ${CMAKE_CURRENT_SOURCE_DIR}/reverselock_tests.cpp ${CMAKE_CURRENT_SOURCE_DIR}/rpc_tests.cpp diff --git a/src/test/raii_event_tests.cpp b/src/test/raii_event_tests.cpp new file mode 100644 index 0000000000000..14a61c9a26359 --- /dev/null +++ b/src/test/raii_event_tests.cpp @@ -0,0 +1,86 @@ +// Copyright (c) 2016 The Bitcoin Core developers +// Distributed under the MIT software license, see the accompanying +// file COPYING or http://www.opensource.org/licenses/mit-license.php. + +#include +#include +#include + +#include "support/events.h" + +#include "test/test_pivx.h" + +#include + +static std::map tags; +static std::map orders; +static uint16_t tagSequence = 0; + +static void* tag_malloc(size_t sz) { + void* mem = malloc(sz); + if (!mem) return mem; + tags[mem]++; + orders[mem] = tagSequence++; + return mem; +} + +static void tag_free(void* mem) { + tags[mem]--; + orders[mem] = tagSequence++; + free(mem); +} + +BOOST_FIXTURE_TEST_SUITE(raii_event_tests, BasicTestingSetup) + +BOOST_AUTO_TEST_CASE(raii_event_creation) +{ + event_set_mem_functions(tag_malloc, realloc, tag_free); + + void* base_ptr = nullptr; + { + auto base = obtain_event_base(); + base_ptr = (void*)base.get(); + BOOST_CHECK(tags[base_ptr] == 1); + } + BOOST_CHECK(tags[base_ptr] == 0); + + void* event_ptr = nullptr; + { + auto base = obtain_event_base(); + auto event = obtain_event(base.get(), -1, 0, nullptr, nullptr); + + base_ptr = (void*)base.get(); + event_ptr = (void*)event.get(); + + BOOST_CHECK(tags[base_ptr] == 1); + BOOST_CHECK(tags[event_ptr] == 1); + } + BOOST_CHECK(tags[base_ptr] == 0); + BOOST_CHECK(tags[event_ptr] == 0); + + event_set_mem_functions(malloc, realloc, free); +} + +BOOST_AUTO_TEST_CASE(raii_event_order) +{ + event_set_mem_functions(tag_malloc, realloc, tag_free); + + void* base_ptr = nullptr; + void* event_ptr = nullptr; + { + auto base = obtain_event_base(); + auto event = obtain_event(base.get(), -1, 0, nullptr, nullptr); + + base_ptr = (void*)base.get(); + event_ptr = (void*)event.get(); + + // base should have allocated before event + BOOST_CHECK(orders[base_ptr] < orders[event_ptr]); + } + // base should be freed after event + BOOST_CHECK(orders[base_ptr] > orders[event_ptr]); + + event_set_mem_functions(malloc, realloc, free); +} + +BOOST_AUTO_TEST_SUITE_END()