commit 55badf7979f57d5f9ed2db726eebf35e84d0dac5
author Alexander Hansen <alexander.hansen@9elements.com> Wed Jul 24 14:35:13 2024 +0200
committer Alexander Hansen <alexander.hansen@9elements.com> Wed Aug 21 14:37:38 2024 +0200
tree 63bdfff261aeb5f64ed5903d8a839d69ab36fda7
parent 30b60d458c289b6879731e80df5b9b72a9f7200b

manager: led priority is optional

The default priority was "Blink" but now the priority is optional.

This helps to implement group priority.
So if someone wants to use group priority they can do so, without the
default led priority getting in the way.

The led priority can still be assigned as "Blink" explicitly if needed.

Change-Id: I9336487bcc2edc0b3308ee76eb5330b8222b7f25
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

test/utest-led-json.cpp

diff --git a/test/utest-led-json.cpp b/test/utest-led-json.cpp
index 36eb404..afd4505 100644
--- a/test/utest-led-json.cpp
+++ b/test/utest-led-json.cpp
@@ -26,7 +26,7 @@
         ASSERT_EQ(group.action, phosphor::led::Layout::Action::On);
         ASSERT_EQ(group.dutyOn, 50);
         ASSERT_EQ(group.period, 0);
-        ASSERT_EQ(group.priority, phosphor::led::Layout::Action::Blink);
+        ASSERT_EQ(group.priority, std::nullopt);
     }
 
     for (const auto& group : powerOnActions)
@@ -45,14 +45,14 @@
             ASSERT_EQ(group.action, phosphor::led::Layout::Action::Blink);
             ASSERT_EQ(group.dutyOn, 50);
             ASSERT_EQ(group.period, 1000);
-            ASSERT_EQ(group.priority, phosphor::led::Layout::Action::Blink);
+            ASSERT_EQ(group.priority, std::nullopt);
         }
         else if (group.name == "rear_id")
         {
             ASSERT_EQ(group.action, phosphor::led::Layout::Action::Blink);
             ASSERT_EQ(group.dutyOn, 50);
             ASSERT_EQ(group.period, 1000);
-            ASSERT_EQ(group.priority, phosphor::led::Layout::Action::Blink);
+            ASSERT_EQ(group.priority, std::nullopt);
         }
         else
         {