diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-09-02 11:47:13 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-09-02 11:49:15 +0000 |
commit | 2e69eeead11986908979a740cd5019362322c6af (patch) | |
tree | 3df57b5184e0aa67eca1d3392a1fad67647cba68 /searchlib/src/tests/engine | |
parent | 10a3ed1cd43054f9ffd644a29a95ed86c769ca35 (diff) |
Add conversion test and convert too...
Diffstat (limited to 'searchlib/src/tests/engine')
-rw-r--r-- | searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp | 15 |
1 files changed, 14 insertions, 1 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 14ef77a4161..3091d167399 100644 --- a/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp +++ b/searchlib/src/tests/engine/proto_converter/proto_converter_test.cpp @@ -494,6 +494,19 @@ TEST_F(DocsumRequestTest, require_that_geo_location_is_converted) { EXPECT_EQ(request.location, "x,y"); } +TEST_F(DocsumRequestTest, require_that_field_list_is_empty_by_default) { + convert(); + EXPECT_TRUE(request.getFields().empty()); +} +TEST_F(DocsumRequestTest, require_that_field_list_is_converted) { + proto.add_fields("f1"); + proto.add_fields("f2"); + convert(); + EXPECT_EQ(2u, request.getFields().size()); + EXPECT_EQ("f1", request.getFields()[0]); + EXPECT_EQ("f2", request.getFields()[1]); +} + TEST_F(DocsumRequestTest, require_that_query_tree_blob_is_converted) { proto.set_query_tree_blob("query-tree-blob"); convert(); @@ -518,7 +531,7 @@ struct DocsumReplyTest : ProtoConverterTest { DocsumReply reply; Converter::ProtoDocsumReply proto; void convert() { Converter::docsum_reply_to_proto(reply, proto); } - DocsumReplyTest(std::unique_ptr<Slime> slime_in) + explicit DocsumReplyTest(std::unique_ptr<Slime> slime_in) : slime(*slime_in), reply(std::move(slime_in)) {} DocsumReplyTest() : DocsumReplyTest(std::make_unique<Slime>()) {} |