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/07/29 01:38:44 UTC

[lucene-solr] branch reference_impl_dev updated: @443 This is now fixed.

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


The following commit(s) were added to refs/heads/reference_impl_dev by this push:
     new c46691e  @443 This is now fixed.
c46691e is described below

commit c46691e21c10c2b3cb0db523b5ab78b70eabf475
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Tue Jul 28 20:38:30 2020 -0500

    @443 This is now fixed.
---
 solr/core/src/test/org/apache/solr/TestHighlightDedupGrouping.java | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/solr/core/src/test/org/apache/solr/TestHighlightDedupGrouping.java b/solr/core/src/test/org/apache/solr/TestHighlightDedupGrouping.java
index 88b3d96..9f36d8e 100644
--- a/solr/core/src/test/org/apache/solr/TestHighlightDedupGrouping.java
+++ b/solr/core/src/test/org/apache/solr/TestHighlightDedupGrouping.java
@@ -111,9 +111,7 @@ public class TestHighlightDedupGrouping extends BaseDistributedSearchTestCase {
           ,"hl", "true", "hl.fl", "*", "hl.requireFieldMatch", "true"
           ));
       // The number of highlit documents should be the same as the de-duplicated docs for this group
-      // but there can be a one off diff with distrib
-      int diff = Math.abs(docsInGroup[group] - rsp.getHighlighting().values().size());
-      assertTrue(diff <= 1);
+      assertEquals(docsInGroup[group], rsp.getHighlighting().values().size());
     }
   }