From 4e6d0f6fe70cb84c0fda0cc8e61baf20f5ab250e Mon Sep 17 00:00:00 2001 From: Miklos Vajna Date: Fri, 22 Jul 2016 09:14:24 +0200 Subject: basic: fix loplugin:cppunitassertequals warnings Change-Id: I80c773f3127ebbce4b9b2a122a91a2e2ca4f505b Reviewed-on: https://gerrit.libreoffice.org/27424 Reviewed-by: Miklos Vajna Tested-by: Jenkins --- basic/qa/cppunit/test_append.cxx | 2 +- basic/qa/cppunit/test_nested_struct.cxx | 8 ++++---- basic/qa/cppunit/test_vba.cxx | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'basic') diff --git a/basic/qa/cppunit/test_append.cxx b/basic/qa/cppunit/test_append.cxx index 98b96c468d4c..986c94119fd7 100644 --- a/basic/qa/cppunit/test_append.cxx +++ b/basic/qa/cppunit/test_append.cxx @@ -53,7 +53,7 @@ void EnableTest::testEnableRuntime() myMacro.Compile(); CPPUNIT_ASSERT_MESSAGE("testEnableRuntime fails with compile error",!myMacro.HasError() ); SbxVariableRef pNew = myMacro.Run(); - CPPUNIT_ASSERT(pNew->GetInteger() == 3 ); + CPPUNIT_ASSERT_EQUAL(static_cast(3), pNew->GetInteger()); } void EnableTest::testDimEnable() diff --git a/basic/qa/cppunit/test_nested_struct.cxx b/basic/qa/cppunit/test_nested_struct.cxx index aaa3ca87a16c..2fc06005072e 100644 --- a/basic/qa/cppunit/test_nested_struct.cxx +++ b/basic/qa/cppunit/test_nested_struct.cxx @@ -173,7 +173,7 @@ void Nested_Struct::testAssign1() myMacro.Compile(); CPPUNIT_ASSERT_MESSAGE("testAssign1 fails with compile error",!myMacro.HasError() ); SbxVariableRef pNew = myMacro.Run(); - CPPUNIT_ASSERT(pNew->GetInteger() == 9 ); + CPPUNIT_ASSERT_EQUAL(static_cast(9), pNew->GetInteger()); } void Nested_Struct::testAssign1Alt() @@ -196,7 +196,7 @@ void Nested_Struct::testOldAssign() myMacro.Compile(); CPPUNIT_ASSERT_MESSAGE("testOldAssign fails with compile error",!myMacro.HasError() ); SbxVariableRef pNew = myMacro.Run(); - CPPUNIT_ASSERT(pNew->GetInteger() == 9 ); + CPPUNIT_ASSERT_EQUAL(static_cast(9), pNew->GetInteger()); } void Nested_Struct::testOldAssignAlt() @@ -220,7 +220,7 @@ void Nested_Struct::testUnfixedVarAssign() CPPUNIT_ASSERT_MESSAGE("testUnfixedVarAssign fails with compile error",!myMacro.HasError() ); // forces a broadcast SbxVariableRef pNew = myMacro.Run(); - CPPUNIT_ASSERT(pNew->GetInteger() == 13 ); + CPPUNIT_ASSERT_EQUAL(static_cast(13), pNew->GetInteger()); } void Nested_Struct::testUnfixedVarAssignAlt() @@ -256,7 +256,7 @@ void Nested_Struct::testFixedVarAssign() myMacro.Compile(); CPPUNIT_ASSERT_MESSAGE("testFixedVarAssign fails with compile error",!myMacro.HasError() ); SbxVariableRef pNew = myMacro.Run(); - CPPUNIT_ASSERT(pNew->GetInteger() == 13 ); + CPPUNIT_ASSERT_EQUAL(static_cast(13), pNew->GetInteger()); } void Nested_Struct::testFixedVarAssignAlt() diff --git a/basic/qa/cppunit/test_vba.cxx b/basic/qa/cppunit/test_vba.cxx index 7d145e832f02..ee0d6dc1ec3b 100644 --- a/basic/qa/cppunit/test_vba.cxx +++ b/basic/qa/cppunit/test_vba.cxx @@ -81,7 +81,7 @@ void VBATest::testMiscVBAFunctions() fprintf(stderr, "macro returned:\n%s\n", OUStringToOString( pReturn->GetOUString(), RTL_TEXTENCODING_UTF8 ).getStr() ); } CPPUNIT_ASSERT_MESSAGE("No return variable huh?", pReturn != nullptr ); - CPPUNIT_ASSERT_MESSAGE("Result not as expected", pReturn->GetOUString() == "OK" ); + CPPUNIT_ASSERT_EQUAL_MESSAGE("Result not as expected", OUString("OK"), pReturn->GetOUString() ); } } -- cgit