Alexei Frolov | 26e3ae6 | 2020-05-04 17:06:17 -0700 | [diff] [blame] | 1 | // Copyright 2020 The Pigweed Authors |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); you may not |
| 4 | // use this file except in compliance with the License. You may obtain a copy of |
| 5 | // the License at |
| 6 | // |
| 7 | // https://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 11 | // WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 12 | // License for the specific language governing permissions and limitations under |
| 13 | // the License. |
| 14 | |
| 15 | #include "pw_rpc/server.h" |
| 16 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 17 | #include <array> |
| 18 | #include <cstdint> |
| 19 | |
Alexei Frolov | 26e3ae6 | 2020-05-04 17:06:17 -0700 | [diff] [blame] | 20 | #include "gtest/gtest.h" |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 21 | #include "pw_assert/assert.h" |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 22 | #include "pw_rpc/internal/packet.h" |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 23 | #include "pw_rpc/internal/service.h" |
Wyatt Hepler | cb9d957 | 2020-06-01 11:25:58 -0700 | [diff] [blame] | 24 | #include "pw_rpc_private/test_utils.h" |
Alexei Frolov | 26e3ae6 | 2020-05-04 17:06:17 -0700 | [diff] [blame] | 25 | |
| 26 | namespace pw::rpc { |
| 27 | namespace { |
| 28 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 29 | using std::byte; |
| 30 | |
| 31 | using internal::Method; |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 32 | using internal::Packet; |
| 33 | using internal::PacketType; |
Alexei Frolov | 5d6d392 | 2020-05-08 13:57:02 -0700 | [diff] [blame] | 34 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 35 | class TestService : public internal::Service { |
| 36 | public: |
| 37 | TestService(uint32_t service_id) |
| 38 | : internal::Service(service_id, methods_), |
| 39 | methods_{ |
| 40 | Method(100), |
| 41 | Method(200), |
| 42 | } {} |
| 43 | |
| 44 | Method& method(uint32_t id) { |
| 45 | for (Method& method : methods_) { |
| 46 | if (method.id() == id) { |
| 47 | return method; |
| 48 | } |
| 49 | } |
| 50 | |
| 51 | PW_CRASH("Invalid method ID %u", static_cast<unsigned>(id)); |
| 52 | } |
| 53 | |
| 54 | private: |
| 55 | std::array<Method, 2> methods_; |
| 56 | }; |
| 57 | |
| 58 | class BasicServer : public ::testing::Test { |
| 59 | protected: |
| 60 | static constexpr byte kDefaultPayload[] = { |
| 61 | byte(0x82), byte(0x02), byte(0xff), byte(0xff)}; |
| 62 | |
| 63 | BasicServer() |
Wyatt Hepler | 51a2eb7 | 2020-06-04 14:47:00 -0700 | [diff] [blame] | 64 | : channels_{ |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 65 | Channel::Create<1>(&output_), |
| 66 | Channel::Create<2>(&output_), |
| 67 | Channel(), // available for assignment |
| 68 | }, |
| 69 | server_(channels_), |
| 70 | service_(42) { |
| 71 | server_.RegisterService(service_); |
| 72 | } |
| 73 | |
Wyatt Hepler | e2cbadf | 2020-06-22 11:21:45 -0700 | [diff] [blame] | 74 | std::span<const byte> EncodeRequest( |
| 75 | PacketType type, |
| 76 | uint32_t channel_id, |
| 77 | uint32_t service_id, |
| 78 | uint32_t method_id, |
| 79 | std::span<const byte> payload = kDefaultPayload) { |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 80 | auto sws = Packet(type, channel_id, service_id, method_id, payload) |
| 81 | .Encode(request_buffer_); |
| 82 | EXPECT_EQ(Status::OK, sws.status()); |
Wyatt Hepler | e2cbadf | 2020-06-22 11:21:45 -0700 | [diff] [blame] | 83 | return std::span(request_buffer_, sws.size()); |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 84 | } |
| 85 | |
Wyatt Hepler | af83568 | 2020-06-17 11:42:53 -0700 | [diff] [blame] | 86 | TestOutput<128> output_; |
| 87 | std::array<Channel, 3> channels_; |
| 88 | Server server_; |
| 89 | TestService service_; |
| 90 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 91 | private: |
| 92 | byte request_buffer_[64]; |
| 93 | }; |
| 94 | |
| 95 | TEST_F(BasicServer, ProcessPacket_ValidMethod_InvokesMethod) { |
| 96 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 42, 100), output_); |
| 97 | |
| 98 | const Method& method = service_.method(100); |
| 99 | EXPECT_EQ(1u, method.last_channel_id()); |
| 100 | EXPECT_EQ(sizeof(kDefaultPayload), method.last_request().size()); |
| 101 | EXPECT_EQ(std::memcmp(kDefaultPayload, |
| 102 | method.last_request().data(), |
| 103 | method.last_request().size()), |
| 104 | 0); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 105 | } |
Alexei Frolov | 5d6d392 | 2020-05-08 13:57:02 -0700 | [diff] [blame] | 106 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 107 | TEST_F(BasicServer, ProcessPacket_ValidMethod_SendsOkResponse) { |
| 108 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 42, 100), output_); |
Alexei Frolov | 5d6d392 | 2020-05-08 13:57:02 -0700 | [diff] [blame] | 109 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 110 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
Wyatt Hepler | af83568 | 2020-06-17 11:42:53 -0700 | [diff] [blame] | 111 | EXPECT_EQ(packet.type(), PacketType::RPC); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 112 | EXPECT_EQ(packet.channel_id(), 1u); |
| 113 | EXPECT_EQ(packet.service_id(), 42u); |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 114 | EXPECT_EQ(packet.method_id(), 100u); |
| 115 | EXPECT_TRUE(packet.payload().empty()); |
| 116 | EXPECT_EQ(packet.status(), Status::OK); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 117 | } |
| 118 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 119 | TEST_F(BasicServer, ProcessPacket_ValidMethod_SendsErrorResponse) { |
| 120 | constexpr byte resp[] = {byte{0xf0}, byte{0x0d}}; |
| 121 | service_.method(200).set_response(resp); |
| 122 | service_.method(200).set_status(Status::FAILED_PRECONDITION); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 123 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 124 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 2, 42, 200), output_); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 125 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 126 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
| 127 | EXPECT_EQ(packet.channel_id(), 2u); |
| 128 | EXPECT_EQ(packet.service_id(), 42u); |
| 129 | EXPECT_EQ(packet.method_id(), 200u); |
| 130 | EXPECT_EQ(packet.status(), Status::FAILED_PRECONDITION); |
| 131 | ASSERT_EQ(sizeof(resp), packet.payload().size()); |
| 132 | EXPECT_EQ(std::memcmp(packet.payload().data(), resp, sizeof(resp)), 0); |
| 133 | } |
| 134 | |
Wyatt Hepler | af83568 | 2020-06-17 11:42:53 -0700 | [diff] [blame] | 135 | TEST_F(BasicServer, ProcessPacket_IncompletePacket_NothingIsInvoked) { |
| 136 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 0, 42, 101), output_); |
| 137 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 0, 101), output_); |
| 138 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 42, 0), output_); |
| 139 | |
| 140 | EXPECT_EQ(0u, service_.method(100).last_channel_id()); |
| 141 | EXPECT_EQ(0u, service_.method(200).last_channel_id()); |
| 142 | } |
| 143 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 144 | TEST_F(BasicServer, ProcessPacket_InvalidMethod_NothingIsInvoked) { |
| 145 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 42, 101), output_); |
| 146 | |
| 147 | EXPECT_EQ(0u, service_.method(100).last_channel_id()); |
| 148 | EXPECT_EQ(0u, service_.method(200).last_channel_id()); |
| 149 | } |
| 150 | |
| 151 | TEST_F(BasicServer, ProcessPacket_InvalidMethod_SendsNotFound) { |
| 152 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 42, 27), output_); |
| 153 | |
| 154 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
Wyatt Hepler | af83568 | 2020-06-17 11:42:53 -0700 | [diff] [blame] | 155 | EXPECT_EQ(packet.type(), PacketType::RPC); |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 156 | EXPECT_EQ(packet.channel_id(), 1u); |
| 157 | EXPECT_EQ(packet.service_id(), 42u); |
| 158 | EXPECT_EQ(packet.method_id(), 0u); // No method ID 27 |
| 159 | EXPECT_EQ(packet.status(), Status::NOT_FOUND); |
| 160 | } |
| 161 | |
| 162 | TEST_F(BasicServer, ProcessPacket_InvalidService_SendsNotFound) { |
| 163 | server_.ProcessPacket(EncodeRequest(PacketType::RPC, 1, 43, 27), output_); |
| 164 | |
| 165 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 166 | EXPECT_EQ(packet.status(), Status::NOT_FOUND); |
| 167 | EXPECT_EQ(packet.channel_id(), 1u); |
| 168 | EXPECT_EQ(packet.service_id(), 0u); |
| 169 | } |
| 170 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 171 | TEST_F(BasicServer, ProcessPacket_UnassignedChannel_AssignsToAvalableSlot) { |
Wyatt Hepler | 51a2eb7 | 2020-06-04 14:47:00 -0700 | [diff] [blame] | 172 | TestOutput<128> unassigned_output; |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 173 | server_.ProcessPacket( |
| 174 | EncodeRequest(PacketType::RPC, /*channel_id=*/99, 42, 27), |
| 175 | unassigned_output); |
| 176 | ASSERT_EQ(channels_[2].id(), 99u); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 177 | |
| 178 | Packet packet = Packet::FromBuffer(unassigned_output.sent_packet()); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 179 | EXPECT_EQ(packet.channel_id(), 99u); |
| 180 | EXPECT_EQ(packet.service_id(), 42u); |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 181 | EXPECT_EQ(packet.method_id(), 0u); // No method ID 27 |
| 182 | EXPECT_EQ(packet.status(), Status::NOT_FOUND); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 183 | } |
| 184 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 185 | TEST_F(BasicServer, |
| 186 | ProcessPacket_UnassignedChannel_SendsResourceExhaustedIfCannotAssign) { |
| 187 | channels_[2] = Channel::Create<3>(&output_); // Occupy only available channel |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 188 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 189 | server_.ProcessPacket( |
| 190 | EncodeRequest(PacketType::RPC, /*channel_id=*/99, 42, 27), output_); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 191 | |
Wyatt Hepler | 80f26ff | 2020-06-01 09:30:17 -0700 | [diff] [blame] | 192 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
Alexei Frolov | 33a1e8f | 2020-05-26 08:39:32 -0700 | [diff] [blame] | 193 | EXPECT_EQ(packet.status(), Status::RESOURCE_EXHAUSTED); |
| 194 | EXPECT_EQ(packet.channel_id(), 0u); |
| 195 | EXPECT_EQ(packet.service_id(), 0u); |
| 196 | EXPECT_EQ(packet.method_id(), 0u); |
Alexei Frolov | 26e3ae6 | 2020-05-04 17:06:17 -0700 | [diff] [blame] | 197 | } |
| 198 | |
Wyatt Hepler | af83568 | 2020-06-17 11:42:53 -0700 | [diff] [blame] | 199 | TEST_F(BasicServer, ProcessPacket_Cancel_ClosesServerWriter) { |
| 200 | // Set up a fake ServerWriter representing an ongoing RPC. |
| 201 | internal::ServerCall call(static_cast<internal::Server&>(server_), |
| 202 | static_cast<internal::Channel&>(channels_[0]), |
| 203 | service_, |
| 204 | service_.method(100)); |
| 205 | internal::BaseServerWriter writer(call); |
| 206 | ASSERT_TRUE(writer.open()); |
| 207 | |
| 208 | server_.ProcessPacket(EncodeRequest(PacketType::CANCEL, 1, 42, 100), output_); |
| 209 | |
| 210 | EXPECT_FALSE(writer.open()); |
| 211 | |
| 212 | Packet packet = Packet::FromBuffer(output_.sent_packet()); |
| 213 | EXPECT_EQ(packet.type(), PacketType::CANCEL); |
| 214 | EXPECT_EQ(packet.channel_id(), 1u); |
| 215 | EXPECT_EQ(packet.service_id(), 42u); |
| 216 | EXPECT_EQ(packet.method_id(), 100u); |
| 217 | EXPECT_TRUE(packet.payload().empty()); |
| 218 | EXPECT_EQ(packet.status(), Status::OK); |
| 219 | } |
| 220 | |
| 221 | TEST_F(BasicServer, ProcessPacket_Cancel_UnknownIdIsIgnored) { |
| 222 | internal::ServerCall call(static_cast<internal::Server&>(server_), |
| 223 | static_cast<internal::Channel&>(channels_[0]), |
| 224 | service_, |
| 225 | service_.method(100)); |
| 226 | internal::BaseServerWriter writer(call); |
| 227 | ASSERT_TRUE(writer.open()); |
| 228 | |
| 229 | // Send packets with incorrect channel, service, and method ID. |
| 230 | server_.ProcessPacket(EncodeRequest(PacketType::CANCEL, 2, 42, 100), output_); |
| 231 | server_.ProcessPacket(EncodeRequest(PacketType::CANCEL, 1, 43, 100), output_); |
| 232 | server_.ProcessPacket(EncodeRequest(PacketType::CANCEL, 1, 42, 101), output_); |
| 233 | |
| 234 | EXPECT_TRUE(writer.open()); |
| 235 | EXPECT_TRUE(output_.sent_packet().empty()); |
| 236 | } |
| 237 | |
Alexei Frolov | 26e3ae6 | 2020-05-04 17:06:17 -0700 | [diff] [blame] | 238 | } // namespace |
| 239 | } // namespace pw::rpc |