commit 3c9f92c8eda1181f1e053b1ee2d285b36f7d8329
author Austin Schuh <austin.schuh@bluerivertech.com> Tue Apr 30 17:56:42 2024 -0700
committer James Kuszmaul <jabukuszmaul+collab@gmail.com> Mon Jul 08 19:53:52 2024 -0700
tree 4d3c38cd2db613ab2921658589eb44add7759c5e
parent 847927d985645468de7957cf72d9ba3e0ae17496

Change LogReader API to be able to replace messages

The mutation API in LogReader was not able to express dropping messages,
or growing messages.  This enables more aggressive mutation.

Change-Id: I477482da4262483a780d15ebf8c98a51e37099f6
Signed-off-by: James Kuszmaul <james.kuszmaul@bluerivertech.com>

aos/events/logging/logfile_utils_test.cc

diff --git a/aos/events/logging/logfile_utils_test.cc b/aos/events/logging/logfile_utils_test.cc
index 4ceca20..87c6f5d 100644
--- a/aos/events/logging/logfile_utils_test.cc
+++ b/aos/events/logging/logfile_utils_test.cc
@@ -274,6 +274,7 @@
                  BootTimestamp{.boot = 0, .time = e + chrono::milliseconds(1)},
              .monotonic_remote_boot = 0xffffff,
              .monotonic_timestamp_boot = 0xffffff,
+             .header = nullptr,
              .data = nullptr};
   Message m2{.channel_index = 0,
              .queue_index = BootQueueIndex{.boot = 0, .index = 0u},
@@ -281,6 +282,7 @@
                  BootTimestamp{.boot = 0, .time = e + chrono::milliseconds(2)},
              .monotonic_remote_boot = 0xffffff,
              .monotonic_timestamp_boot = 0xffffff,
+             .header = nullptr,
              .data = nullptr};
 
   EXPECT_LT(m1, m2);
@@ -960,24 +962,24 @@
 
   EXPECT_EQ(output[0].timestamp.boot, 0u);
   EXPECT_EQ(output[0].timestamp.time, e + chrono::milliseconds(101000));
-  EXPECT_FALSE(output[0].data->has_monotonic_timestamp_time);
+  EXPECT_FALSE(output[0].header->has_monotonic_timestamp_time);
 
   EXPECT_EQ(output[1].timestamp.boot, 0u);
   EXPECT_EQ(output[1].timestamp.time, e + chrono::milliseconds(101001));
-  EXPECT_TRUE(output[1].data->has_monotonic_timestamp_time);
-  EXPECT_EQ(output[1].data->monotonic_timestamp_time,
+  EXPECT_TRUE(output[1].header->has_monotonic_timestamp_time);
+  EXPECT_EQ(output[1].header->monotonic_timestamp_time,
             monotonic_clock::time_point(std::chrono::nanoseconds(971)));
 
   EXPECT_EQ(output[2].timestamp.boot, 0u);
   EXPECT_EQ(output[2].timestamp.time, e + chrono::milliseconds(101002));
-  EXPECT_TRUE(output[2].data->has_monotonic_timestamp_time);
-  EXPECT_EQ(output[2].data->monotonic_timestamp_time,
+  EXPECT_TRUE(output[2].header->has_monotonic_timestamp_time);
+  EXPECT_EQ(output[2].header->monotonic_timestamp_time,
             monotonic_clock::time_point(std::chrono::nanoseconds(972)));
 
   EXPECT_EQ(output[3].timestamp.boot, 0u);
   EXPECT_EQ(output[3].timestamp.time, e + chrono::milliseconds(101003));
-  EXPECT_TRUE(output[3].data->has_monotonic_timestamp_time);
-  EXPECT_EQ(output[3].data->monotonic_timestamp_time,
+  EXPECT_TRUE(output[3].header->has_monotonic_timestamp_time);
+  EXPECT_EQ(output[3].header->monotonic_timestamp_time,
             monotonic_clock::time_point(std::chrono::nanoseconds(973)));
 }