diff options
| author | Xisco Fauli <anistenis@gmail.com> | 2016-01-21 20:56:31 +0100 |
|---|---|---|
| committer | jan iversen <jani@documentfoundation.org> | 2016-01-30 07:40:22 +0000 |
| commit | 13d4398820ded5914f635757865e258db2db2b57 (patch) | |
| tree | 356431bab06546113e07462070948388e3d6fc6e /sd/qa/unit/import-tests.cxx | |
| parent | 2416a2965f4d51e1b14b8539436f5f8fc16cc399 (diff) | |
tdf#91762: Check wether the row and the cell have text...
... in order to set the row's min height
Regression from 4f2c8194f485b1527fb4f4dfe23ce804937f1f9c
After this commit, the row's min height was set based only on
the cells containing text in the row, but the problem appeared
when the row didn't have any cell with text.
Change logic to check wether there's text in the cell and in the row.
Now, height in SdImportTest::testRowHeight() is 507 instead of
508 but I can't figure it out why. However, I believe there's
no harm in change the test from 508 to 507 as, visually speaking,
the difference can't be distinguish.
Change-Id: I0b3a14c34eaeaa8e77227860ca290fb79a0302ce
Reviewed-on: https://gerrit.libreoffice.org/21692
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
Diffstat (limited to 'sd/qa/unit/import-tests.cxx')
| -rw-r--r-- | sd/qa/unit/import-tests.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx index 49db223e8d10..9f4c2e343131 100644 --- a/sd/qa/unit/import-tests.cxx +++ b/sd/qa/unit/import-tests.cxx @@ -1093,9 +1093,8 @@ void SdImportTest::testRowHeight() uno::Reference< css::table::XTableRows > xRows( xTable->getRows(), uno::UNO_QUERY_THROW); uno::Reference< beans::XPropertySet > xRefRow( xRows->getByIndex(0), uno::UNO_QUERY_THROW ); xRefRow->getPropertyValue( sHeight ) >>= nHeight; - CPPUNIT_ASSERT_EQUAL( sal_Int32(508), nHeight); + CPPUNIT_ASSERT_EQUAL( sal_Int32(507), nHeight); } - void SdImportTest::testTdf93830() { // Text shape offset was ignored |