diff options
author | Andrew Jeffery <andrew@aj.id.au> | 2018-05-25 13:04:39 +0930 |
---|---|---|
committer | Andrew Jeffery <andrew@aj.id.au> | 2019-04-08 13:48:50 +0930 |
commit | 04275e06d2aab6a5e13807e0a295d3b48a979118 (patch) | |
tree | 47cf992aa94f491b58d697b79cea59d30106e950 /test/physical.cpp | |
parent | 56692708507e58e889c97123578b376dd84399d9 (diff) | |
download | phosphor-led-sysfs-04275e06d2aab6a5e13807e0a295d3b48a979118.tar.gz phosphor-led-sysfs-04275e06d2aab6a5e13807e0a295d3b48a979118.zip |
test: physical: Cover trigger:none, brigtness:asserted branch
The test causes execution to take the following path in
setInitialState(), previously outlined in 264d909d3dc9 ("test: Add tests
for Physical class") as being missed:
auto brightness = led.getBrightness();
if (brightness == ASSERT)
{
// LED is in Solid ON
sdbusplus::xyz::openbmc_project::Led::server ::Physical::state(
Action::On);
}
This brings the line coverage to 97.6% (function coverage remains
unchanged by this patch).
Change-Id: I7a04fcd97819559575e69d267c62f16195495010
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Diffstat (limited to 'test/physical.cpp')
-rw-r--r-- | test/physical.cpp | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/test/physical.cpp b/test/physical.cpp index 897a43f..9312598 100644 --- a/test/physical.cpp +++ b/test/physical.cpp @@ -115,3 +115,13 @@ TEST(Physical, blink) phosphor::led::Physical phy(bus, LED_OBJ, led); phy.state(Action::Blink); } + +TEST(Physical, ctor_none_trigger_asserted_brightness) +{ + sdbusplus::bus::bus bus = sdbusplus::bus::new_default(); + NiceMock<MockLed> led; + EXPECT_CALL(led, getTrigger()).WillRepeatedly(Return("none")); + constexpr auto val = phosphor::led::ASSERT; + EXPECT_CALL(led, getBrightness()).WillRepeatedly(Return(val)); + phosphor::led::Physical phy(bus, LED_OBJ, led); +} |