You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by th...@apache.org on 2020/09/16 14:39:19 UTC

[lucene-solr] branch reference_impl_dev updated: Minor updates to wiremock tests

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

thelabdude pushed a commit to branch reference_impl_dev
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/reference_impl_dev by this push:
     new 6844a66  Minor updates to wiremock tests
6844a66 is described below

commit 6844a660f2abe98bf63f0c5dd4dd437578d38e46
Author: Timothy Potter <th...@gmail.com>
AuthorDate: Wed Sep 16 08:39:05 2020 -0600

    Minor updates to wiremock tests
---
 .../apache/solr/client/solrj/impl/BaseSolrClientWireMockTest.java | 2 +-
 .../solr/client/solrj/impl/CloudHttp2SolrClientWireMockTest.java  | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/impl/BaseSolrClientWireMockTest.java b/solr/solrj/src/test/org/apache/solr/client/solrj/impl/BaseSolrClientWireMockTest.java
index 4f41b0b..66b4535 100644
--- a/solr/solrj/src/test/org/apache/solr/client/solrj/impl/BaseSolrClientWireMockTest.java
+++ b/solr/solrj/src/test/org/apache/solr/client/solrj/impl/BaseSolrClientWireMockTest.java
@@ -292,7 +292,7 @@ public abstract class BaseSolrClientWireMockTest extends SolrTestCase {
     UpdateRequest req = new UpdateRequest();
     for (int i = 0; i < numDocs; i++) {
       SolrInputDocument doc = new SolrInputDocument();
-      doc.addField("id", threadName+1000+i);
+      doc.addField("id", threadName+(1000+i));
       req.add(doc);
     }
     return req;
diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/impl/CloudHttp2SolrClientWireMockTest.java b/solr/solrj/src/test/org/apache/solr/client/solrj/impl/CloudHttp2SolrClientWireMockTest.java
index 0a0e14a..4a6699a 100644
--- a/solr/solrj/src/test/org/apache/solr/client/solrj/impl/CloudHttp2SolrClientWireMockTest.java
+++ b/solr/solrj/src/test/org/apache/solr/client/solrj/impl/CloudHttp2SolrClientWireMockTest.java
@@ -23,7 +23,6 @@ import java.util.Map;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
-import java.util.concurrent.TimeUnit;
 
 import com.github.tomakehurst.wiremock.stubbing.ServeEvent;
 import org.apache.solr.client.solrj.SolrQuery;
@@ -90,7 +89,8 @@ public class CloudHttp2SolrClientWireMockTest extends BaseSolrClientWireMockTest
     final String shard1Route = mockSolr.baseUrl()+SHARD1_PATH+"/";
     final String shard2Route = mockSolr.baseUrl()+SHARD2_PATH+"/";
 
-    UpdateRequest ur = buildUpdateRequest(15);
+    final int numDocs = 20;
+    UpdateRequest ur = buildUpdateRequest(numDocs);
     Map<String,List<String>> urlMap = testClient.buildUrlMap(mockDocCollection, new ShufflingReplicaListTransformer(random()));
     assertEquals(2, urlMap.size());
     List<String> shard1 = urlMap.get("shard1");
@@ -110,8 +110,8 @@ public class CloudHttp2SolrClientWireMockTest extends BaseSolrClientWireMockTest
 
     final String threadName = Thread.currentThread().getName();
     ur = new UpdateRequest();
-    for (int i=0; i < 15; i++) {
-      ur.deleteById(threadName+1000+i);
+    for (int i=0; i < numDocs; i++) {
+      ur.deleteById(threadName+(1000+i));
     }
     routes = ur.getRoutesToCollection(mockDocCollection.getRouter(), mockDocCollection, urlMap, ur.getParams(), "id");
     assertEquals(2, routes.size());