You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2013/11/03 11:11:10 UTC

svn commit: r1538330 - /poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java

Author: centic
Date: Sun Nov  3 10:11:09 2013
New Revision: 1538330

URL: http://svn.apache.org/r1538330
Log:
Add more output to assertion in test-case that is sometimes failing, see e.g. Jenkins build #170...

Modified:
    poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java

Modified: poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java?rev=1538330&r1=1538329&r2=1538330&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java (original)
+++ poi/trunk/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBug47563.java Sun Nov  3 10:11:09 2013
@@ -65,7 +65,7 @@ public class TestBug47563 extends TestCa
 
 				par.insertBefore("" + (rowIdx * row.numCells() + colIdx));
 				par.sanityCheck();
-
+				
 				row.sanityCheck();
 				table.sanityCheck();
 				range.sanityCheck();
@@ -77,10 +77,9 @@ public class TestBug47563 extends TestCa
 		int mustBeAfter = 0;
 		for (int i = 0; i < rows * columns; i++) {
 			int next = text.indexOf(Integer.toString(i), mustBeAfter);
-			assertFalse(next == -1);
+			assertTrue("Test with " + rows + "/" + columns + ": Should not find " + i + " but found it at " + next + " in " + text, 
+					next != -1);
 			mustBeAfter = next;
 		}
 	}
-
-
 }



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