1 // Copyright 2022 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/pwpb/internal/method.h"
16
17 #include <array>
18
19 #include "pw_containers/algorithm.h"
20 #include "pw_rpc/internal/lock.h"
21 #include "pw_rpc/internal/method_impl_tester.h"
22 #include "pw_rpc/internal/test_utils.h"
23 #include "pw_rpc/pwpb/internal/method_union.h"
24 #include "pw_rpc/service.h"
25 #include "pw_rpc_pwpb_private/internal_test_utils.h"
26 #include "pw_rpc_test_protos/test.pwpb.h"
27 #include "pw_unit_test/framework.h"
28
29 PW_MODIFY_DIAGNOSTICS_PUSH();
30 PW_MODIFY_DIAGNOSTIC(ignored, "-Wmissing-field-initializers");
31
32 namespace pw::rpc::internal {
33 namespace {
34
35 using std::byte;
36
37 struct FakePb {};
38
39 // Create a fake service for use with the MethodImplTester.
40 class TestPwpbService final : public Service {
41 public:
42 // Unary signatures
43
Unary(const FakePb &,FakePb &)44 Status Unary(const FakePb&, FakePb&) { return Status(); }
45
StaticUnary(const FakePb &,FakePb &)46 static Status StaticUnary(const FakePb&, FakePb&) { return Status(); }
47
AsyncUnary(const FakePb &,PwpbUnaryResponder<FakePb> &)48 void AsyncUnary(const FakePb&, PwpbUnaryResponder<FakePb>&) {}
49
StaticAsyncUnary(const FakePb &,PwpbUnaryResponder<FakePb> &)50 static void StaticAsyncUnary(const FakePb&, PwpbUnaryResponder<FakePb>&) {}
51
UnaryWrongArg(FakePb &,FakePb &)52 Status UnaryWrongArg(FakePb&, FakePb&) { return Status(); }
53
StaticUnaryVoidReturn(const FakePb &,FakePb &)54 static void StaticUnaryVoidReturn(const FakePb&, FakePb&) {}
55
56 // Server streaming signatures
57
ServerStreaming(const FakePb &,PwpbServerWriter<FakePb> &)58 void ServerStreaming(const FakePb&, PwpbServerWriter<FakePb>&) {}
59
StaticServerStreaming(const FakePb &,PwpbServerWriter<FakePb> &)60 static void StaticServerStreaming(const FakePb&, PwpbServerWriter<FakePb>&) {}
61
ServerStreamingBadReturn(const FakePb &,PwpbServerWriter<FakePb> &)62 int ServerStreamingBadReturn(const FakePb&, PwpbServerWriter<FakePb>&) {
63 return 5;
64 }
65
StaticServerStreamingMissingArg(PwpbServerWriter<FakePb> &)66 static void StaticServerStreamingMissingArg(PwpbServerWriter<FakePb>&) {}
67
68 // Client streaming signatures
69
ClientStreaming(PwpbServerReader<FakePb,FakePb> &)70 void ClientStreaming(PwpbServerReader<FakePb, FakePb>&) {}
71
StaticClientStreaming(PwpbServerReader<FakePb,FakePb> &)72 static void StaticClientStreaming(PwpbServerReader<FakePb, FakePb>&) {}
73
ClientStreamingBadReturn(PwpbServerReader<FakePb,FakePb> &)74 int ClientStreamingBadReturn(PwpbServerReader<FakePb, FakePb>&) { return 0; }
75
StaticClientStreamingMissingArg()76 static void StaticClientStreamingMissingArg() {}
77
78 // Bidirectional streaming signatures
79
BidirectionalStreaming(PwpbServerReaderWriter<FakePb,FakePb> &)80 void BidirectionalStreaming(PwpbServerReaderWriter<FakePb, FakePb>&) {}
81
StaticBidirectionalStreaming(PwpbServerReaderWriter<FakePb,FakePb> &)82 static void StaticBidirectionalStreaming(
83 PwpbServerReaderWriter<FakePb, FakePb>&) {}
84
BidirectionalStreamingBadReturn(PwpbServerReaderWriter<FakePb,FakePb> &)85 int BidirectionalStreamingBadReturn(PwpbServerReaderWriter<FakePb, FakePb>&) {
86 return 0;
87 }
88
StaticBidirectionalStreamingMissingArg()89 static void StaticBidirectionalStreamingMissingArg() {}
90 };
91
92 struct WrongPb;
93
94 // Test matches() rejects incorrect request/response types.
95 // clang-format off
96 static_assert(!PwpbMethod::template matches<&TestPwpbService::Unary, WrongPb, FakePb>());
97 static_assert(!PwpbMethod::template matches<&TestPwpbService::Unary, FakePb, WrongPb>());
98 static_assert(!PwpbMethod::template matches<&TestPwpbService::Unary, WrongPb, WrongPb>());
99 static_assert(!PwpbMethod::template matches<&TestPwpbService::StaticUnary, FakePb, WrongPb>());
100
101 static_assert(!PwpbMethod::template matches<&TestPwpbService::ServerStreaming, WrongPb, FakePb>());
102 static_assert(!PwpbMethod::template matches<&TestPwpbService::StaticServerStreaming, FakePb, WrongPb>());
103
104 static_assert(!PwpbMethod::template matches<&TestPwpbService::ClientStreaming, WrongPb, FakePb>());
105 static_assert(!PwpbMethod::template matches<&TestPwpbService::StaticClientStreaming, FakePb, WrongPb>());
106
107 static_assert(!PwpbMethod::template matches<&TestPwpbService::BidirectionalStreaming, WrongPb, FakePb>());
108 static_assert(!PwpbMethod::template matches<&TestPwpbService::StaticBidirectionalStreaming, FakePb, WrongPb>());
109 // clang-format on
110
111 static_assert(MethodImplTests<PwpbMethod, TestPwpbService>().Pass(
112 MatchesTypes<FakePb, FakePb>(),
113 std::tuple<const PwpbMethodSerde&>(kPwpbMethodSerde<nullptr, nullptr>)));
114
115 template <typename Impl>
116 class FakeServiceBase : public Service {
117 public:
FakeServiceBase(uint32_t id)118 FakeServiceBase(uint32_t id) : Service(id, kMethods) {}
119
120 static constexpr std::array<PwpbMethodUnion, 5> kMethods = {
121 PwpbMethod::SynchronousUnary<&Impl::DoNothing>(
122 10u,
123 kPwpbMethodSerde<&pw::rpc::test::pwpb::Empty::kMessageFields,
124 &pw::rpc::test::pwpb::Empty::kMessageFields>),
125 PwpbMethod::AsynchronousUnary<&Impl::AddFive>(
126 11u,
127 kPwpbMethodSerde<&pw::rpc::test::pwpb::TestRequest::kMessageFields,
128 &pw::rpc::test::pwpb::TestResponse::kMessageFields>),
129 PwpbMethod::ServerStreaming<&Impl::StartStream>(
130 12u,
131 kPwpbMethodSerde<&pw::rpc::test::pwpb::TestRequest::kMessageFields,
132 &pw::rpc::test::pwpb::TestResponse::kMessageFields>),
133 PwpbMethod::ClientStreaming<&Impl::ClientStream>(
134 13u,
135 kPwpbMethodSerde<&pw::rpc::test::pwpb::TestRequest::kMessageFields,
136 &pw::rpc::test::pwpb::TestResponse::kMessageFields>),
137 PwpbMethod::BidirectionalStreaming<&Impl::BidirectionalStream>(
138 14u,
139 kPwpbMethodSerde<&pw::rpc::test::pwpb::TestRequest::kMessageFields,
140 &pw::rpc::test::pwpb::TestResponse::kMessageFields>),
141 };
142 };
143
144 class FakeService : public FakeServiceBase<FakeService> {
145 public:
FakeService(uint32_t id)146 FakeService(uint32_t id) : FakeServiceBase(id) {}
147
DoNothing(const pw::rpc::test::pwpb::Empty::Message &,pw::rpc::test::pwpb::Empty::Message &)148 Status DoNothing(const pw::rpc::test::pwpb::Empty::Message&,
149 pw::rpc::test::pwpb::Empty::Message&) {
150 return Status::Unknown();
151 }
152
AddFive(const pw::rpc::test::pwpb::TestRequest::Message & request,PwpbUnaryResponder<pw::rpc::test::pwpb::TestResponse::Message> & responder)153 void AddFive(const pw::rpc::test::pwpb::TestRequest::Message& request,
154 PwpbUnaryResponder<pw::rpc::test::pwpb::TestResponse::Message>&
155 responder) {
156 last_request = request;
157
158 if (fail_to_encode_async_unary_response) {
159 pw::rpc::test::pwpb::TestResponse::Message response = {};
160 response.repeated_field.SetEncoder(
161 [](const pw::rpc::test::pwpb::TestResponse::StreamEncoder&) {
162 return Status::Internal();
163 });
164 ASSERT_EQ(OkStatus(), responder.Finish(response, Status::NotFound()));
165 } else {
166 ASSERT_EQ(
167 OkStatus(),
168 responder.Finish({.value = static_cast<int32_t>(request.integer + 5)},
169 Status::Unauthenticated()));
170 }
171 }
172
StartStream(const pw::rpc::test::pwpb::TestRequest::Message & request,PwpbServerWriter<pw::rpc::test::pwpb::TestResponse::Message> & writer)173 void StartStream(
174 const pw::rpc::test::pwpb::TestRequest::Message& request,
175 PwpbServerWriter<pw::rpc::test::pwpb::TestResponse::Message>& writer) {
176 last_request = request;
177 last_writer = std::move(writer);
178 }
179
ClientStream(PwpbServerReader<pw::rpc::test::pwpb::TestRequest::Message,pw::rpc::test::pwpb::TestResponse::Message> & reader)180 void ClientStream(
181 PwpbServerReader<pw::rpc::test::pwpb::TestRequest::Message,
182 pw::rpc::test::pwpb::TestResponse::Message>& reader) {
183 last_reader = std::move(reader);
184 }
185
BidirectionalStream(PwpbServerReaderWriter<pw::rpc::test::pwpb::TestRequest::Message,pw::rpc::test::pwpb::TestResponse::Message> & reader_writer)186 void BidirectionalStream(
187 PwpbServerReaderWriter<pw::rpc::test::pwpb::TestRequest::Message,
188 pw::rpc::test::pwpb::TestResponse::Message>&
189 reader_writer) {
190 last_reader_writer = std::move(reader_writer);
191 }
192
193 bool fail_to_encode_async_unary_response = false;
194
195 pw::rpc::test::pwpb::TestRequest::Message last_request;
196 PwpbServerWriter<pw::rpc::test::pwpb::TestResponse::Message> last_writer;
197 PwpbServerReader<pw::rpc::test::pwpb::TestRequest::Message,
198 pw::rpc::test::pwpb::TestResponse::Message>
199 last_reader;
200 PwpbServerReaderWriter<pw::rpc::test::pwpb::TestRequest::Message,
201 pw::rpc::test::pwpb::TestResponse::Message>
202 last_reader_writer;
203 };
204
205 constexpr const PwpbMethod& kSyncUnary =
206 std::get<0>(FakeServiceBase<FakeService>::kMethods).pwpb_method();
207 constexpr const PwpbMethod& kAsyncUnary =
208 std::get<1>(FakeServiceBase<FakeService>::kMethods).pwpb_method();
209 constexpr const PwpbMethod& kServerStream =
210 std::get<2>(FakeServiceBase<FakeService>::kMethods).pwpb_method();
211 constexpr const PwpbMethod& kClientStream =
212 std::get<3>(FakeServiceBase<FakeService>::kMethods).pwpb_method();
213 constexpr const PwpbMethod& kBidirectionalStream =
214 std::get<4>(FakeServiceBase<FakeService>::kMethods).pwpb_method();
215
TEST(PwpbMethod,AsyncUnaryRpc_SendsResponse)216 TEST(PwpbMethod, AsyncUnaryRpc_SendsResponse) {
217 PW_ENCODE_PB(pw::rpc::test::pwpb::TestRequest,
218 request,
219 .integer = 123,
220 .status_code = 0);
221
222 ServerContextForTest<FakeService> context(kAsyncUnary);
223 rpc_lock().lock();
224 kAsyncUnary.Invoke(context.get(), context.request(request));
225
226 const Packet& response = context.output().last_packet();
227 EXPECT_EQ(response.status(), Status::Unauthenticated());
228
229 // Field 1 (encoded as 1 << 3) with 128 as the value.
230 constexpr std::byte expected[]{
231 std::byte{0x08}, std::byte{0x80}, std::byte{0x01}};
232
233 EXPECT_EQ(sizeof(expected), response.payload().size());
234 EXPECT_EQ(0,
235 std::memcmp(expected, response.payload().data(), sizeof(expected)));
236
237 EXPECT_EQ(123, context.service().last_request.integer);
238 }
239
TEST(PwpbMethod,SyncUnaryRpc_InvalidPayload_SendsError)240 TEST(PwpbMethod, SyncUnaryRpc_InvalidPayload_SendsError) {
241 std::array<byte, 8> bad_payload{byte{0xFF}, byte{0xAA}, byte{0xDD}};
242
243 ServerContextForTest<FakeService> context(kSyncUnary);
244 rpc_lock().lock();
245 kSyncUnary.Invoke(context.get(), context.request(bad_payload));
246
247 const Packet& packet = context.output().last_packet();
248 EXPECT_EQ(pwpb::PacketType::SERVER_ERROR, packet.type());
249 EXPECT_EQ(Status::DataLoss(), packet.status());
250 EXPECT_EQ(context.service_id(), packet.service_id());
251 EXPECT_EQ(kSyncUnary.id(), packet.method_id());
252 }
253
TEST(PwpbMethod,AsyncUnaryRpc_ResponseEncodingFails_SendsInternalError)254 TEST(PwpbMethod, AsyncUnaryRpc_ResponseEncodingFails_SendsInternalError) {
255 constexpr int64_t value = 0x7FFFFFFF'FFFFFF00ll;
256 PW_ENCODE_PB(pw::rpc::test::pwpb::TestRequest,
257 request,
258 .integer = value,
259 .status_code = 0);
260
261 ServerContextForTest<FakeService> context(kAsyncUnary);
262 context.service().fail_to_encode_async_unary_response = true;
263
264 rpc_lock().lock();
265 kAsyncUnary.Invoke(context.get(), context.request(request));
266
267 const Packet& packet = context.output().last_packet();
268 EXPECT_EQ(pwpb::PacketType::SERVER_ERROR, packet.type());
269 EXPECT_EQ(Status::Internal(), packet.status());
270 EXPECT_EQ(context.service_id(), packet.service_id());
271 EXPECT_EQ(kAsyncUnary.id(), packet.method_id());
272
273 EXPECT_EQ(value, context.service().last_request.integer);
274 }
275
TEST(PwpbMethod,ServerStreamingRpc_SendsNothingWhenInitiallyCalled)276 TEST(PwpbMethod, ServerStreamingRpc_SendsNothingWhenInitiallyCalled) {
277 PW_ENCODE_PB(pw::rpc::test::pwpb::TestRequest,
278 request,
279 .integer = 555,
280 .status_code = 0);
281
282 ServerContextForTest<FakeService> context(kServerStream);
283
284 rpc_lock().lock();
285 kServerStream.Invoke(context.get(), context.request(request));
286
287 EXPECT_EQ(0u, context.output().total_packets());
288 EXPECT_EQ(555, context.service().last_request.integer);
289 }
290
TEST(PwpbMethod,ServerWriter_SendsResponse)291 TEST(PwpbMethod, ServerWriter_SendsResponse) {
292 ServerContextForTest<FakeService> context(kServerStream);
293
294 rpc_lock().lock();
295 kServerStream.Invoke(context.get(), context.request({}));
296
297 EXPECT_EQ(OkStatus(), context.service().last_writer.Write({.value = 100}));
298
299 PW_ENCODE_PB(pw::rpc::test::pwpb::TestResponse, payload, .value = 100);
300 std::array<byte, 128> encoded_response = {};
301 auto encoded = context.server_stream(payload).Encode(encoded_response);
302 ASSERT_EQ(OkStatus(), encoded.status());
303
304 ConstByteSpan sent_payload = context.output().last_packet().payload();
305 EXPECT_TRUE(pw::containers::Equal(payload, sent_payload));
306 }
307
TEST(PwpbMethod,ServerWriter_WriteWhenClosed_ReturnsFailedPrecondition)308 TEST(PwpbMethod, ServerWriter_WriteWhenClosed_ReturnsFailedPrecondition) {
309 ServerContextForTest<FakeService> context(kServerStream);
310
311 rpc_lock().lock();
312 kServerStream.Invoke(context.get(), context.request({}));
313
314 EXPECT_EQ(OkStatus(), context.service().last_writer.Finish());
315 EXPECT_TRUE(context.service()
316 .last_writer.Write({.value = 100})
317 .IsFailedPrecondition());
318 }
319
TEST(PwpbMethod,ServerWriter_WriteAfterMoved_ReturnsFailedPrecondition)320 TEST(PwpbMethod, ServerWriter_WriteAfterMoved_ReturnsFailedPrecondition) {
321 ServerContextForTest<FakeService> context(kServerStream);
322
323 rpc_lock().lock();
324 kServerStream.Invoke(context.get(), context.request({}));
325 PwpbServerWriter<pw::rpc::test::pwpb::TestResponse::Message> new_writer =
326 std::move(context.service().last_writer);
327
328 EXPECT_EQ(OkStatus(), new_writer.Write({.value = 100}));
329
330 EXPECT_EQ(Status::FailedPrecondition(),
331 context.service().last_writer.Write({.value = 100}));
332 EXPECT_EQ(Status::FailedPrecondition(),
333 context.service().last_writer.Finish());
334
335 EXPECT_EQ(OkStatus(), new_writer.Finish());
336 }
337
TEST(PwpbMethod,ServerStreamingRpc_ResponseEncodingFails_InternalError)338 TEST(PwpbMethod, ServerStreamingRpc_ResponseEncodingFails_InternalError) {
339 ServerContextForTest<FakeService> context(kServerStream);
340
341 rpc_lock().lock();
342 kServerStream.Invoke(context.get(), context.request({}));
343
344 EXPECT_EQ(OkStatus(), context.service().last_writer.Write({}));
345
346 pw::rpc::test::pwpb::TestResponse::Message response = {};
347 response.repeated_field.SetEncoder(
348 [](const pw::rpc::test::pwpb::TestResponse::StreamEncoder&) {
349 return Status::Internal();
350 });
351 EXPECT_EQ(Status::Internal(), context.service().last_writer.Write(response));
352 }
353
TEST(PwpbMethod,ServerReader_HandlesRequests)354 TEST(PwpbMethod, ServerReader_HandlesRequests) {
355 ServerContextForTest<FakeService> context(kClientStream);
356
357 rpc_lock().lock();
358 kClientStream.Invoke(context.get(), context.request({}));
359
360 pw::rpc::test::pwpb::TestRequest::Message request_struct{};
361 context.service().last_reader.set_on_next(
362 [&request_struct](const pw::rpc::test::pwpb::TestRequest::Message& req) {
363 request_struct = req;
364 });
365
366 PW_ENCODE_PB(pw::rpc::test::pwpb::TestRequest,
367 request,
368 .integer = 1 << 30,
369 .status_code = 9);
370 std::array<byte, 128> encoded_request = {};
371 auto encoded = context.client_stream(request).Encode(encoded_request);
372 ASSERT_EQ(OkStatus(), encoded.status());
373 ASSERT_EQ(OkStatus(), context.server().ProcessPacket(*encoded));
374
375 EXPECT_EQ(request_struct.integer, 1 << 30);
376 EXPECT_EQ(request_struct.status_code, 9u);
377 }
378
TEST(PwpbMethod,ServerReaderWriter_WritesResponses)379 TEST(PwpbMethod, ServerReaderWriter_WritesResponses) {
380 ServerContextForTest<FakeService> context(kBidirectionalStream);
381
382 rpc_lock().lock();
383 kBidirectionalStream.Invoke(context.get(), context.request({}));
384
385 EXPECT_EQ(OkStatus(),
386 context.service().last_reader_writer.Write({.value = 100}));
387
388 PW_ENCODE_PB(pw::rpc::test::pwpb::TestResponse, payload, .value = 100);
389 std::array<byte, 128> encoded_response = {};
390 auto encoded = context.server_stream(payload).Encode(encoded_response);
391 ASSERT_EQ(OkStatus(), encoded.status());
392
393 ConstByteSpan sent_payload = context.output().last_packet().payload();
394 EXPECT_TRUE(pw::containers::Equal(payload, sent_payload));
395 }
396
TEST(PwpbMethod,ServerReaderWriter_HandlesRequests)397 TEST(PwpbMethod, ServerReaderWriter_HandlesRequests) {
398 ServerContextForTest<FakeService> context(kBidirectionalStream);
399
400 rpc_lock().lock();
401 kBidirectionalStream.Invoke(context.get(), context.request({}));
402
403 pw::rpc::test::pwpb::TestRequest::Message request_struct{};
404 context.service().last_reader_writer.set_on_next(
405 [&request_struct](const pw::rpc::test::pwpb::TestRequest::Message& req) {
406 request_struct = req;
407 });
408
409 PW_ENCODE_PB(pw::rpc::test::pwpb::TestRequest,
410 request,
411 .integer = 1 << 29,
412 .status_code = 8);
413 std::array<byte, 128> encoded_request = {};
414 auto encoded = context.client_stream(request).Encode(encoded_request);
415 ASSERT_EQ(OkStatus(), encoded.status());
416 ASSERT_EQ(OkStatus(), context.server().ProcessPacket(*encoded));
417
418 EXPECT_EQ(request_struct.integer, 1 << 29);
419 EXPECT_EQ(request_struct.status_code, 8u);
420 }
421
422 } // namespace
423 } // namespace pw::rpc::internal
424
425 PW_MODIFY_DIAGNOSTICS_POP();
426