summaryrefslogtreecommitdiffstats
path: root/searchlib/src/tests/engine
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahooinc.com>2021-10-14 14:12:44 +0000
committerArne H Juul <arnej@yahooinc.com>2021-10-14 14:12:44 +0000
commite28a07fab5f83fd06f84bee95f20f457738b7ddb (patch)
tree1444ff8a098f71af59770d23886c876acdd6566d /searchlib/src/tests/engine
parentdf689ff589e185f51dec5a6cfa70e4009ebc176a (diff)
more useful DocsumReply
Diffstat (limited to 'searchlib/src/tests/engine')
-rw-r--r--searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp1
-rw-r--r--searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp6
2 files changed, 3 insertions, 4 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 7eb7d1d0d80..23b95f489c6 100644
--- a/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
+++ b/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp
@@ -501,7 +501,6 @@ TEST_F(DocsumReplyTest, require_that_missing_root_slime_gives_empty_payload) {
}
TEST_F(DocsumReplyTest, require_that_issues_are_converted_to_errors) {
- reply.setIssues(std::make_unique<UniqueIssues>());
reply.issues().handle(vespalib::Issue("a"));
reply.issues().handle(vespalib::Issue("b"));
reply.issues().handle(vespalib::Issue("c"));
diff --git a/searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp b/searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp
index 7aa72b50ba4..33a40b76a51 100644
--- a/searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp
+++ b/searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp
@@ -49,11 +49,11 @@ struct MyDocsumServer : DocsumServer {
DocsumReply::UP getDocsums(DocsumRequest::Source src, DocsumClient &client) override {
auto req = src.release();
assert(req);
- auto reply = std::make_unique<DocsumReply>(std::make_unique<Slime>());
- auto &list = reply->slime().setArray();
+ auto slime = std::make_unique<Slime>();
+ auto &list = slime->setArray();
list.addObject().setBool("use_root_slime", true);
list.addObject().setString("ranking", req->ranking);
- reply->setRequest(std::move(req));
+ auto reply = std::make_unique<DocsumReply>(std::move(slime), std::move(req));
std::this_thread::sleep_for(std::chrono::milliseconds(5));
client.getDocsumsDone(std::move(reply)); // simplified async response
return std::unique_ptr<DocsumReply>();