From 31990490fa4808e4e0bc23fa0308cd1ddbf22457 Mon Sep 17 00:00:00 2001 From: Nils Winkler Date: Thu, 2 Nov 2017 06:15:34 +0100 Subject: [PATCH] Added test for the "no" energy tool case --- plugins/available/battery.plugin.bash | 6 +++++- test/plugins/battery.plugin.bats | 12 ++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/plugins/available/battery.plugin.bash b/plugins/available/battery.plugin.bash index b9d00514..b9bee172 100644 --- a/plugins/available/battery.plugin.bash +++ b/plugins/available/battery.plugin.bash @@ -74,7 +74,11 @@ battery_percentage(){ COMMAND_OUTPUT="no" fi - printf "%02d" "${COMMAND_OUTPUT:--1}" + if [ "${COMMAND_OUTPUT}" != "no" ]; then + printf "%02d" "${COMMAND_OUTPUT:--1}" + else + echo "${COMMAND_OUTPUT}" + fi } battery_charge(){ diff --git a/test/plugins/battery.plugin.bats b/test/plugins/battery.plugin.bats index 685b2072..88e20f25 100644 --- a/test/plugins/battery.plugin.bats +++ b/test/plugins/battery.plugin.bats @@ -34,6 +34,18 @@ function setup_command_exists { } } +####################### +# +# no tool +# + +@test 'plugins battery: battery-percentage with no tool' { + setup_command_exists "fooooo" + + run battery_percentage + assert_output "no" +} + ####################### # # pmset