diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2019-03-11 17:46:15 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-11 17:46:15 +0100 |
commit | 5d5aba2046564c5dfb3ac992178304f873757c21 (patch) | |
tree | 8952b8f7ff2186c5aca0b281e3e3b399e3dc65d7 /searchcore | |
parent | 41731a3f9f206e6ab24c3e8fe0e448ff84648936 (diff) | |
parent | 2271b5f3f984ed91b358c0313155af68df0cd4cc (diff) |
Merge pull request #8730 from vespa-engine/toregge/reduce-use-of-docbuilder-1
Stop using DocBuilder for attribute populator test
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/tests/proton/attribute/attribute_populator/attribute_populator_test.cpp | 31 |
1 files changed, 17 insertions, 14 deletions
diff --git a/searchcore/src/tests/proton/attribute/attribute_populator/attribute_populator_test.cpp b/searchcore/src/tests/proton/attribute/attribute_populator/attribute_populator_test.cpp index ed71477d853..01dc2306ef6 100644 --- a/searchcore/src/tests/proton/attribute/attribute_populator/attribute_populator_test.cpp +++ b/searchcore/src/tests/proton/attribute/attribute_populator/attribute_populator_test.cpp @@ -3,17 +3,19 @@ LOG_SETUP("attribute_populator_test"); #include <vespa/vespalib/testkit/testapp.h> -#include <vespa/searchcommon/common/schema.h> +#include <vespa/document/repo/configbuilder.h> +#include <vespa/document/fieldvalue/intfieldvalue.h> #include <vespa/searchcore/proton/attribute/attribute_populator.h> #include <vespa/searchcore/proton/attribute/attributemanager.h> #include <vespa/searchcore/proton/common/hw_info.h> #include <vespa/searchcore/proton/test/test.h> #include <vespa/searchlib/common/foregroundtaskexecutor.h> -#include <vespa/searchlib/index/docbuilder.h> #include <vespa/searchlib/index/dummyfileheadercontext.h> #include <vespa/searchlib/test/directory_handler.h> #include <vespa/vespalib/util/stringfmt.h> +using document::config_builder::DocumenttypesConfigBuilderHelper; +using document::config_builder::Struct; using namespace document; using namespace proton; using namespace search; @@ -27,29 +29,30 @@ using AVConfig = search::attribute::Config; const vespalib::string TEST_DIR = "testdir"; const uint64_t CREATE_SERIAL_NUM = 8u; -Schema -createSchema() +std::unique_ptr<const DocumentTypeRepo> +makeDocTypeRepo() { - Schema schema; - schema.addAttributeField(Schema::AttributeField("a1", Schema::DataType::INT32)); - return schema; + DocumenttypesConfigBuilderHelper builder; + builder.document(-645763131, "searchdocument", + Struct("searchdocument.header"), + Struct("searchdocument.body"). + addField("a1", DataType::T_INT)); + return std::unique_ptr<const DocumentTypeRepo>(new DocumentTypeRepo(builder.config())); } struct DocContext { - Schema _schema; - DocBuilder _builder; + std::unique_ptr<const DocumentTypeRepo> _repo; DocContext() - : _schema(createSchema()), - _builder(_schema) + : _repo(makeDocTypeRepo()) { } std::shared_ptr<Document> create(uint32_t id, int64_t fieldValue) { vespalib::string docId = vespalib::make_string("id:searchdocument:searchdocument::%u", id); - return _builder.startDocument(docId). - startAttributeField("a1").addInt(fieldValue).endField(). - endDocument(); + auto doc = std::make_shared<Document>(*_repo->getDocumentType("searchdocument"), DocumentId(docId)); + doc->setValue("a1", IntFieldValue(fieldValue)); + return doc; } }; |