diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-31 13:20:41 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-01 09:14:25 +0200 |
| commit | 23b08449736ba825a9c582ba18b7a5fdba178e47 (patch) | |
| tree | 8513c824f1964f84f957a41658f6d173a008c469 /sax/qa | |
| parent | 8e63d451b2aeb646ece98c4e219f92957f4482bd (diff) | |
loplugin: look for CPPUNIT_ASSERT_EQUALS with params swapped
idea originally from either tml or moggi, can't remember which
Change-Id: Id78d75035036d3aa1666e33469c6eeb38f9e624d
Reviewed-on: https://gerrit.libreoffice.org/55126
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sax/qa')
| -rw-r--r-- | sax/qa/cppunit/attributes.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sax/qa/cppunit/attributes.cxx b/sax/qa/cppunit/attributes.cxx index 244da02eb535..c0ffdd7178ab 100644 --- a/sax/qa/cppunit/attributes.cxx +++ b/sax/qa/cppunit/attributes.cxx @@ -46,10 +46,10 @@ void AttributesTest::test() CPPUNIT_ASSERT( xAttributeList->hasAttribute(1) ); CPPUNIT_ASSERT( !xAttributeList->hasAttribute(3) ); - CPPUNIT_ASSERT_EQUAL( xAttributeList->getOptionalValue(2), OUString("2") ); - CPPUNIT_ASSERT_EQUAL( xAttributeList->getOptionalValue(3), OUString() ); + CPPUNIT_ASSERT_EQUAL( OUString("2"), xAttributeList->getOptionalValue(2) ); + CPPUNIT_ASSERT_EQUAL( OUString(), xAttributeList->getOptionalValue(3) ); - CPPUNIT_ASSERT_EQUAL( xAttributeList->getValue(1), OUString("1") ); + CPPUNIT_ASSERT_EQUAL( OUString("1"), xAttributeList->getValue(1) ); CPPUNIT_ASSERT_THROW( xAttributeList->getValue(3), xml::sax::SAXException ); xAttributeList->addUnknown("a", "a"); |