diff options
author | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-08-10 09:08:24 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-08-10 10:56:02 +0000 |
commit | 583e41b9c8f734cb403b9ba9ffe2d0844fab981f (patch) | |
tree | 33f9051430ed8d9c210b9a6d5a43d1656cea454d /comphelper/qa/string | |
parent | b17b4da3a713b935e7eb8cf23978689800d10f7f (diff) |
comphelper: fix loplugin:cppunitassertequals warnings
Change-Id: If042462d8f250dbe2e1be278b7524dab243b08e8
Reviewed-on: https://gerrit.libreoffice.org/28012
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'comphelper/qa/string')
-rw-r--r-- | comphelper/qa/string/test_string.cxx | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/comphelper/qa/string/test_string.cxx b/comphelper/qa/string/test_string.cxx index 0d18ac03a73b..94349ba904c5 100644 --- a/comphelper/qa/string/test_string.cxx +++ b/comphelper/qa/string/test_string.cxx @@ -220,8 +220,8 @@ void TestString::testNatural() // --- Some generic tests to ensure we do not alter original behavior // outside what we want - CPPUNIT_ASSERT( - compareNatural("ABC", "ABC", xCollator, xBI, lang::Locale()) == 0 + CPPUNIT_ASSERT_EQUAL( + static_cast<sal_Int32>(0), compareNatural("ABC", "ABC", xCollator, xBI, lang::Locale()) ); // Case sensitivity CPPUNIT_ASSERT( @@ -271,8 +271,8 @@ void TestString::testNatural() CPPUNIT_ASSERT( compareNatural("abc010", "abc08", xCollator, xBI, lang::Locale()) > 0 ); - CPPUNIT_ASSERT( - compareNatural("apple10apple", "apple10apple", xCollator, xBI, lang::Locale()) == 0 + CPPUNIT_ASSERT_EQUAL( + static_cast<sal_Int32>(0), compareNatural("apple10apple", "apple10apple", xCollator, xBI, lang::Locale()) ); } @@ -282,10 +282,10 @@ void TestString::testStripStart() OString aOut; aOut = ::comphelper::string::stripStart(aIn, 'b'); - CPPUNIT_ASSERT(aOut == "abc"); + CPPUNIT_ASSERT_EQUAL(OString("abc"), aOut); aOut = ::comphelper::string::stripStart(aIn, 'a'); - CPPUNIT_ASSERT(aOut == "bc"); + CPPUNIT_ASSERT_EQUAL(OString("bc"), aOut); aIn = "aaa"; aOut = ::comphelper::string::stripStart(aIn, 'a'); @@ -293,7 +293,7 @@ void TestString::testStripStart() aIn = "aba"; aOut = ::comphelper::string::stripStart(aIn, 'a'); - CPPUNIT_ASSERT(aOut == "ba"); + CPPUNIT_ASSERT_EQUAL(OString("ba"), aOut); } void TestString::testStripEnd() @@ -302,10 +302,10 @@ void TestString::testStripEnd() OString aOut; aOut = ::comphelper::string::stripEnd(aIn, 'b'); - CPPUNIT_ASSERT(aOut == "abc"); + CPPUNIT_ASSERT_EQUAL(OString("abc"), aOut); aOut = ::comphelper::string::stripEnd(aIn, 'c'); - CPPUNIT_ASSERT(aOut == "ab"); + CPPUNIT_ASSERT_EQUAL(OString("ab"), aOut); aIn = "aaa"; aOut = ::comphelper::string::stripEnd(aIn, 'a'); @@ -313,7 +313,7 @@ void TestString::testStripEnd() aIn = "aba"; aOut = ::comphelper::string::stripEnd(aIn, 'a'); - CPPUNIT_ASSERT(aOut == "ab"); + CPPUNIT_ASSERT_EQUAL(OString("ab"), aOut); } void TestString::testStrip() @@ -322,10 +322,10 @@ void TestString::testStrip() OString aOut; aOut = ::comphelper::string::strip(aIn, 'b'); - CPPUNIT_ASSERT(aOut == "abc"); + CPPUNIT_ASSERT_EQUAL(OString("abc"), aOut); aOut = ::comphelper::string::strip(aIn, 'c'); - CPPUNIT_ASSERT(aOut == "ab"); + CPPUNIT_ASSERT_EQUAL(OString("ab"), aOut); aIn = "aaa"; aOut = ::comphelper::string::strip(aIn, 'a'); @@ -333,7 +333,7 @@ void TestString::testStrip() aIn = "aba"; aOut = ::comphelper::string::strip(aIn, 'a'); - CPPUNIT_ASSERT(aOut == "b"); + CPPUNIT_ASSERT_EQUAL(OString("b"), aOut); } void TestString::testToken() @@ -345,13 +345,13 @@ void TestString::testToken() CPPUNIT_ASSERT(aOut.isEmpty()); aOut = aIn.getToken(0, '.'); - CPPUNIT_ASSERT(aOut == "10"); + CPPUNIT_ASSERT_EQUAL(OString("10"), aOut); aOut = aIn.getToken(1, '.'); - CPPUNIT_ASSERT(aOut == "11"); + CPPUNIT_ASSERT_EQUAL(OString("11"), aOut); aOut = aIn.getToken(2, '.'); - CPPUNIT_ASSERT(aOut == "12"); + CPPUNIT_ASSERT_EQUAL(OString("12"), aOut); aOut = aIn.getToken(3, '.'); CPPUNIT_ASSERT(aOut.isEmpty()); @@ -363,13 +363,13 @@ void TestString::testTokenCount() sal_Int32 nOut; nOut = ::comphelper::string::getTokenCount(aIn, '.'); - CPPUNIT_ASSERT(nOut == 3); + CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(3), nOut); nOut = ::comphelper::string::getTokenCount(aIn, 'X'); - CPPUNIT_ASSERT(nOut == 1); + CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), nOut); nOut = ::comphelper::string::getTokenCount(OString(), 'X'); - CPPUNIT_ASSERT(nOut == 0); + CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(0), nOut); } void TestString::testReverseString() @@ -377,7 +377,7 @@ void TestString::testReverseString() OString aIn("ABC"); OString aOut = ::comphelper::string::reverseString(aIn); - CPPUNIT_ASSERT(aOut == "CBA"); + CPPUNIT_ASSERT_EQUAL(OString("CBA"), aOut); } void TestString::testSplit() |