diff options
Diffstat (limited to 'searchlib/src/tests/attribute/enumstore/enumstore_test.cpp')
-rw-r--r-- | searchlib/src/tests/attribute/enumstore/enumstore_test.cpp | 26 |
1 files changed, 9 insertions, 17 deletions
diff --git a/searchlib/src/tests/attribute/enumstore/enumstore_test.cpp b/searchlib/src/tests/attribute/enumstore/enumstore_test.cpp index 5476f0f8e66..3a70073226c 100644 --- a/searchlib/src/tests/attribute/enumstore/enumstore_test.cpp +++ b/searchlib/src/tests/attribute/enumstore/enumstore_test.cpp @@ -54,7 +54,7 @@ private: void testCompaction(); template <typename EnumStoreType> - void testCompaction(bool hasPostings, bool disableReEnumerate); + void testCompaction(bool hasPostings); void testReset(); template <typename EnumStoreType> @@ -396,15 +396,13 @@ EnumStoreTest::testUniques void EnumStoreTest::testCompaction() { - testCompaction<StringEnumStore>(false, false); - testCompaction<StringEnumStore>(true, false); - testCompaction<StringEnumStore>(false, true); - testCompaction<StringEnumStore>(true, true); + testCompaction<StringEnumStore>(false); + testCompaction<StringEnumStore>(true); } template <typename EnumStoreType> void -EnumStoreTest::testCompaction(bool hasPostings, bool disableReEnumerate) +EnumStoreTest::testCompaction(bool hasPostings) { // entrySize = 15 before alignment uint32_t entrySize = EnumStoreType::alignEntrySize(15); @@ -462,30 +460,24 @@ EnumStoreTest::testCompaction(bool hasPostings, bool disableReEnumerate) EXPECT_EQUAL(entrySize + RESERVED_BYTES, ses.getBuffer(0).getDeadElems()); // perform compaction - if (disableReEnumerate) { - ses.disableReEnumerate(); - } EnumStoreBase::EnumIndexMap old2New; EXPECT_TRUE(ses.performCompaction(3 * entrySize, old2New)); - if (disableReEnumerate) { - ses.enableReEnumerate(); - } EXPECT_TRUE(ses.getRemaining() >= 3 * entrySize); EXPECT_TRUE(ses.getBuffer(1).remaining() >= 3 * entrySize); EXPECT_TRUE(ses.getBuffer(1).size() == entrySize * 4); EXPECT_TRUE(ses.getBuffer(1).getDeadElems() == 0); - EXPECT_EQUAL((disableReEnumerate ? 4u : 3u), ses.getLastEnum()); + EXPECT_EQUAL(3u, ses.getLastEnum()); // add new unique strings ses.addEnum("enum05", idx); - EXPECT_EQUAL((disableReEnumerate ? 5u : 4u), ses.getEnum(idx)); + EXPECT_EQUAL(4u, ses.getEnum(idx)); ses.addEnum("enum06", idx); - EXPECT_EQUAL((disableReEnumerate ? 6u : 5u), ses.getEnum(idx)); + EXPECT_EQUAL(5u, ses.getEnum(idx)); ses.addEnum("enum00", idx); - EXPECT_EQUAL((disableReEnumerate ? 7u : 6u), ses.getEnum(idx)); + EXPECT_EQUAL(6u, ses.getEnum(idx)); - EXPECT_EQUAL((disableReEnumerate ? 7u : 6u), ses.getLastEnum()); + EXPECT_EQUAL(6u, ses.getLastEnum()); // compare old and new indices for (uint32_t i = 0; i < indices.size(); ++i) { |