You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sa...@apache.org on 2017/08/02 13:07:52 UTC

[2/3] lucene-solr:branch_7x: SOLR-10033: TestDistributedSearch: ignore response header warnings section when comparing distributed and control responses

SOLR-10033: TestDistributedSearch: ignore response header warnings section when comparing distributed and control responses


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

Branch: refs/heads/branch_7x
Commit: 3644cd6ecda9a44fc3a852c4e4c99bcc7800ffbd
Parents: f306d7c
Author: Steve Rowe <sa...@apache.org>
Authored: Wed Aug 2 09:07:19 2017 -0400
Committer: Steve Rowe <sa...@apache.org>
Committed: Wed Aug 2 09:07:35 2017 -0400

----------------------------------------------------------------------
 .../java/org/apache/solr/BaseDistributedSearchTestCase.java  | 8 ++++++++
 1 file changed, 8 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/3644cd6e/solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java
----------------------------------------------------------------------
diff --git a/solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java b/solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java
index c46d346..253ba40 100644
--- a/solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java
+++ b/solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java
@@ -929,6 +929,14 @@ public abstract class BaseDistributedSearchTestCase extends SolrTestCaseJ4 {
         }
       }
     }
+    { // we don't care if one has a warnings section in the header and the other doesn't - control vs distrib
+      if (a.getHeader() != null) {
+        a.getHeader().remove("warnings");
+      }
+      if (b.getHeader() != null) {
+        b.getHeader().remove("warnings");
+      }
+    }
     compareSolrResponses(a, b);
   }