diff --git a/lib/DnAppUnity/src/DnApp/Unity/asserts.h b/lib/DnAppUnity/src/DnApp/Unity/asserts.h index de8e34e..4abea50 100644 --- a/lib/DnAppUnity/src/DnApp/Unity/asserts.h +++ b/lib/DnAppUnity/src/DnApp/Unity/asserts.h @@ -4,14 +4,16 @@ #include "DnApp/Unity/functions.h" -namespace DnApp::Unity { - template - inline void TEST_ASSERT_INSTANCE_OF(const ObjectClass *object) { - TEST_ASSERT_TRUE(instance_of(object)); - } +#define TEST_ASSERT_INSTANCE_OF(expectedClass, actualObject) \ + UNITY_TEST_ASSERT( \ + (DnApp::Unity::instance_of(actualObject)), \ + __LINE__, \ + "Expected `" #actualObject "` to be an instance of `" #expectedClass "`" \ + ) - template - inline void TEST_ASSERT_NOT_INSTANCE_OF(const ObjectClass *object) { - TEST_ASSERT_FALSE(instance_of(object)); - } -} +#define TEST_ASSERT_NOT_INSTANCE_OF(expectedClass, actualObject) \ + UNITY_TEST_ASSERT( \ + !(DnApp::Unity::instance_of(actualObject)), \ + __LINE__, \ + "Expected `" #actualObject "` not to an instance of `" #expectedClass "`" \ + ) diff --git a/test/native/DnAppUnity/test_asserts/assertsTest.cpp b/test/native/DnAppUnity/test_asserts/assertsTest.cpp index 05951c8..d23d410 100644 --- a/test/native/DnAppUnity/test_asserts/assertsTest.cpp +++ b/test/native/DnAppUnity/test_asserts/assertsTest.cpp @@ -4,8 +4,6 @@ #include "DnApp/Unity/functions.h" using DnApp::Unity::instance_of; -using DnApp::Unity::TEST_ASSERT_INSTANCE_OF; -using DnApp::Unity::TEST_ASSERT_NOT_INSTANCE_OF; class Parent {} parent; @@ -21,18 +19,18 @@ void test_instance_of_itself() { TEST_ASSERT_TRUE(instance_of(&child)); TEST_ASSERT_TRUE(instance_of(&grandChild)); - TEST_ASSERT_INSTANCE_OF(&parent); - TEST_ASSERT_INSTANCE_OF(&otherParent); - TEST_ASSERT_INSTANCE_OF(&child); - TEST_ASSERT_INSTANCE_OF(&grandChild); + TEST_ASSERT_INSTANCE_OF(Parent, &parent); + TEST_ASSERT_INSTANCE_OF(OtherParent, &otherParent); + TEST_ASSERT_INSTANCE_OF(Child, &child); + TEST_ASSERT_INSTANCE_OF(GrandChild, &grandChild); } void test_child_is_instance_of_parents() { TEST_ASSERT_TRUE(instance_of(&child)); TEST_ASSERT_TRUE(instance_of(&child)); - TEST_ASSERT_INSTANCE_OF(&child); - TEST_ASSERT_INSTANCE_OF(&child); + TEST_ASSERT_INSTANCE_OF(Parent, &child); + TEST_ASSERT_INSTANCE_OF(OtherParent, &child); } void test_grandchild_is_instance_of_child_and_parents() { @@ -40,25 +38,25 @@ void test_grandchild_is_instance_of_child_and_parents() { TEST_ASSERT_TRUE(instance_of(&grandChild)); TEST_ASSERT_TRUE(instance_of(&grandChild)); - TEST_ASSERT_INSTANCE_OF(&grandChild); - TEST_ASSERT_INSTANCE_OF(&grandChild); - TEST_ASSERT_INSTANCE_OF(&grandChild); + TEST_ASSERT_INSTANCE_OF(Parent, &grandChild); + TEST_ASSERT_INSTANCE_OF(OtherParent, &grandChild); + TEST_ASSERT_INSTANCE_OF(Child, &grandChild); } void test_parent_is_not_instance_of_other_parent() { TEST_ASSERT_FALSE(instance_of(&otherParent)); TEST_ASSERT_FALSE(instance_of(&parent)); - TEST_ASSERT_NOT_INSTANCE_OF(&otherParent); - TEST_ASSERT_NOT_INSTANCE_OF(&parent); + TEST_ASSERT_NOT_INSTANCE_OF(Parent, &otherParent); + TEST_ASSERT_NOT_INSTANCE_OF(OtherParent, &parent); } void test_parent_is_not_instance_of_child_and_grandchild() { TEST_ASSERT_FALSE(instance_of(&parent)); TEST_ASSERT_FALSE(instance_of(&parent)); - TEST_ASSERT_NOT_INSTANCE_OF(&parent); - TEST_ASSERT_NOT_INSTANCE_OF(&parent); + TEST_ASSERT_NOT_INSTANCE_OF(Child, &parent); + TEST_ASSERT_NOT_INSTANCE_OF(GrandChild, &parent); } int main() { diff --git a/test/native/DnLogger/test_NullLogger/NullLoggerTest.cpp b/test/native/DnLogger/test_NullLogger/NullLoggerTest.cpp index c83c642..496ac1e 100644 --- a/test/native/DnLogger/test_NullLogger/NullLoggerTest.cpp +++ b/test/native/DnLogger/test_NullLogger/NullLoggerTest.cpp @@ -12,7 +12,7 @@ using DnApp::Logger::NullLogger; NullLogger logger; void test_it_is_a_Logger() { - DnApp::Unity::TEST_ASSERT_INSTANCE_OF(&logger); + TEST_ASSERT_INSTANCE_OF(Logger, &logger); } void test_logging_chars() {