diff options
author | Moaz <moaz.eldefrawy@gmail.com> | 2021-02-21 20:22:40 +0200 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2021-02-25 07:50:29 +0100 |
commit | a9a350c13d8fb454c3f1db7ca6a7f10c1e7a19bb (patch) | |
tree | c3115fa9b7018eb6629cdaf607ef4a49adb147dc /lotuswordpro/qa | |
parent | befc0c2b0e0b3bf293a2446d71e38717ed53985b (diff) |
tdf#139734 Remove redundant asserts after MacrosTest::loadFromDesktop
MacrosTest::loadFromDesktop itself asserts on its return value.
Thus, the additional checks in unit tests are redundant, and only create
noise unrelated to the tested functionality.
Change-Id: If616001b296afdde38f5a23ececee3d44b4a395d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111290
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'lotuswordpro/qa')
-rw-r--r-- | lotuswordpro/qa/cppunit/import_test.cxx | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/lotuswordpro/qa/cppunit/import_test.cxx b/lotuswordpro/qa/cppunit/import_test.cxx index efd40267a9f9..fe2128f72ab5 100644 --- a/lotuswordpro/qa/cppunit/import_test.cxx +++ b/lotuswordpro/qa/cppunit/import_test.cxx @@ -57,8 +57,6 @@ CPPUNIT_TEST_FIXTURE(LotusWordProTest, testTdf129993) { OUString aURL = m_directories.getURLFromSrc(DATA_DIRECTORY) + "tdf129993.lwp"; getComponent() = loadFromDesktop(aURL); - CPPUNIT_ASSERT(getComponent().is()); - uno::Reference<text::XTextDocument> textDocument(getComponent(), uno::UNO_QUERY); uno::Reference<container::XEnumerationAccess> xParaEnumAccess(textDocument->getText(), uno::UNO_QUERY); @@ -74,8 +72,6 @@ CPPUNIT_TEST_FIXTURE(LotusWordProTest, paragraphProperties) { OUString aURL = m_directories.getURLFromSrc(DATA_DIRECTORY) + "paragraphProperties.lwp"; getComponent() = loadFromDesktop(aURL); - CPPUNIT_ASSERT(getComponent().is()); - uno::Reference<text::XTextDocument> textDocument(getComponent(), uno::UNO_QUERY); uno::Reference<container::XEnumerationAccess> xParaEnumAccess(textDocument->getText(), uno::UNO_QUERY); |