From 4d59e63579de007b3470ec28f45185b8d5996f9e Mon Sep 17 00:00:00 2001 From: Tor Egge Date: Wed, 25 Jan 2023 11:18:52 +0100 Subject: Propagate schema to summary manager. --- .../documentdb/configurer/configurer_test.cpp | 29 +++++++++++++++++++++- 1 file changed, 28 insertions(+), 1 deletion(-) (limited to 'searchcore/src/tests/proton/documentdb/configurer/configurer_test.cpp') diff --git a/searchcore/src/tests/proton/documentdb/configurer/configurer_test.cpp b/searchcore/src/tests/proton/documentdb/configurer/configurer_test.cpp index 0bcbf1269a1..ee6edf228d9 100644 --- a/searchcore/src/tests/proton/documentdb/configurer/configurer_test.cpp +++ b/searchcore/src/tests/proton/documentdb/configurer/configurer_test.cpp @@ -211,7 +211,7 @@ Fixture::initViewSet(ViewSet &views) auto matchView = std::make_shared(matchers, indexSearchable, attrMgr, _sessionMgr, metaStore, views._docIdLimit); views.searchView.set(SearchView::create (summaryMgr->createSummarySetup(SummaryConfig(), - JuniperrcConfig(), views.repo, attrMgr), + JuniperrcConfig(), views.repo, attrMgr, *schema), std::move(matchView))); views.feedView.set( make_shared(StoreOnlyFeedView::Context(summaryAdapter, @@ -684,6 +684,33 @@ TEST("require that subdbs should change if relevant config changed") TEST_DO(assertSubDbsShouldChange(CCR().set_alloc_config_changed(true))); } +void +assertSummaryManagerShouldNotChange(DocumentDBConfig::ComparisonResult result) +{ + ReconfigParams params(result); + EXPECT_FALSE(params.configHasChanged()); + EXPECT_FALSE(params.shouldSummaryManagerChange()); +} + +void +assertSummaryManagerShouldChange(DocumentDBConfig::ComparisonResult result) +{ + ReconfigParams params(result); + EXPECT_TRUE(params.configHasChanged()); + EXPECT_TRUE(params.shouldSummaryManagerChange()); +} + +TEST("require that summary manager should change if relevant config changed") +{ + TEST_DO(assertSummaryManagerShouldNotChange(CCR())); + TEST_DO(assertSummaryManagerShouldChange(CCR().setSummaryChanged(true))); + TEST_DO(assertSummaryManagerShouldChange(CCR().setJuniperrcChanged(true))); + TEST_DO(assertSummaryManagerShouldChange(CCR().setDocumenttypesChanged(true))); + TEST_DO(assertSummaryManagerShouldChange(CCR().setDocumentTypeRepoChanged(true))); + TEST_DO(assertSummaryManagerShouldChange(CCR().setStoreChanged(true))); + TEST_DO(assertSummaryManagerShouldChange(CCR().setSchemaChanged(true))); +} + TEST_MAIN() { TEST_RUN_ALL(); -- cgit v1.2.3