From 0224453c61f9479d621d75c7fdcb59ebaacb3f5f Mon Sep 17 00:00:00 2001 From: Tor Egge Date: Tue, 12 Mar 2024 13:48:13 +0100 Subject: Test extra initialized element in source selector. --- .../src/tests/attribute/sourceselector/sourceselector_test.cpp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/searchlib/src/tests/attribute/sourceselector/sourceselector_test.cpp b/searchlib/src/tests/attribute/sourceselector/sourceselector_test.cpp index b2711a5949e..2d6806257e6 100644 --- a/searchlib/src/tests/attribute/sourceselector/sourceselector_test.cpp +++ b/searchlib/src/tests/attribute/sourceselector/sourceselector_test.cpp @@ -59,8 +59,14 @@ TEST_F(SourceSelectorTest, test_fixed) FixedSourceSelector selector(default_source, base_file_name, 10); EXPECT_EQ(default_source, selector.getDefaultSource()); EXPECT_EQ(10u, selector.getDocIdLimit()); -// EXPECT_EQ(default_source, selector.createIterator()->getSource(maxDocId + 1)); setSources(selector); + /* + * One extra element beyond highest explicitly set element is + * initialized to accommodate a match loop optimization. See + * setSource() and reserve() member functions in + * FixedSourceSelector for details. + */ + EXPECT_EQ(default_source, selector.createIterator()->getSource(maxDocId + 1)); testSourceSelector(docSource, sz, selector.getDefaultSource(), selector, false); EXPECT_EQ(maxDocId+1, selector.getDocIdLimit()); } -- cgit v1.2.3