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

[lucene-solr] 05/05: @805 Test hardening.

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

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

commit 662ed375e617c4102b304ca4b1c9be366fc15c74
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Mon Sep 7 23:08:32 2020 -0500

    @805 Test hardening.
---
 .../apache/solr/client/solrj/SolrExampleTestsBase.java    | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTestsBase.java b/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTestsBase.java
index f1f4a7e..475517a 100644
--- a/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTestsBase.java
+++ b/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTestsBase.java
@@ -67,9 +67,10 @@ abstract public class SolrExampleTestsBase extends SolrJettyTestBase {
     up.add(doc3);
     up.setCommitWithin(10);
     up.process(client);
-    
-    rsp = client.query(new SolrQuery("*:*"));
-    Assert.assertEquals(0, rsp.getResults().getNumFound());
+
+    // terrible, flakey way to test and we test commitWithin like this a lot already
+//    rsp = client.query(new SolrQuery("*:*"));
+//    Assert.assertEquals(0, rsp.getResults().getNumFound());
     
     // now check that it comes out...
     rsp = client.query(new SolrQuery("id:id3"));
@@ -83,7 +84,7 @@ abstract public class SolrExampleTestsBase extends SolrJettyTestBase {
         break;
       }
       
-      Thread.sleep(50);
+      Thread.sleep(10);
       
       rsp = client.query(new SolrQuery("id:id3"));
     }
@@ -95,7 +96,7 @@ abstract public class SolrExampleTestsBase extends SolrJettyTestBase {
     doc4.addField("id", "id4");
     doc4.addField("name", "doc4");
     doc4.addField("price", 10);
-    client.add(doc4, 50);
+    client.add(doc4, 10);
 
     // now check that it comes out...
     rsp = client.query(new SolrQuery("id:id4"));
@@ -109,7 +110,7 @@ abstract public class SolrExampleTestsBase extends SolrJettyTestBase {
         break;
       }
       
-      Thread.sleep(50);
+      Thread.sleep(10);
       
       rsp = client.query(new SolrQuery("id:id3"));
     }
@@ -152,7 +153,7 @@ abstract public class SolrExampleTestsBase extends SolrJettyTestBase {
     // check if the doc has been deleted every 250 ms for 30 seconds
     TimeOut timeout = new TimeOut(30, TimeUnit.SECONDS, TimeSource.NANO_TIME);
     do {
-      Thread.sleep(250); // wait 250 ms
+      Thread.sleep(50); // wait 250 ms
       
       rsp = client.query(new SolrQuery("id:id3"));
       if (rsp.getResults().getNumFound() == 0) {