SOLR-6956: A better message for DeleteReplicaTest failures

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1660453 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Shalin Shekhar Mangar 2015-02-17 17:31:24 +00:00
parent 26fdf474dd
commit aa2bc27bac
1 changed files with 1 additions and 1 deletions

View File

@ -117,7 +117,7 @@ public class DeleteReplicaTest extends AbstractFullDistribZkTestBase {
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'",