commit f645c7dcfc28cff2e898f0e9404558aa7309a0eb
author James Kuszmaul <jabukuszmaul@gmail.com> Thu Feb 23 14:21:04 2023 -0800
committer James Kuszmaul <jabukuszmaul@gmail.com> Thu Feb 23 14:28:38 2023 -0800
tree edc12a1d8f78a7b7938a002c999fb9c175e51039
parent f9f5ccd031f7532e8b9d011f4fc7182b1630c67b

Add more detail to CHECK failures in configuration.cc

Change-Id: I6d853c07b4c0eaebd2d4c0d29e887f4c872efb09
Signed-off-by: James Kuszmaul <jabukuszmaul@gmail.com>

aos/configuration.cc

diff --git a/aos/configuration.cc b/aos/configuration.cc
index 1d5e60a..ad0a489 100644
--- a/aos/configuration.cc
+++ b/aos/configuration.cc
@@ -1355,8 +1355,10 @@
     case LoggerConfig::LOCAL_LOGGER:
       return channel->source_node()->string_view() == node_name;
     case LoggerConfig::LOCAL_AND_REMOTE_LOGGER:
-      CHECK(channel->has_logger_nodes());
-      CHECK_GT(channel->logger_nodes()->size(), 0u);
+      CHECK(channel->has_logger_nodes())
+          << "Missing logger nodes on " << StrippedChannelToString(channel);
+      CHECK_GT(channel->logger_nodes()->size(), 0u)
+          << "Missing logger nodes on " << StrippedChannelToString(channel);
 
       if (channel->source_node()->string_view() == node_name) {
         return true;
@@ -1364,8 +1366,10 @@
 
       [[fallthrough]];
     case LoggerConfig::REMOTE_LOGGER:
-      CHECK(channel->has_logger_nodes());
-      CHECK_GT(channel->logger_nodes()->size(), 0u);
+      CHECK(channel->has_logger_nodes())
+          << "Missing logger nodes on " << StrippedChannelToString(channel);
+      CHECK_GT(channel->logger_nodes()->size(), 0u)
+          << "Missing logger nodes on " << StrippedChannelToString(channel);
       for (const flatbuffers::String *logger_node : *channel->logger_nodes()) {
         if (logger_node->string_view() == node_name) {
           return true;