config: allow led priority to be Off
This makes the configuration more flexible and eliminates the edge case.
I don't see why this should not be possible.
Change-Id: I28e33535539b3214e18d7816314656df5d3c370e
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
diff --git a/test/utest-led-json.cpp b/test/utest-led-json.cpp
index afd4505..fa9161b 100644
--- a/test/utest-led-json.cpp
+++ b/test/utest-led-json.cpp
@@ -10,14 +10,17 @@
std::string objPath = "/xyz/openbmc_project/led/groups";
std::string bmcBooted = objPath + "/bmc_booted";
std::string powerOn = objPath + "/power_on";
+ std::string powerOff = objPath + "/power_off";
std::string enclosureIdentify = objPath + "/enclosure_identify";
ASSERT_EQ(ledMap.contains(bmcBooted), true);
ASSERT_EQ(ledMap.contains(powerOn), true);
+ ASSERT_EQ(ledMap.contains(powerOff), true);
ASSERT_EQ(ledMap.contains(enclosureIdentify), true);
auto& bmcBootedActions = ledMap.at(bmcBooted).actionSet;
auto& powerOnActions = ledMap.at(powerOn).actionSet;
+ auto& powerOffActions = ledMap.at(powerOff).actionSet;
auto& enclosureIdentifyActions = ledMap.at(enclosureIdentify).actionSet;
for (const auto& group : bmcBootedActions)
@@ -37,6 +40,12 @@
ASSERT_EQ(group.period, 0);
ASSERT_EQ(group.priority, phosphor::led::Layout::Action::On);
}
+ for (const auto& group : powerOffActions)
+ {
+ ASSERT_EQ(group.name, "power_off_led");
+ ASSERT_EQ(group.action, phosphor::led::Layout::Action::Off);
+ ASSERT_EQ(group.priority, phosphor::led::Layout::Action::Off);
+ }
for (const auto& group : enclosureIdentifyActions)
{