You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by ho...@apache.org on 2023/05/26 00:30:47 UTC

[solr] branch main updated: SOLR-16819: Fix HttpPartitionTest

This is an automated email from the ASF dual-hosted git repository.

hossman pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr.git


The following commit(s) were added to refs/heads/main by this push:
     new 358d01d3354 SOLR-16819: Fix HttpPartitionTest
358d01d3354 is described below

commit 358d01d3354e2489d8399f800b2cc1a6d156ae50
Author: Chris Hostetter <ho...@apache.org>
AuthorDate: Thu May 25 17:30:36 2023 -0700

    SOLR-16819: Fix HttpPartitionTest
    
    Completes partial changes made in SOLR-10466
---
 solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java b/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
index b14ec3dd5ae..c1092da34c7 100644
--- a/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
@@ -338,7 +338,7 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     String testCollectionName = "c8n_1x3";
     createCollectionRetry(testCollectionName, "conf1", 1, 3);
 
-    sendDoc(1);
+    sendDoc(testCollectionName, 1);
 
     List<Replica> notLeaders =
         ensureAllReplicasAreActive(testCollectionName, "shard1", 1, 3, maxWaitSecsToSeeAllActive);
@@ -382,7 +382,7 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     notLeaders =
         ensureAllReplicasAreActive(testCollectionName, "shard1", 1, 3, maxWaitSecsToSeeAllActive);
 
-    sendDoc(4);
+    sendDoc(testCollectionName, 4);
 
     assertDocsExistInAllReplicas(notLeaders, testCollectionName, 1, 4);
 
@@ -555,10 +555,6 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     }
   }
 
-  protected int sendDoc(int docId) throws Exception {
-    return sendDoc(docId, cloudClient, cloudClient.getDefaultCollection());
-  }
-
   protected int sendDoc(String collectionName, int docId) throws Exception {
     return sendDoc(docId, cloudClient, collectionName);
   }