You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ho...@apache.org on 2016/12/16 01:04:35 UTC

lucene-solr:jira/solr-5944: Fix stupid test mistake

Repository: lucene-solr
Updated Branches:
  refs/heads/jira/solr-5944 fc436d216 -> c21d8a005


Fix stupid test mistake

Aparently the first 5 times i tried this test the boolean was always false!


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/c21d8a00
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/c21d8a00
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/c21d8a00

Branch: refs/heads/jira/solr-5944
Commit: c21d8a005387eae4d0f0adde4de7e4e465fb73c8
Parents: fc436d2
Author: Chris Hostetter <ho...@apache.org>
Authored: Thu Dec 15 18:04:27 2016 -0700
Committer: Chris Hostetter <ho...@apache.org>
Committed: Thu Dec 15 18:04:27 2016 -0700

----------------------------------------------------------------------
 solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/c21d8a00/solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java b/solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java
index 9206021..bd268d9 100644
--- a/solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java
+++ b/solr/core/src/test/org/apache/solr/cloud/TestSegmentSorting.java
@@ -82,7 +82,7 @@ public class TestSegmentSorting extends SolrCloudTestCase {
       .setProperties(collectionProperties);
 
     if (random().nextBoolean()) {
-      assertEquals(RequestStatusState.COMPLETED, cmd.process(cloudSolrClient));
+      assertTrue( cmd.process(cloudSolrClient).isSuccess() );
     } else { // async
       assertEquals(RequestStatusState.COMPLETED, cmd.processAndWait(cloudSolrClient, 30));
     }