Remove comment and check that both strict iterators are at end

boeker · boeker · commit ec225076c26c · 2026-03-06T12:23:46.000Z
diff --git a/searchlib/src/tests/queryeval/array_bool/array_bool_test.cpp b/searchlib/src/tests/queryeval/array_bool/array_bool_test.cpp
@@ -538,8 +538,7 @@ void verify_same_strict_iterator_behavior(SearchIterator& it1, TermFieldMatchDat
         EXPECT_EQ(docid, it2.seekNext(old_docid + 1));
 
     }
-    // An iterator must not necessarily update the docid if we do not have a match
-    //EXPECT_EQ(it1.isAtEnd(), it2.isAtEnd());
+    EXPECT_EQ(it1.isAtEnd(), it2.isAtEnd());
 }
 
 void verify_same_iterator_behavior(SearchIterator& it1, TermFieldMatchData* tfmd1, SearchIterator& it2, TermFieldMatchData* tfmd2, uint32_t begin_id, uint32_t end_id, bool strict) {

Original file line number	Diff line number	Diff line change
`@@ -538,8 +538,7 @@ void verify_same_strict_iterator_behavior(SearchIterator& it1, TermFieldMatchDat`
`538`	`538`	`EXPECT_EQ(docid, it2.seekNext(old_docid + 1));`
`539`	`539`
`540`	`540`	`}`
`541`		`- // An iterator must not necessarily update the docid if we do not have a match`
`542`		`- //EXPECT_EQ(it1.isAtEnd(), it2.isAtEnd());`
	`541`	`+ EXPECT_EQ(it1.isAtEnd(), it2.isAtEnd());`
`543`	`542`	`}`
`544`	`543`
`545`	`544`	`void verify_same_iterator_behavior(SearchIterator& it1, TermFieldMatchData* tfmd1, SearchIterator& it2, TermFieldMatchData* tfmd2, uint32_t begin_id, uint32_t end_id, bool strict) {`