diff options
author | Arne H Juul <arnej@yahooinc.com> | 2021-10-14 14:12:44 +0000 |
---|---|---|
committer | Arne H Juul <arnej@yahooinc.com> | 2021-10-14 14:12:44 +0000 |
commit | e28a07fab5f83fd06f84bee95f20f457738b7ddb (patch) | |
tree | 1444ff8a098f71af59770d23886c876acdd6566d /searchlib/src/tests/engine | |
parent | df689ff589e185f51dec5a6cfa70e4009ebc176a (diff) |
more useful DocsumReply
Diffstat (limited to 'searchlib/src/tests/engine')
-rw-r--r-- | searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp | 1 | ||||
-rw-r--r-- | searchlib/src/tests/engine/proto_rpc_adapter/proto_rpc_adapter_test.cpp | 6 |
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>(); |