summaryrefslogtreecommitdiffstats
path: root/searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahooinc.com>2021-11-12 13:09:14 +0000
committerArne H Juul <arnej@yahooinc.com>2021-11-12 13:09:14 +0000
commit9fd989bfb6490b364026ba3031648aedf7f4ad8a (patch)
tree8a0fbd09f9472405a4767774ea3bb076e76a3753 /searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp
parent41012921305c4f6a8a9f6c153f5fb106cb9e8945 (diff)
use more accessors
* use getDocId() instead of directly accessing _docId some places * use getRank() instead of directly accessing _rankValue some places
Diffstat (limited to 'searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp')
-rw-r--r--searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp34
1 files changed, 17 insertions, 17 deletions
diff --git a/searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp b/searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp
index 4076194542a..de54386e4af 100644
--- a/searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp
+++ b/searchlib/src/tests/attribute/searchcontext/searchcontext_test.cpp
@@ -487,7 +487,7 @@ SearchContextTest::checkResultSet(const ResultSet & rs, const DocSet & expected,
ASSERT_TRUE(array != nullptr);
uint32_t i = 0;
for (auto iter = expected.begin(); iter != expected.end(); ++iter, ++i) {
- EXPECT_TRUE(array[i]._docId == *iter);
+ EXPECT_TRUE(array[i].getDocId() == *iter);
}
}
}
@@ -1517,10 +1517,10 @@ SearchContextTest::requireThatSearchIsWorkingAfterClearDoc(const vespalib::strin
EXPECT_EQUAL(4u, rs->getNumHits());
ASSERT_TRUE(4u == rs->getNumHits());
const RankedHit * array = rs->getArray();
- EXPECT_EQUAL(1u, array[0]._docId);
- EXPECT_EQUAL(2u, array[1]._docId);
- EXPECT_EQUAL(3u, array[2]._docId);
- EXPECT_EQUAL(4u, array[3]._docId);
+ EXPECT_EQUAL(1u, array[0].getDocId());
+ EXPECT_EQUAL(2u, array[1].getDocId());
+ EXPECT_EQUAL(3u, array[2].getDocId());
+ EXPECT_EQUAL(4u, array[3].getDocId());
}
a->clearDoc(1);
a->clearDoc(3);
@@ -1529,8 +1529,8 @@ SearchContextTest::requireThatSearchIsWorkingAfterClearDoc(const vespalib::strin
ResultSetPtr rs = performSearch(v, term);
EXPECT_EQUAL(2u, rs->getNumHits());
const RankedHit * array = rs->getArray();
- EXPECT_EQUAL(2u, array[0]._docId);
- EXPECT_EQUAL(4u, array[1]._docId);
+ EXPECT_EQUAL(2u, array[0].getDocId());
+ EXPECT_EQUAL(4u, array[1].getDocId());
}
}
@@ -1578,9 +1578,9 @@ SearchContextTest::requireThatSearchIsWorkingAfterLoadAndClearDoc(const vespalib
const RankedHit * array = rs->getArray();
for (uint32_t i = 0; i < 14; ++i) {
if (i < 5) {
- EXPECT_EQUAL(i + 1, array[i]._docId);
+ EXPECT_EQUAL(i + 1, array[i].getDocId());
} else
- EXPECT_EQUAL(i + 2, array[i]._docId);
+ EXPECT_EQUAL(i + 2, array[i].getDocId());
}
}
ValueType buf;
@@ -1682,15 +1682,15 @@ SearchContextTest::requireThatFlagAttributeIsWorkingWhenNewDocsAreAdded()
{
ResultSetPtr rs = performSearch(fa, "<24");
EXPECT_EQUAL(2u, rs->getNumHits());
- EXPECT_EQUAL(1u, rs->getArray()[0]._docId);
- EXPECT_EQUAL(2u, rs->getArray()[1]._docId);
+ EXPECT_EQUAL(1u, rs->getArray()[0].getDocId());
+ EXPECT_EQUAL(2u, rs->getArray()[1].getDocId());
}
{
ResultSetPtr rs = performSearch(fa, "24");
EXPECT_EQUAL(3u, rs->getNumHits());
- EXPECT_EQUAL(1u, rs->getArray()[0]._docId);
- EXPECT_EQUAL(2u, rs->getArray()[1]._docId);
- EXPECT_EQUAL(4u, rs->getArray()[2]._docId);
+ EXPECT_EQUAL(1u, rs->getArray()[0].getDocId());
+ EXPECT_EQUAL(2u, rs->getArray()[1].getDocId());
+ EXPECT_EQUAL(4u, rs->getArray()[2].getDocId());
}
}
{
@@ -1717,15 +1717,15 @@ SearchContextTest::requireThatFlagAttributeIsWorkingWhenNewDocsAreAdded()
EXPECT_EQUAL(exp50.size(), rs1->getNumHits());
EXPECT_EQUAL(exp50.size(), rs2->getNumHits());
for (size_t j = 0; j < exp50.size(); ++j) {
- EXPECT_EQUAL(exp50[j], rs1->getArray()[j]._docId);
- EXPECT_EQUAL(exp50[j], rs2->getArray()[j]._docId);
+ EXPECT_EQUAL(exp50[j], rs1->getArray()[j].getDocId());
+ EXPECT_EQUAL(exp50[j], rs2->getArray()[j].getDocId());
}
}
{
ResultSetPtr rs = performSearch(fa, "60");
EXPECT_EQUAL(exp60.size(), rs->getNumHits());
for (size_t j = 0; j < exp60.size(); ++j) {
- EXPECT_EQUAL(exp60[j], rs->getArray()[j]._docId);
+ EXPECT_EQUAL(exp60[j], rs->getArray()[j].getDocId());
}
}
}