summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahooinc.com>2021-10-13 13:10:58 +0000
committerArne H Juul <arnej@yahooinc.com>2021-10-13 13:20:19 +0000
commit3534365cd5acc7c9e427bee0876587c43d5c2ac5 (patch)
treea2e0c320705f1aa78c00389f06f5ca425f356582 /searchcore
parent556f30a0d3afef8ff19fd1bcefe9b8fd11b52002 (diff)
remove and simplify even more
* we can specify data with "x" prefix * remove no-longer-used code
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/tests/proton/docsummary/docsummary.cpp1
-rw-r--r--searchcore/src/tests/proton/summaryengine/summaryengine.cpp53
2 files changed, 11 insertions, 43 deletions
diff --git a/searchcore/src/tests/proton/docsummary/docsummary.cpp b/searchcore/src/tests/proton/docsummary/docsummary.cpp
index 349076542c3..916b88c19c4 100644
--- a/searchcore/src/tests/proton/docsummary/docsummary.cpp
+++ b/searchcore/src/tests/proton/docsummary/docsummary.cpp
@@ -564,7 +564,6 @@ TEST("requireThatSummariesTimeout")
req.resultClassName = "class2";
req.hits.push_back(DocsumRequest::Hit(gid1));
DocsumReply::UP rep = dc._ddb->getDocsums(req);
- //fprintf(stderr, "SLIME:\n%s\n", rep->root().toString().c_str());
const auto & field = rep->root()["errors"];
EXPECT_TRUE(field.valid());
EXPECT_EQUAL(field[0]["type"].asString(), "timeout");
diff --git a/searchcore/src/tests/proton/summaryengine/summaryengine.cpp b/searchcore/src/tests/proton/summaryengine/summaryengine.cpp
index f15d45d332c..6660b7bfd55 100644
--- a/searchcore/src/tests/proton/summaryengine/summaryengine.cpp
+++ b/searchcore/src/tests/proton/summaryengine/summaryengine.cpp
@@ -230,16 +230,7 @@ verify(vespalib::stringref exp, const Slime &slime) {
vespalib::Slime expSlime;
size_t used = vespalib::slime::JsonFormat::decode(expMemory, expSlime);
EXPECT_TRUE(used > 0);
- vespalib::SimpleBuffer output;
- vespalib::slime::JsonFormat::encode(slime, output, true);
- Slime reSlimed;
- used = vespalib::slime::JsonFormat::decode(output.get(), reSlimed);
- EXPECT_TRUE(used > 0);
- if (!EXPECT_EQUAL(expSlime, reSlimed)) {
- fprintf(stderr, "exp -> %.*s\n", (int)exp.size(), exp.data());
- Memory enc = output.get();
- fprintf(stderr, "enc -> %.*s\n", (int)enc.size, enc.data);
- }
+ EXPECT_EQUAL(expSlime, slime);
}
Slime
@@ -280,8 +271,8 @@ TEST("requireThatSlimeRequestIsConvertedCorrectly") {
TEST_DO(verify("{"
" class: 'your-summary',"
" gids: ["
- " '0x6162636465666768696A6B6C',"
- " '0x62636465666768696A6B6C6D'"
+ " x6162636465666768696A6B6C,"
+ " x62636465666768696A6B6C6D"
" ]"
"}", slimeRequest));
DocsumRequest::UP r = DocsumBySlime::slimeToRequest(slimeRequest.get());
@@ -298,11 +289,11 @@ TEST("require that presence of sessionid affect both request.sessionid and enabl
vespalib::Slime slimeRequest = createSlimeRequest("1.some.key.7", "my-rank-profile");
TEST_DO(verify("{"
" class: 'your-summary',"
- " sessionid: '0x312E736F6D652E6B65792E37',"
+ " sessionid: x312E736F6D652E6B65792E37,"
" ranking: 'my-rank-profile',"
" gids: ["
- " '0x6162636465666768696A6B6C',"
- " '0x62636465666768696A6B6C6D'"
+ " x6162636465666768696A6B6C,"
+ " x62636465666768696A6B6C6D"
" ]"
"}", slimeRequest));
DocsumRequest::UP r = DocsumBySlime::slimeToRequest(slimeRequest.get());
@@ -320,12 +311,12 @@ TEST("require that 'doctype' affects DocTypeName in a good way...") {
vespalib::Slime slimeRequest = createSlimeRequest("1.some.key.7", "my-rank-profile", "my-document-type");
TEST_DO(verify("{"
" class: 'your-summary',"
- " sessionid: '0x312E736F6D652E6B65792E37',"
+ " sessionid: x312E736F6D652E6B65792E37,"
" ranking: 'my-rank-profile',"
" doctype: 'my-document-type',"
" gids: ["
- " '0x6162636465666768696A6B6C',"
- " '0x62636465666768696A6B6C6D'"
+ " x6162636465666768696A6B6C,"
+ " x62636465666768696A6B6C6D"
" ]"
"}", slimeRequest));
DocsumRequest::UP r = DocsumBySlime::slimeToRequest(slimeRequest.get());
@@ -343,28 +334,7 @@ TEST("require that 'doctype' affects DocTypeName in a good way...") {
EXPECT_EQUAL(GlobalId(GID2), r->hits[1].gid);
}
-void
-createSummary(search::RawBuf &buf) {
- vespalib::Slime summary;
- summary.setObject().setLong("long", 982);
- uint32_t magic = search::docsummary::SLIME_MAGIC_ID;
- buf.append(&magic, sizeof(magic));
- search::SlimeOutputRawBufAdapter adapter(buf);
- BinaryFormat::encode(summary, adapter);
-}
-
-class BaseServer {
-protected:
- BaseServer() : buf(100)
- {
- createSummary(buf);
- }
-
-protected:
- search::RawBuf buf;
-};
-
-class Server : public BaseServer {
+class Server {
public:
Server();
~Server();
@@ -378,8 +348,7 @@ public:
};
Server::Server()
- : BaseServer(),
- engine(2),
+ : engine(2),
handler(std::make_shared<MySearchHandler>("slime", "some other value")),
docsumBySlime(engine),
docsumByRPC(docsumBySlime)