Add the channel name into the sender creation failure message
It used to just fail saying "Too Many Senders". This isn't actionable.
We want the chanel name in there.
The fix is to propegate the failure up to the event loop where it can
make a more educated decision. Also, add a test (and make sure
simulation matches).
Change-Id: If70397ee319ad25ce5ab7ed146e8f5057d8af100
diff --git a/aos/events/event_loop_param_test.cc b/aos/events/event_loop_param_test.cc
index fbcb62e..71bef37 100644
--- a/aos/events/event_loop_param_test.cc
+++ b/aos/events/event_loop_param_test.cc
@@ -604,6 +604,18 @@
"/test");
}
+// Verify that creating too many senders fails.
+TEST_P(AbstractEventLoopDeathTest, TooManySenders) {
+ auto loop = Make();
+ std::vector<aos::Sender<TestMessage>> senders;
+ for (int i = 0; i < 10; ++i) {
+ senders.emplace_back(loop->MakeSender<TestMessage>("/test"));
+ }
+ EXPECT_DEATH({ loop->MakeSender<TestMessage>("/test"); },
+ "Failed to create sender on \\{ \"name\": \"/test\", \"type\": "
+ "\"aos.TestMessage\" \\}, too many senders.");
+}
+
// Verify that we can't create a sender inside OnRun.
TEST_P(AbstractEventLoopDeathTest, SenderInOnRun) {
auto loop1 = MakePrimary();