diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2020-07-09 11:26:07 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-09 11:26:07 +0200 |
commit | cbca38828c11f8d0e369b0e6a263ca5701796f59 (patch) | |
tree | 3ea7c69f90c2ff49bdc4afcb79a4e125583cd07b /messagebus/src | |
parent | 637de9a2313414da6930f703e2a63eae4637d0e6 (diff) | |
parent | 1d12f4b4b92771dfe7e245b88ce89adc911c509e (diff) |
Merge branch 'master' into balder/skip-communicationmanager-only
Diffstat (limited to 'messagebus/src')
-rw-r--r-- | messagebus/src/tests/bucketsequence/bucketsequence.cpp | 15 | ||||
-rw-r--r-- | messagebus/src/vespa/messagebus/message.h | 2 | ||||
-rw-r--r-- | messagebus/src/vespa/messagebus/network/rpcsend.cpp | 2 |
3 files changed, 7 insertions, 12 deletions
diff --git a/messagebus/src/tests/bucketsequence/bucketsequence.cpp b/messagebus/src/tests/bucketsequence/bucketsequence.cpp index 57d1f99e5c3..671f50efc03 100644 --- a/messagebus/src/tests/bucketsequence/bucketsequence.cpp +++ b/messagebus/src/tests/bucketsequence/bucketsequence.cpp @@ -11,19 +11,13 @@ using namespace mbus; -TEST_SETUP(Test); - class MyMessage : public SimpleMessage { public: MyMessage() : SimpleMessage("foo") { } - bool hasBucketSequence() override { return true; } + bool hasBucketSequence() const override { return true; } }; -int -Test::Main() -{ - TEST_INIT("bucketsequence_test"); - +TEST("bucketsequence_test") { Slobrok slobrok; TestServer server(MessageBusParams() .addProtocol(std::make_shared<SimpleProtocol>()) @@ -37,9 +31,10 @@ Test::Main() msg->setRoute(Route::parse("foo")); ASSERT_TRUE(session->send(std::move(msg)).isAccepted()); Reply::UP reply = receptor.getReply(); - ASSERT_TRUE(reply.get() != nullptr); + ASSERT_TRUE(reply); EXPECT_EQUAL(1u, reply->getNumErrors()); EXPECT_EQUAL((uint32_t)ErrorCode::SEQUENCE_ERROR, reply->getError(0).getCode()); - TEST_DONE(); } + +TEST_MAIN() { TEST_RUN_ALL(); } diff --git a/messagebus/src/vespa/messagebus/message.h b/messagebus/src/vespa/messagebus/message.h index 15e7384707c..22fb1e282b6 100644 --- a/messagebus/src/vespa/messagebus/message.h +++ b/messagebus/src/vespa/messagebus/message.h @@ -135,7 +135,7 @@ public: * * @return True to enable bucket sequencing. */ - virtual bool hasBucketSequence() { return false; } + virtual bool hasBucketSequence() const { return false; } /** * Obtain the approximate size of this message object in bytes. This enables diff --git a/messagebus/src/vespa/messagebus/network/rpcsend.cpp b/messagebus/src/vespa/messagebus/network/rpcsend.cpp index 2422638dc05..dca7f0c997f 100644 --- a/messagebus/src/vespa/messagebus/network/rpcsend.cpp +++ b/messagebus/src/vespa/messagebus/network/rpcsend.cpp @@ -114,7 +114,7 @@ RPCSend::send(RoutingNode &recipient, const vespalib::Version &version, const PayLoadFiller & payload, duration timeRemaining) { auto ctx = std::make_unique<SendContext>(recipient, timeRemaining); - RPCServiceAddress &address = static_cast<RPCServiceAddress&>(recipient.getServiceAddress()); + auto &address = static_cast<RPCServiceAddress&>(recipient.getServiceAddress()); const Message &msg = recipient.getMessage(); Route route = recipient.getRoute(); Hop hop = route.removeHop(0); |