From 74579fbadf0f89154cba5d9157a57f59fcda8f70 Mon Sep 17 00:00:00 2001 From: Mattes D Date: Mon, 26 Aug 2019 21:38:34 +0200 Subject: Improved testing framework. (#4376) --- tests/ByteBuffer/ByteBufferTest.cpp | 51 +++++++++++++++---------------------- tests/ByteBuffer/CMakeLists.txt | 7 +++-- 2 files changed, 23 insertions(+), 35 deletions(-) (limited to 'tests/ByteBuffer') diff --git a/tests/ByteBuffer/ByteBufferTest.cpp b/tests/ByteBuffer/ByteBufferTest.cpp index ddfe9e1d2..7278c5f0f 100644 --- a/tests/ByteBuffer/ByteBufferTest.cpp +++ b/tests/ByteBuffer/ByteBufferTest.cpp @@ -4,6 +4,7 @@ // Implements the main app entrypoint for the cByteBuffer class test #include "Globals.h" +#include "../TestHelpers.h" #include "ByteBuffer.h" @@ -15,11 +16,14 @@ static void TestRead(void) cByteBuffer buf(50); buf.Write("\x05\xac\x02\x00", 4); UInt32 v1; - assert_test(buf.ReadVarInt(v1) && (v1 == 5)); + TEST_TRUE(buf.ReadVarInt(v1)); + TEST_EQUAL(v1, 5); UInt32 v2; - assert_test(buf.ReadVarInt(v2) && (v2 == 300)); + TEST_TRUE(buf.ReadVarInt(v2)); + TEST_EQUAL(v2, 300); UInt32 v3; - assert_test(buf.ReadVarInt(v3) && (v3 == 0)); + TEST_TRUE(buf.ReadVarInt(v3)); + TEST_EQUAL(v3, 0); } @@ -34,8 +38,8 @@ static void TestWrite(void) buf.WriteVarInt32(0); AString All; buf.ReadAll(All); - assert_test(All.size() == 4); - assert_test(memcmp(All.data(), "\x05\xac\x02\x00", All.size()) == 0); + TEST_EQUAL(All.size(), 4); + TEST_EQUAL(memcmp(All.data(), "\x05\xac\x02\x00", All.size()), 0); } @@ -48,17 +52,17 @@ static void TestWrap(void) for (int i = 0; i < 1000; i++) { size_t FreeSpace = buf.GetFreeSpace(); - assert_test(buf.GetReadableSpace() == 0); - assert_test(FreeSpace > 0); - assert_test(buf.Write("a", 1)); - assert_test(buf.CanReadBytes(1)); - assert_test(buf.GetReadableSpace() == 1); + TEST_EQUAL(buf.GetReadableSpace(), 0); + TEST_GREATER_THAN_OR_EQUAL(FreeSpace, 1); + TEST_TRUE(buf.Write("a", 1)); + TEST_TRUE(buf.CanReadBytes(1)); + TEST_EQUAL(buf.GetReadableSpace(), 1); UInt8 v = 0; - assert_test(buf.ReadBEUInt8(v)); - assert_test(v == 'a'); - assert_test(buf.GetReadableSpace() == 0); + TEST_TRUE(buf.ReadBEUInt8(v)); + TEST_EQUAL(v, 'a'); + TEST_EQUAL(buf.GetReadableSpace(), 0); buf.CommitRead(); - assert_test(buf.GetFreeSpace() == FreeSpace); // We're back to normal + TEST_EQUAL(buf.GetFreeSpace(), FreeSpace); // We're back to normal } } @@ -66,23 +70,8 @@ static void TestWrap(void) -int main(int argc, char * argv[]) -{ - LOGD("Test started"); - - LOGD("Testing reads"); +IMPLEMENT_TEST_MAIN("ByteBuffer", TestRead(); - - LOGD("Testing writes"); TestWrite(); - - LOGD("Testing wraps"); TestWrap(); - - LOG("ByteBuffer test finished."); -} - - - - - +) diff --git a/tests/ByteBuffer/CMakeLists.txt b/tests/ByteBuffer/CMakeLists.txt index aa053e80f..989fabc23 100644 --- a/tests/ByteBuffer/CMakeLists.txt +++ b/tests/ByteBuffer/CMakeLists.txt @@ -8,14 +8,13 @@ add_definitions(-DTEST_GLOBALS=1) set (SHARED_SRCS ${CMAKE_SOURCE_DIR}/src/ByteBuffer.cpp - ${CMAKE_SOURCE_DIR}/src/OSSupport/StackTrace.cpp - ${CMAKE_SOURCE_DIR}/src/OSSupport/WinStackWalker.cpp + ${CMAKE_SOURCE_DIR}/src/StringUtils.cpp ) set (SHARED_HDRS + ../TestHelpers.h ${CMAKE_SOURCE_DIR}/src/ByteBuffer.h - ${CMAKE_SOURCE_DIR}/src/OSSupport/StackTrace.h - ${CMAKE_SOURCE_DIR}/src/OSSupport/WinStackWalker.h + ${CMAKE_SOURCE_DIR}/src/StringUtils.h ) set (SRCS -- cgit v1.2.3