aboutsummaryrefslogtreecommitdiffstats
path: root/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp')
-rw-r--r--searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp19
1 files changed, 9 insertions, 10 deletions
diff --git a/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp b/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
index 973d2bcc15c..7eb7d1d0d80 100644
--- a/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
+++ b/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
@@ -478,14 +478,13 @@ TEST_F(DocsumRequestTest, require_that_global_ids_are_converted) {
//-----------------------------------------------------------------------------
struct DocsumReplyTest : ::testing::Test {
- DocsumReply reply;
+ DocsumReply reply{std::make_unique<vespalib::Slime>()};
Converter::ProtoDocsumReply proto;
void convert() { Converter::docsum_reply_to_proto(reply, proto); }
};
TEST_F(DocsumReplyTest, require_that_slime_summaries_are_converted) {
- reply._root = std::make_unique<Slime>();
- auto &list = reply._root->setArray();
+ auto &list = reply.slime().setArray();
auto &doc0 = list.addObject();
doc0.setLong("my_field", 42);
convert();
@@ -496,18 +495,18 @@ TEST_F(DocsumReplyTest, require_that_slime_summaries_are_converted) {
}
TEST_F(DocsumReplyTest, require_that_missing_root_slime_gives_empty_payload) {
- reply._root.reset();
+ reply.releaseSlime();
convert();
EXPECT_EQ(proto.slime_summaries().size(), 0);
}
TEST_F(DocsumReplyTest, require_that_issues_are_converted_to_errors) {
- reply.my_issues = std::make_unique<UniqueIssues>();
- reply.my_issues->handle(vespalib::Issue("a"));
- reply.my_issues->handle(vespalib::Issue("b"));
- reply.my_issues->handle(vespalib::Issue("c"));
- reply.my_issues->handle(vespalib::Issue("a"));
- reply.my_issues->handle(vespalib::Issue("b"));
+ reply.setIssues(std::make_unique<UniqueIssues>());
+ reply.issues().handle(vespalib::Issue("a"));
+ reply.issues().handle(vespalib::Issue("b"));
+ reply.issues().handle(vespalib::Issue("c"));
+ reply.issues().handle(vespalib::Issue("a"));
+ reply.issues().handle(vespalib::Issue("b"));
convert();
ASSERT_EQ(proto.errors_size(), 3);
EXPECT_EQ(proto.errors(0).message(), "a");