summaryrefslogtreecommitdiffstats
path: root/streamingvisitors
diff options
context:
space:
mode:
authorGeir Storli <geirst@yahooinc.com>2023-03-13 13:55:29 +0000
committerGeir Storli <geirst@yahooinc.com>2023-03-13 13:59:08 +0000
commit0c657642eff7d85c000464e9edba16ded4845e7d (patch)
tree3315f420e49dd85739089780388007ce1649672a /streamingvisitors
parenta40183094fd45dbf320dd57f8d56a5a6fac7f247 (diff)
Reduce creation of Document instances without DocumentTypeRepo.
Diffstat (limited to 'streamingvisitors')
-rw-r--r--streamingvisitors/src/tests/document/document.cpp2
-rw-r--r--streamingvisitors/src/tests/hitcollector/hitcollector_test.cpp4
-rw-r--r--streamingvisitors/src/tests/matching_elements_filler/matching_elements_filler_test.cpp2
3 files changed, 4 insertions, 4 deletions
diff --git a/streamingvisitors/src/tests/document/document.cpp b/streamingvisitors/src/tests/document/document.cpp
index 1e97d232a64..acebd9ed4a4 100644
--- a/streamingvisitors/src/tests/document/document.cpp
+++ b/streamingvisitors/src/tests/document/document.cpp
@@ -29,7 +29,7 @@ DocumentTest::testStorageDocument()
dt.addField(fa);
dt.addField(fb);
- document::Document::UP doc(new document::Document(dt, DocumentId()));
+ auto doc = document::Document::make_without_repo(dt, DocumentId());
doc->setValue(fa, StringFieldValue("foo"));
doc->setValue(fb, StringFieldValue("bar"));
diff --git a/streamingvisitors/src/tests/hitcollector/hitcollector_test.cpp b/streamingvisitors/src/tests/hitcollector/hitcollector_test.cpp
index 9715b44b27b..6950c90f097 100644
--- a/streamingvisitors/src/tests/hitcollector/hitcollector_test.cpp
+++ b/streamingvisitors/src/tests/hitcollector/hitcollector_test.cpp
@@ -77,8 +77,8 @@ HitCollectorTest::assertHit(SearchResult::RankType expRank, uint32_t expDocId, u
void
HitCollectorTest::addHit(HitCollector &hc, uint32_t docId, double score, const char *sortData, size_t sortDataSize)
{
- document::Document::UP doc(new document::Document(_docType, DocumentId("id:ns:testdoc::")));
- StorageDocument::UP sdoc(new StorageDocument(std::move(doc), SharedFieldPathMap(), 0));
+ auto doc = document::Document::make_without_repo(_docType, DocumentId("id:ns:testdoc::"));
+ auto sdoc = std::make_unique<StorageDocument>(std::move(doc), SharedFieldPathMap(), 0);
ASSERT_TRUE(sdoc->valid());
MatchData md(MatchData::params());
hc.addHit(sdoc.get(), docId, md, score, sortData, sortDataSize);
diff --git a/streamingvisitors/src/tests/matching_elements_filler/matching_elements_filler_test.cpp b/streamingvisitors/src/tests/matching_elements_filler/matching_elements_filler_test.cpp
index 59e84b7d08c..458031b5f01 100644
--- a/streamingvisitors/src/tests/matching_elements_filler/matching_elements_filler_test.cpp
+++ b/streamingvisitors/src/tests/matching_elements_filler/matching_elements_filler_test.cpp
@@ -211,7 +211,7 @@ MyDocType::make_field_path(vespalib::string path) const
std::unique_ptr<document::Document>
MyDocType::make_test_doc() const
{
- auto doc = std::make_unique<document::Document>(_document_type, DocumentId("id::test::1"));
+ auto doc = document::Document::make_without_repo(_document_type, DocumentId("id::test::1"));
doc->setValue("elem_array", *make_elem_array({{"foo", 10},{"bar", 20},{"baz", 30},{"foo", 40}, {"zap", 20}, {"zap", 20}}));
// the elements in maps are ordered on the key
doc->setValue("elem_map", *make_elem_map({{"@foo", {"foo", 10}}, {"@bar", {"bar", 20}},{"@baz", {"baz", 30}},{"@foo@", {"foo", 40}},{"@zap", {"zap", 20}}, {"@zap@", {"zap", 20}}}));