Replace CPPUNIT_XXX macros with CHECKs

This results in better error messages and also allows the test to
continue executing even if a check fails.
This commit is contained in:
Vadim Zeitlin
2019-11-16 17:05:06 +01:00
parent 505e694347
commit b690caba84

View File

@@ -33,29 +33,29 @@ TEST_CASE("StringPrintf", "[wxString][Printf][vararg]")
// test passing literals: // test passing literals:
s.Printf("%s %i", "foo", 42); s.Printf("%s %i", "foo", 42);
CPPUNIT_ASSERT( s == "foo 42" ); CHECK( s == "foo 42" );
s.Printf("%s %s %i", wxT("bar"), "=", 11); s.Printf("%s %s %i", wxT("bar"), "=", 11);
// test passing c_str(): // test passing c_str():
CPPUNIT_ASSERT( s == "bar = 11" ); CHECK( s == "bar = 11" );
s2.Printf("(%s)", s.c_str()); s2.Printf("(%s)", s.c_str());
CPPUNIT_ASSERT( s2 == "(bar = 11)" ); CHECK( s2 == "(bar = 11)" );
s2.Printf(wxT("[%s](%s)"), s.c_str(), "str"); s2.Printf(wxT("[%s](%s)"), s.c_str(), "str");
CPPUNIT_ASSERT( s2 == "[bar = 11](str)" ); CHECK( s2 == "[bar = 11](str)" );
s2.Printf("%s mailbox", wxString("Opening").c_str()); s2.Printf("%s mailbox", wxString("Opening").c_str());
CPPUNIT_ASSERT( s2 == "Opening mailbox" ); CHECK( s2 == "Opening mailbox" );
// test passing wxString directly: // test passing wxString directly:
s2.Printf(wxT("[%s](%s)"), s, "str"); s2.Printf(wxT("[%s](%s)"), s, "str");
CPPUNIT_ASSERT( s2 == "[bar = 11](str)" ); CHECK( s2 == "[bar = 11](str)" );
// test passing wxCharBufferType<T>: // test passing wxCharBufferType<T>:
s = "FooBar"; s = "FooBar";
s2.Printf(wxT("(%s)"), s.mb_str()); s2.Printf(wxT("(%s)"), s.mb_str());
CPPUNIT_ASSERT( s2 == "(FooBar)" ); CHECK( s2 == "(FooBar)" );
s2.Printf(wxT("value=%s;"), s.wc_str()); s2.Printf(wxT("value=%s;"), s.wc_str());
CPPUNIT_ASSERT( s2 == "value=FooBar;" ); CHECK( s2 == "value=FooBar;" );
// this tests correct passing of wxCStrData constructed from string // this tests correct passing of wxCStrData constructed from string
// literal (and we disable the warnings related to the use of a literal // literal (and we disable the warnings related to the use of a literal
@@ -77,16 +77,16 @@ TEST_CASE("CharPrintf", "[wxString][Printf][vararg]")
// test using wchar_t: // test using wchar_t:
s.Printf("char=%c", L'c'); s.Printf("char=%c", L'c');
CPPUNIT_ASSERT_EQUAL( "char=c", s ); CHECK( s == "char=c" );
// test wxUniCharRef: // test wxUniCharRef:
s.Printf("string[1] is %c", foo[1]); s.Printf("string[1] is %c", foo[1]);
CPPUNIT_ASSERT_EQUAL( "string[1] is o", s ); CHECK( s == "string[1] is o" );
// test char // test char
char c = 'z'; char c = 'z';
s.Printf("%c to %c", 'a', c); s.Printf("%c to %c", 'a', c);
CPPUNIT_ASSERT_EQUAL( "a to z", s ); CHECK( s == "a to z" );
// test char used as integer: // test char used as integer:
#ifdef _MSC_VER #ifdef _MSC_VER
@@ -100,12 +100,12 @@ TEST_CASE("CharPrintf", "[wxString][Printf][vararg]")
#endif #endif
#ifndef __CHAR_UNSIGNED__ #ifndef __CHAR_UNSIGNED__
s.Printf("value is %i (int)", c); s.Printf("value is %i (int)", c);
CPPUNIT_ASSERT_EQUAL( wxString("value is -16 (int)"), s ); CHECK( s == wxString("value is -16 (int)") );
#endif #endif
unsigned char u = 240; unsigned char u = 240;
s.Printf("value is %i (int)", u); s.Printf("value is %i (int)", u);
CPPUNIT_ASSERT_EQUAL( "value is 240 (int)", s ); CHECK( s == "value is 240 (int)" );
} }
TEST_CASE("SizetPrintf", "[wxString][Printf][vararg]") TEST_CASE("SizetPrintf", "[wxString][Printf][vararg]")
@@ -113,17 +113,11 @@ TEST_CASE("SizetPrintf", "[wxString][Printf][vararg]")
size_t i = 1; size_t i = 1;
ssize_t j = -2; ssize_t j = -2;
CPPUNIT_ASSERT_EQUAL CHECK( wxString::Format("size_t=%zu ssize_t=%zd", i, j)
( == "size_t=1 ssize_t=-2" );
"size_t=1 ssize_t=-2",
wxString::Format("size_t=%zu ssize_t=%zd", i, j)
);
CPPUNIT_ASSERT_EQUAL CHECK( wxString::Format("size_t=0x%zX", static_cast<size_t>(160))
( == "size_t=0xA0" );
"size_t=0xA0",
wxString::Format("size_t=0x%zX", static_cast<size_t>(160))
);
} }
#if wxUSE_STD_STRING #if wxUSE_STD_STRING
@@ -136,10 +130,10 @@ TEST_CASE("StdString", "[wxString][Printf][vararg]")
std::string wc("widechar"); std::string wc("widechar");
s.Printf("string %s(%i).", mb, 1); s.Printf("string %s(%i).", mb, 1);
CPPUNIT_ASSERT_EQUAL( "string multi-byte(1).", s ); CHECK( s == "string multi-byte(1)." );
s.Printf("string %s(%i).", wc, 2); s.Printf("string %s(%i).", wc, 2);
CPPUNIT_ASSERT_EQUAL( "string widechar(2).", s ); CHECK( s == "string widechar(2)." );
} }
#endif // wxUSE_STD_STRING #endif // wxUSE_STD_STRING
@@ -148,10 +142,10 @@ TEST_CASE("LongLongPrintf", "[wxString][Printf][vararg]")
{ {
const char * const llfmt = "%" wxLongLongFmtSpec "d"; const char * const llfmt = "%" wxLongLongFmtSpec "d";
CPPUNIT_ASSERT_EQUAL( "17", wxString::Format(llfmt, wxLL(17)) ); CHECK( wxString::Format(llfmt, wxLL(17)) == "17" );
wxLongLong ll = 1234567890; wxLongLong ll = 1234567890;
CPPUNIT_ASSERT_EQUAL( "1234567890", wxString::Format(llfmt, ll) ); CHECK( wxString::Format(llfmt, ll) == "1234567890" );
} }
#endif // wxUSE_LONGLONG #endif // wxUSE_LONGLONG
@@ -163,8 +157,8 @@ TEST_CASE("Sscanf", "[wxSscanf][vararg]")
wxString input("42 test"); wxString input("42 test");
wxSscanf(input, "%d %s", &i, &str); wxSscanf(input, "%d %s", &i, &str);
CPPUNIT_ASSERT( i == 42 ); CHECK( i == 42 );
CPPUNIT_ASSERT( wxString(str) == "test" ); CHECK( wxString(str) == "test" );
#if !(defined(__MINGW32__) && \ #if !(defined(__MINGW32__) && \
defined(__USE_MINGW_ANSI_STDIO) && __USE_MINGW_ANSI_STDIO == 1) defined(__USE_MINGW_ANSI_STDIO) && __USE_MINGW_ANSI_STDIO == 1)
@@ -175,8 +169,8 @@ TEST_CASE("Sscanf", "[wxSscanf][vararg]")
i = 0; i = 0;
wxSscanf(input, L"%d %s", &i, &wstr); wxSscanf(input, L"%d %s", &i, &wstr);
CPPUNIT_ASSERT( i == 42 ); CHECK( i == 42 );
CPPUNIT_ASSERT( wxString(wstr) == "test" ); CHECK( wxString(wstr) == "test" );
#endif #endif
} }
@@ -190,10 +184,10 @@ TEST_CASE("RepeatedPrintf", "[wxString][Printf][vararg]")
wxString s; wxString s;
s = wxString::Format("buffer %s, len %d", buffer, (int)wxStrlen(buffer)); s = wxString::Format("buffer %s, len %d", buffer, (int)wxStrlen(buffer));
CPPUNIT_ASSERT_EQUAL("buffer hi, len 2", s); CHECK( s == "buffer hi, len 2" );
s = wxString::Format("buffer %s, len %d", buffer, (int)wxStrlen(buffer)); s = wxString::Format("buffer %s, len %d", buffer, (int)wxStrlen(buffer));
CPPUNIT_ASSERT_EQUAL("buffer hi, len 2", s); CHECK( s == "buffer hi, len 2" );
} }
TEST_CASE("ArgsValidation", "[wxString][vararg][error]") TEST_CASE("ArgsValidation", "[wxString][vararg][error]")
@@ -235,7 +229,7 @@ TEST_CASE("ArgsValidation", "[wxString][vararg][error]")
#ifndef wxNO_PRINTF_PERCENT_N #ifndef wxNO_PRINTF_PERCENT_N
wxString::Format("foo%i%n", 42, &written); wxString::Format("foo%i%n", 42, &written);
CPPUNIT_ASSERT_EQUAL( 5, written ); CHECK( written == 5 );
#endif #endif
// but these are not: // but these are not: