Make ValueSchedule trivially copiable

kivancsikert · Feb 1, 2024 · 6c868ac · 6c868ac
1 parent 911d2fa
commit 6c868ac
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/peripherals/valve/ValveScheduler.hpp b/src/peripherals/valve/ValveScheduler.hpp
@@ -39,9 +39,10 @@ class ValveSchedule {
         return duration;
     }
 
-    const time_point<system_clock> start;
-    const seconds period;
-    const seconds duration;
+private:
+    time_point<system_clock> start;
+    seconds period;
+    seconds duration;
 };
 
 struct ValveStateUpdate {

diff --git a/test/ValveSchedulerTest.cpp b/test/ValveSchedulerTest.cpp
@@ -31,9 +31,9 @@ class ValveSchedulerTest : public testing::Test {
 
 TEST_F(ValveSchedulerTest, can_create_schedule) {
     ValveSchedule schedule(base, hours { 1 }, minutes { 1 });
-    EXPECT_EQ(schedule.start, base);
-    EXPECT_EQ(schedule.period, hours { 1 });
-    EXPECT_EQ(schedule.duration, minutes { 1 });
+    EXPECT_EQ(schedule.getStart(), base);
+    EXPECT_EQ(schedule.getPeriod(), hours { 1 });
+    EXPECT_EQ(schedule.getDuration(), minutes { 1 });
 }
 
 // TEST_F(ValveSchedulerTest, can_create_schedule_from_json) {