svn commit: r1660453 - /lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java

[sh...@apache.org]

Author: shalin
Date: Tue Feb 17 17:31:24 2015
New Revision: 1660453

URL: http://svn.apache.org/r1660453
Log:
SOLR-6956: A better message for DeleteReplicaTest failures

Modified:
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java?rev=1660453&r1=1660452&r2=1660453&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java Tue Feb 17 17:31:24 2015
@@ -117,7 +117,7 @@ public class DeleteReplicaTest extends A
         fail("Should have thrown an exception here because the replica is NOT down");
       } catch (SolrException se) {
         assertEquals("Should see 400 here ", se.code(), 400);
-        assertTrue("Should have had a good message here", se.getMessage().contains("with onlyIfDown='true', but state is 'active'"));
+        assertTrue("Expected DeleteReplica to fail because node state is 'active' but returned message was: " + se.getMessage(), se.getMessage().contains("with onlyIfDown='true', but state is 'active'"));
         // This bit is a little weak in that if we're screwing up and actually deleting the replica, we might get back
         // here _before_ the datadir is deleted. But I'd rather not introduce a delay here.
         assertTrue("dataDir for " + replica1.getName() + " should NOT have been deleted by deleteReplica API with onlyIfDown='true'",