| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| |
|
|
|
|
|
| |
Change-Id: I36ceddb5a8a1f8e96b36a2e851338366cda1808d
Reviewed-on: https://gerrit.libreoffice.org/85662
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
|
|
|
|
| |
Change-Id: I6ebc31181ccfe379be96e979cda3717a991cf513
Reviewed-on: https://gerrit.libreoffice.org/85663
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
Change-Id: I8a0d7a0b51b5c537dbcfa8fdd34e816605b1f32e
Reviewed-on: https://gerrit.libreoffice.org/71573
Reviewed-by: Tomáš Chvátal <tchvatal@suse.cz>
Tested-by: Tomáš Chvátal <tchvatal@suse.cz>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
doc/other_documentation.dox:
* add instructions to build documentation
* replace a dead link with it's archived version on WaybackMachine
doc/Money.dox:
* replace `configure.in` with `configure.ac`
* update MoneyApp.cpp with the version at `examples/money/MoneyApp.cpp`
* remove all occurrences of `#include "stdafx.h"` (stdafx.h does not exist in this example)
* replace all occurrences of CppUnit:: with CPPUNIT_NS::
* Undo a fix in commit 652d51e653568fbf652a248c7b9e01e72e6ec50f
* This commit fixed the following error:
```
Money( double amount, std::string currency )
: m_amount( amount )
, m_currency( m_currency )
```
But this error was introduced on purpose, so that the user can see that an assertion failed
* fix some spelling errors
* other minor edits
examples/money/MoneyTest.h:
* Updated to match doc/Money.dox
Change-Id: I4e90b1a3afadab94f7112c36fcb0d1c467169269
Reviewed-on: https://gerrit.libreoffice.org/67787
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
| | |
|
| |
|
|
|
|
|
|
|
| |
For more info, see: https://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation
Change-Id: If27065b6d9c9245717ca24e0c0e1fa4c29bc3e5c
Reviewed-on: https://gerrit.libreoffice.org/67808
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
|
| |
belongs to the if body
|
| |
|
|
|
|
|
|
|
|
| |
Provide an extension trait message_to_string that allows client code
to call ASSERT_MESSAGE with their own string types.
Also provide a default extension trait that accepts std::ostream
messages.
Change-Id: I516f97063c34d86bc91c40e0ec147d5393e7b6ea
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This helps avoid issues like <https://gerrit.libreoffice.org/plugins/gitiles/
core/+/9808486a89c6368f836579f8d8c0dda63fd0063c%5E%21> "Avoid
-Werror,-Wimplicit-fallthrough with clang-cl ...where CPPUNIT_FAIL is marked as
noreturn only for __GNUC__".
Change-Id: Idb33af7375f103f2dd7a7b4c3dbf20ce731b17ad
Reviewed-on: https://gerrit.libreoffice.org/67247
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
| |
|
|
|
|
|
|
|
|
| |
This is similar to googletest's TEST_F() macro, allows to avoid spelling
out the test name in a suite only once, not 3 times.
Change-Id: I90804d271d046d8158678617d8db75ed6ca7c420
Reviewed-on: https://gerrit.libreoffice.org/61732
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...when an implicitly-defined copy function is used that may in a future C++
standard be defined as deleted because of the user-declared destructor. Just
declare all the four copy/move functions as defaulted for a consistent
interface.
(Originally addressed with LibreOffice commit
<https://gerrit.libreoffice.org/58042> "external/cppunit: silence
-Werror=deprecated-copy (GCC trunk towards GCC 9)".)
Change-Id: Ie38ac3a613e6fbc2e4045cb5b14c3f6d167c79c5
Reviewed-on: https://gerrit.libreoffice.org/58690
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
The cppunit2junit XSL stylesheet has a hardcoded name for the testsuite.
Instead of putting a meaningless name, allow users to parametrize this
name when doing the transformation.
Change-Id: I2d0ebaf92cd0aa4d8ff3b3a2c423f4a8edd94807
Signed-off-by: Simon Barth <Simon.Pe.Barth@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/46357
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
| |
|
|
|
|
|
| |
Change-Id: If42a50386402b7a601268cf8db80236c147009c1
Reviewed-on: https://gerrit.libreoffice.org/42206
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
|
|
|
|
|
|
|
|
| |
...instead of as characters, as those values often represent numerical values
(e.g., sal_uInt8 aka unsigned char in LibreOffice), and often even have values
outside the printing ASCII range
Change-Id: I72727e98f5af616f6b0c03c57fc1119c0c11c1fc
Reviewed-on: https://gerrit.libreoffice.org/40138
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
in my last commit Gerrit's web interface changed the encoding
of the file. The page doesn't look correct any more.
Change-Id: I5de59620ef4ca78a009da2bbf2c6fe37af0dd54c
Reviewed-on: https://gerrit.libreoffice.org/35401
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
| |
|
|
|
|
|
| |
Change-Id: I03c1e1118c0574863f9460d1d7fc543b44d59efe
Reviewed-on: https://gerrit.libreoffice.org/35152
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
| |
|
|
|
|
|
| |
Change-Id: I58762dfb08c6f85ccb11d0b634c738606cd6379f
Reviewed-on: https://gerrit.libreoffice.org/34649
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows to execute the same test with different parameters and
treats each execution as an own test. The change consists of two parts,
the TestCaller can now handle any callable which also makes it easy to
generate programatically more complex test cases as well as the new
CPPUNIT_TEST_PARAMETERIZED macro. That macro takes the test name as well
as an iteratable, e.g. std::initializer_list.
An example for this usage is:
class SimpleTest : public CppUnit::TestFixture
{
public:
CPPUNIT_TEST_SUITE(SimpleTest);
CPPUNIT_TEST_PARAMETERIZED(test, {1, 2, 3, 4});
CPPUNIT_TEST_SUITE_END();
void test(int i)
{
CPPUNIT_ASSERT(i < 5);
}
};
which will execute test 4 times with the values 1 to 4.
|
| |
|
|
| |
This allows us to pass in any callable e.g. results of std::bind.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
... so check just for existence.
|
| | |
|