Add colour to remote schedule config (#34)

matthewturner · Feb 3, 2024 · 1069f9c · 1069f9c
1 parent 4ec7b35
commit 1069f9c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/src/main.cpp b/src/main.cpp
@@ -73,6 +73,7 @@ bool setSchedule(EvtListener *, EvtContext *, long data)
   Serial.print(F("Command: SET SCHEDULE "));
   Serial.println(data);
   displaySchedule.update(data);
+  colorSchedule.update(data);
   if (showAfterSet)
   {
     stateMachine.transition(SHOWING);

diff --git a/src/main.h b/src/main.h
@@ -38,7 +38,7 @@ RTC_DS3231 clock;
 EvtManager mgr;
 Timezone *timezone;
 CRGB::HTMLColorCode knownColors[4];
-Schedule<byte> colorSchedule(0);
+Schedule<byte> colorSchedule(0, 2);
 Schedule<Flags> displaySchedule(Flags::NONE, 1);
 SoftwareSerial bluetoothSerial(RECEIVE_PIN, TRANSMIT_PIN);
 EvtCommandListener commandListener(&bluetoothSerial, 20);

diff --git a/test/test_schedule/test_schedule.cpp b/test/test_schedule/test_schedule.cpp
@@ -91,6 +91,13 @@ void test_updates_only_specified_second_half(void)
     TEST_ASSERT_EQUAL(2, target.valueFor(14, 59));
 }
 
+void test_does_not_update_non_matching_identifier(void)
+{
+    bool actual = target.update(914022);
+
+    TEST_ASSERT_FALSE(actual);
+}
+
 int main(int argc, char **argv)
 {
     UNITY_BEGIN();
@@ -103,6 +110,7 @@ int main(int argc, char **argv)
     RUN_TEST(test_does_not_update_invalid_quarter_flag);
     RUN_TEST(test_updates_only_specified_first_half);
     RUN_TEST(test_updates_only_specified_second_half);
+    RUN_TEST(test_does_not_update_non_matching_identifier);
     UNITY_END();
 
     return 0;