You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ki...@apache.org on 2016/06/10 21:29:47 UTC

svn commit: r1747803 - /poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java

Author: kiwiwings
Date: Fri Jun 10 21:29:47 2016
New Revision: 1747803

URL: http://svn.apache.org/viewvc?rev=1747803&view=rev
Log:
fix text height test

Modified:
    poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java

Modified: poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java?rev=1747803&r1=1747802&r2=1747803&view=diff
==============================================================================
--- poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java (original)
+++ poi/trunk/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java Fri Jun 10 21:29:47 2016
@@ -185,8 +185,10 @@ public class TestXSLFTable {
         XSLFTableCell tc0 = tr.addCell();
         tc0.setText("bla bla bla bla");
         tab.setColumnWidth(0, 50);
-        
-        assertEquals(88, tc0.getTextHeight(), 0);
+
+        // usually text height == 88, but font rendering is plattform dependent
+        // so we use something more reliable
+        assertTrue(tc0.getTextHeight() > 50);
         assertEquals(0, tc0.getLineWidth(), 0);
         
         ppt.close();



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org