summaryrefslogtreecommitdiff
path: root/sal/cppunittester
AgeCommit message (Collapse)AuthorFilesLines
2018-10-24remove more rtl::OUString and OString prefixesNoel Grandin1-9/+9
which seem to have snuck back in since the great rounds of removals. Change-Id: I85f7f5f4801c0b48dae8b50f51f83595b286d6a1 Reviewed-on: https://gerrit.libreoffice.org/62229 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-10-12loplugin:constfields in reportdesign,sal,saxNoel Grandin1-1/+1
and improve the rewriter so I spend less time fixing formatting Change-Id: Ic2a6e5e31a5a202d2d02a47d77c484a57a5ec514 Reviewed-on: https://gerrit.libreoffice.org/61676 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-08-29Replace find_if with proper quantifier algorithmsArkadiy Illarionov1-1/+1
Change-Id: Icc820a47ac891c358883f9c01224f676c58fdd11 Reviewed-on: https://gerrit.libreoffice.org/59744 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-08-06Add missing sal/log.hxx headersGabor Kelemen1-0/+1
rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx> (and don't make use of it themselves), but many other files happen to depend on it. This is a continuation of commit 6ff2d84ade299cb3d14d4110e4cf1a4b8070c030 to be able to remove those unneeded includes. This commit adds missing headers to every file found by: grep -FwL sal/log.hxx $(git grep -Elw 'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF|SAL_DETAIL_LOG_STREAM|SAL_WHERE|SAL_STREAM|SAL_DEBUG') in files formerly omitted for oversight or non-cxx extension Change-Id: I327c573f44076c6ccfecf737eafccba2da72e1bd Reviewed-on: https://gerrit.libreoffice.org/58600 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2018-02-21New loplugin:nestedunnamedStephan Bergmann1-4/+0
Change-Id: Ifb434589ef08428ce609bc7a40b015d4df13224c Reviewed-on: https://gerrit.libreoffice.org/50048 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-01-12More loplugin:cstylecast: salStephan Bergmann1-2/+2
auto-rewrite with <https://gerrit.libreoffice.org/#/c/47798/> "Enable loplugin:cstylecast for some more cases" plus solenv/clang-format/reformat-formatted-files Change-Id: I7d89b011464ba5d2dd12e04d5fc9f65cb4daebde
2017-10-23loplugin:includeform: salStephan Bergmann1-22/+22
Change-Id: I539ca8b9dee5edc5fc2282a2b9b0ffd78bad8b11
2017-09-18Some more WIN32_LEAN_AND_MEANMike Kaganski1-0/+3
Change-Id: Iadb0ebb66809c192fb817b8c7cf2f8cdb4d0b874 Reviewed-on: https://gerrit.libreoffice.org/42419 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2017-04-27sal: cppunittester: verify that CPPUNIT_TEST_NAME contains a testMichael Stahl1-3/+13
Fail the test if CPPUNIT_TEST_NAME does not contain at least one existing test, which is much better than silently reporting OK(0). Change-Id: If2137764b24aa52dc35fe2368ee8fe38cb7dbece
2017-03-22Use rtl::isAscii* instead of ctype.h is* (and fix passing plain char)Stephan Bergmann1-1/+2
Change-Id: I93eeb586310cb1896b067b7c665e1fdee92d47b4
2016-08-19coverity#1371449 Uninitialized scalar fieldCaolán McNamara1-1/+4
Change-Id: I9e43b01f40cc325b83d23903fcec9010f848fb28
2016-07-01cppunittester: always print execution time of each testMichael Stahl1-6/+3
Not much point in disabling this by default, nobody but the author will find it. Change-Id: I637b214644d2cc7bc70b8bca4f214955355ea951
2016-04-11tdf#94306 replace boost::noncopyable r.. to sdextJochen Nitschke1-4/+8
... in modules reportdesign to sdext Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes. Make some overloaded ctors explicit (most in sd slidesorter). Add deleted copy-assignment in sc/inc/chart2uno.hxx. Change-Id: I21d4209f0ddb00063ca827474516a05ab4bb2f9a Reviewed-on: https://gerrit.libreoffice.org/23970 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-2/+2
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-16use consistent #define checks for the Windows platformNoel Grandin1-1/+1
stage 1 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f Reviewed-on: https://gerrit.libreoffice.org/22390 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-02-08cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I74585aa5a50cd335f4fd0a3ba4fed230b98993c3
2016-01-26fix the new CPPUNIT_TEST_NAME codeMarkus Mohrhard1-1/+5
Change-Id: I45260555778a733a8c8ba9e39e9ffcce0c9ab78d Reviewed-on: https://gerrit.libreoffice.org/21818 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-01-26support unnamed namespaces with CPPUNIT_TEST_NAMEMarkus Mohrhard1-1/+17
Change-Id: I1ce50ce0ce8a4a461d1b2a34de132cbf57dd7d25 Reviewed-on: https://gerrit.libreoffice.org/21802 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann1-3/+3
Change-Id: I1bc6c87fcd6e5e96362623be94c59be216a3b2b8
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-6/+6
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-09-30Fix typosAndrea Gelmini1-1/+1
Change-Id: Ia41f4f0ca30ae3346d0720271478ec5bcdab797b Reviewed-on: https://gerrit.libreoffice.org/18967 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-08-31sal: oops, forgot to commit that bitMichael Stahl1-1/+1
Change-Id: Iee85958a6589624bc9a61242fca41214bdd1df85
2015-08-31sal: don't use --target as parameter to cppunittesterMichael Stahl1-4/+8
CppunitTest_libreofficekit_tiledrendering hangs because the soffice_main exits due to the unrecognized command line arg --target. Use a -env:VARIABLE argument instead, which is filtered out by rtl_getAppCommandArg() so does not reach the soffice_main code. (regression from 87514b0907dfbb479e2646b5ff951c68babf3417) Change-Id: I2c801305398dccfb447e4e5c44726f42bf2a72ef
2015-08-31sal: loplugin:stringconstantMiklos Vajna1-1/+1
Change-Id: Ib4e7f4f71494abd654a70cf8ba8b0ca0afcd7df1
2015-08-31log resource usage of unit tests on UNX platformsMarkus Mohrhard1-0/+50
Change-Id: I3788eae60f73bc42488bf2e4961922962f7e505b Reviewed-on: https://gerrit.libreoffice.org/18155 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-06-17Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe1-2/+2
This may reduce some degree of dependency on boost. Done by running a script like: git grep -l '#include *.boost/scoped_array.hpp.' \ | xargs sed -i -e 's@#include *.boost/scoped_array.hpp.@#include <memory>@' git grep -l '\(boost::\)\?scoped_array<\([^<>]*\)>' \ | xargs sed -i -e 's/\(boost::\)\?scoped_array<\([^<>]*\)>/std::unique_ptr<\2[]>/' ... and then killing duplicate or unnecessary includes, while changing manually m_xOutlineStylesCandidates in xmloff/source/text/txtimp.cxx, extensions/source/ole/unoconversionutilities.hxx, and extensions/source/ole/oleobjw.cxx. Change-Id: I3955ed3ad99b94499a7bd0e6e3a09078771f9bfd Reviewed-on: https://gerrit.libreoffice.org/16289 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-09coverity#1297235 Uncaught exceptionCaolán McNamara1-65/+74
Change-Id: I6b135f6d47352c1fbb729788848913402b7bb742
2015-04-30provide a way to execute single test case from test suiteMarkus Mohrhard1-2/+31
CPPUNIT_TEST_NAME is the environment variable that needs to be set and contain the name of the tests. The test names need to be fully qualified to be recognized. Examples: CPPUNIT_TEST_NAME="ScFiltersTest::testOutlineODS ScFiltersTest::testRangeNameXLS" make CppunitTest_sc_subsequent_filters_test CPPUNIT_TEST_NAME="ScFiltersTest" make CppunitTest_sc_subsequent_filters_test Change-Id: I78a8a076638e19c918ca1c411086bb353f5100a2 Reviewed-on: https://gerrit.libreoffice.org/15579 Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-03-27loplugin:staticfunctionNoel Grandin1-3/+3
Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
2015-02-05convert all remaining BOOST_STATIC_ASSERT to static_assertCaolán McNamara1-2/+1
and we can include a few less headers Change-Id: Id742849ff4c1c37a2b861aa3d6ab823f00ea87f8
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini1-1/+1
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-15cppunittester: make tracking down temp file leaks easierMichael Stahl1-0/+3
Temp files created before first test method don't see LO_TESTNAME Change-Id: Iff74abf574f5151980f463646f11c1b32ad0c241
2014-06-04Keep TestFixture alive until outputter is done with itStephan Bergmann1-15/+19
Change-Id: Ifd921efa971f7a9a5c47f42c5d56a3deb1c3a0b3
2014-06-04Cover ctors/dtors of our TestFixtures under the protectors, tooStephan Bergmann1-3/+8
Change-Id: I4c3030e5a205bd102d0427e74d2fe3b828ff5ae6
2014-05-28dlclose confuses LeakSanitizerStephan Bergmann1-4/+3
...so just do not bother unloading the protector libs again Change-Id: I33caa7beaac3b5e6c4a4836061def24fc5372b70
2014-05-24cppunit: always compile with debugging symbols on windows.Michael Meeks1-0/+1
Also cleanup some makefile pieces and give better error warnings. Change-Id: I9f20d60f26d5828f60664941946c266c7d166f32
2014-05-20Win32 - warn of extraordinary behaviour with LoadLibrary and long PATHsMichael Meeks1-0/+8
Change-Id: I0a8ee157aa548a5b04df67a2355e3f4fd15e84cd
2014-05-06Avoid the annoying line wrappingTor Lillqvist1-1/+3
Change-Id: I3ac2f0bf676491231bcd60eb91e8a36bc6c00620 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-04-29show the failure message again at the end of the testMarkus Mohrhard1-4/+1
Change-Id: Ia0bcbf0943aaa20dc5333dc4d18030ecf88e594e
2014-04-24Add a test listener that outputs failures to stderr as they happenTor Lillqvist1-1/+40
Makes them intermix properly with debugging output from the tested functionality. After the test has run, don't repeat the failure messages, just print a short summary. Change-Id: I6b491d3a34094df2da11df5e404adb63299de342
2014-04-24Add informative comment about cross-compiling to iOS in particularTor Lillqvist1-0/+8
Change-Id: I7c38efdafea13bb5ae5e8a5e1cb07e77853b5c4b
2014-04-24Avoid possible memory leaks in case of exceptionsTakeshi Abe1-5/+5
Change-Id: I047fd88a89900153089a55b6af123f11fb8bde55
2014-04-03Properly cover in #ifdef TIMETESTSStephan Bergmann1-0/+2
Change-Id: Iff8f992e5260150b9ba108698b506f5a9ec7dea2
2014-03-31Move invariant writes outside loopStephan Bergmann1-2/+2
Change-Id: Ia254fe382a00be13a631fcc1b23f940352d8d3ba
2014-03-30decorate tempfile name's to help identify leaks' authorNorbert Thiebaud1-0/+37
unottols TempFile class check in debug bod for and env variable LO_TESTNAME.. if present it use that as the root for the tempfile filename. cppunitteset framework is augmented so that it export such LO_TESNAME using the cppunit testName(), after replacement of non-alphnum characters by _ Change-Id: Iebb9545d3bd789083afbeaf4c64eab086b56049a
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann1-3/+3
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2013-12-02Drop duplicate #includeTakeshi Abe1-1/+0
Change-Id: Ife8ebbb5e46704c5d2ff46cc345b4ed926e7dc12
2012-12-10-Werror,-Wtautological-constant-out-of-range-compareStephan Bergmann1-6/+4
Change-Id: I99ae1590067d66f0af968833bc75d773ad3cb15c
2012-11-22Fail fastStephan Bergmann1-4/+2
Change-Id: I17d6ab71ec7bcf40dccb2c9ffe3d9f2e6b10f01c
2012-11-21android: fix cppunit testingMichael Meeks1-1/+2