You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@lucene.apache.org by jd...@apache.org on 2012/11/07 17:20:26 UTC

svn commit: r1406698 - /lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java

Author: jdyer
Date: Wed Nov  7 16:20:25 2012
New Revision: 1406698

URL: http://svn.apache.org/viewvc?rev=1406698&view=rev
Log:
SOLR-3856: Better test failure output

Modified:
    lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java

Modified: lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java?rev=1406698&r1=1406697&r2=1406698&view=diff
==============================================================================
--- lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java (original)
+++ lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/AbstractDIHJdbcTestCase.java Wed Nov  7 16:20:25 2012
@@ -126,7 +126,9 @@ public abstract class AbstractDIHJdbcTes
           "//*[@numFound='1']");
     }
     assertQ(req("AddAColumn_s:Added"), "//*[@numFound='" + (totalPeople() + numDeleted) + "']");
-    Assert.assertTrue(totalDatabaseRequests() == numToExpect);
+    Assert.assertTrue("Expecting " + numToExpect
+        + " database calls, but DIH reported " + totalDatabaseRequests(),
+        totalDatabaseRequests() == numToExpect);
   }  
   protected void withChildEntities(boolean cached, boolean checkDatabaseRequests) throws Exception {
     rootTransformerName = random().nextBoolean() ? null : "AddAColumnTransformer";