Add indicator_power_device_get_icon_names().

The main goal of this change is to make it possible to test the
device's icon. A secondary goal is to clarify in the code how
indicator-power's icons differ from the ones recommended by GSD.

4 files changed, 451 insertions, 114 deletions

diff --git a/src/device.c b/src/device.c
index bd182bf..3dbfd4e 100644
--- a/src/device.c
+++ b/src/device.c
@@ -211,7 +211,7 @@ set_property (GObject * o, guint prop_id, const GValue * value, GParamSpec * psp
 }
 
 /***
-****
+****  Accessors
 ***/
 
 UpDeviceKind
@@ -276,6 +276,247 @@ indicator_power_device_get_time (const IndicatorPowerDevice * device)
 
 /***
 ****
+****
+***/
+
+/* taken from GSD's power plugin, (c) Richard Hughes and licensed GPL >=2 */
+static const gchar *
+gpm_upower_get_device_icon_suffix (gdouble percentage)
+{
+        if (percentage < 10) return "caution";
+        if (percentage < 30) return "low";
+        if (percentage < 60) return "good";
+        return "full";
+}
+
+/* taken from GSD's power plugin, (c) Richard Hughes and licensed GPL >=2 */
+static const gchar *
+gpm_upower_get_device_icon_index (gdouble percentage)
+{
+        if (percentage < 10) return "000";
+        if (percentage < 30) return "020";
+        if (percentage < 50) return "040";
+        if (percentage < 70) return "060";
+        if (percentage < 90) return "080";
+        return "100";
+}
+
+/**
+  indicator_power_device_get_icon_names:
+  @device: #IndicatorPowerDevice to generate the icon names from
+
+  Based on GSD's power plugin, (c) Richard Hughes and licensed GPL >= 2.
+  It differs in these ways:
+
+  (1) all charging batteries use the same icon regardless of progress:
+  <https://bugs.launchpad.net/indicator-power/+bug/824629/comments/7>
+
+  (2) discharging batteries are keyed off of time left, rather than
+  percentage left, s.t. <= 30 minutes remaining gives the 'caution' icon.
+  <https://bugs.launchpad.net/indicator-power/+bug/743823>
+
+  Return value: (array zero-terminated=1) (transfer full):
+  a GStrv of icon names suitable for passing to g_themed_icon_new_from_names().
+  Free with g_strfreev() when done.
+*/
+GStrv
+indicator_power_device_get_icon_names (const IndicatorPowerDevice * device)
+{
+  char ** ret = NULL;
+  const gchar *suffix_str;
+  const gchar *index_str;
+
+  /* LCOV_EXCL_START */
+  g_return_val_if_fail (INDICATOR_IS_POWER_DEVICE(device), UP_DEVICE_KIND_UNKNOWN);
+  /* LCOV_EXCL_STOP */
+
+  gdouble percentage = indicator_power_device_get_percentage (device);
+  const UpDeviceKind kind = indicator_power_device_get_kind (device);
+  const UpDeviceState state = indicator_power_device_get_state (device);
+  const char * kind_str = kind_str = up_device_kind_to_string (kind);
+
+  /* get correct icon prefix */
+  GString * filename = g_string_new (NULL);
+
+  /* get the icon from some simple rules */
+  if (kind == UP_DEVICE_KIND_LINE_POWER) {
+    g_string_append (filename, "ac-adapter-symbolic;");
+    g_string_append (filename, "ac-adapter;");
+  } else if (kind == UP_DEVICE_KIND_MONITOR) {
+    g_string_append (filename, "gpm-monitor-symbolic;");
+    g_string_append (filename, "gpm-monitor;");
+  } else switch (state) {
+    case UP_DEVICE_STATE_EMPTY:
+      g_string_append (filename, "battery-empty-symbolic;");
+      g_string_append_printf (filename, "gpm-%s-empty;", kind_str);
+      g_string_append_printf (filename, "gpm-%s-000;", kind_str);
+      g_string_append (filename, "battery-empty;");
+      break;
+    case UP_DEVICE_STATE_FULLY_CHARGED:
+      g_string_append (filename, "battery-full-charged-symbolic;");
+      g_string_append (filename, "battery-full-charging-symbolic;");
+      g_string_append_printf (filename, "gpm-%s-full;", kind_str);
+      g_string_append_printf (filename, "gpm-%s-100;", kind_str);
+      g_string_append (filename, "battery-full-charged;");
+      g_string_append (filename, "battery-full-charging;");
+      break;
+    case UP_DEVICE_STATE_CHARGING:
+    case UP_DEVICE_STATE_PENDING_CHARGE:
+      /* When charging, always use the same icon regardless of percentage.
+         <http://bugs.launchpad.net/indicator-power/+bug/824629> */
+      percentage = 0;
+
+      suffix_str = gpm_upower_get_device_icon_suffix (percentage);
+      index_str = gpm_upower_get_device_icon_index (percentage);
+      g_string_append_printf (filename, "battery-%s-charging-symbolic;", suffix_str);
+      g_string_append_printf (filename, "gpm-%s-%s-charging;", kind_str, index_str);
+      g_string_append_printf (filename, "battery-%s-charging;", suffix_str);
+      break;
+    case UP_DEVICE_STATE_DISCHARGING:
+    case UP_DEVICE_STATE_PENDING_DISCHARGE: {
+      /* Don't show the caution/red icons unless we have <=30 min left.
+         <https://bugs.launchpad.net/indicator-power/+bug/743823>
+         Themes use the caution color when the percentage is 0% or 20%,
+         so if we have >30 min left, use 30% as the icon's percentage floor */
+      if (indicator_power_device_get_time (device) > (30*60))
+        percentage = MAX(percentage, 30);
+
+      suffix_str = gpm_upower_get_device_icon_suffix (percentage);
+      index_str = gpm_upower_get_device_icon_index (percentage);
+      g_string_append_printf (filename, "battery-%s-symbolic;", suffix_str);
+      g_string_append_printf (filename, "gpm-%s-%s;", kind_str, index_str);
+      g_string_append_printf (filename, "battery-%s;", suffix_str);
+      break;
+    }
+    default:
+      g_string_append (filename, "battery-missing-symbolic;");
+      g_string_append (filename, "gpm-battery-missing;");
+      g_string_append (filename, "battery-missing;");
+    }
+
+    ret = g_strsplit (filename->str, ";", -1);
+    g_string_free (filename, TRUE);
+    return ret;
+}
+
+/**
+  indicator_power_device_get_gicon:
+  @device: #IndicatorPowerDevice to generate the icon names from
+
+  A convenience function to call g_themed_icon_new_from_names()
+  with the names returned by indicator_power_device_get_icon_names()
+
+  Return value: (transfer full): A themed GIcon
+*/
+GIcon *
+indicator_power_device_get_gicon (const IndicatorPowerDevice * device)
+{
+  GStrv names = indicator_power_device_get_icon_names (device);
+  GIcon * icon = g_themed_icon_new_from_names (names, -1);
+  g_strfreev (names);
+  return icon;
+}
+
+
+
+#if 0
+static const gchar *
+get_icon_percentage_for_status (const gchar *status)
+{
+
+  if (g_strcmp0 (status, "caution") == 0)
+    return "000";
+  else if (g_strcmp0 (status, "low") == 0)
+    return "040";
+  else if (g_strcmp0 (status, "good") == 0)
+    return "080";
+  else
+    return "100";
+}
+
+static GIcon*
+build_battery_icon (UpDeviceState  state,
+                    gchar         *suffix_str)
+{
+  GIcon *gicon;
+
+  GString *filename;
+  gchar **iconnames;
+
+  filename = g_string_new (NULL);
+
+  if (state == UP_DEVICE_STATE_FULLY_CHARGED)
+    {
+      g_string_append (filename, "battery-charged;");
+      g_string_append (filename, "battery-full-charged-symbolic;");
+      g_string_append (filename, "battery-full-charged;");
+      g_string_append (filename, "gpm-battery-charged;");
+      g_string_append (filename, "gpm-battery-100-charging;");
+    }
+  else if (state == UP_DEVICE_STATE_CHARGING)
+    {
+      g_string_append (filename, "battery-000-charging;");
+      g_string_append (filename, "battery-caution-charging-symbolic;");
+      g_string_append (filename, "battery-caution-charging;");
+      g_string_append (filename, "gpm-battery-000-charging;");
+    }
+  else if (state == UP_DEVICE_STATE_DISCHARGING)
+    {
+      const gchar *percentage = get_icon_percentage_for_status (suffix_str);
+      g_string_append_printf (filename, "battery-%s;", suffix_str);
+      g_string_append_printf (filename, "battery-%s-symbolic;", suffix_str);
+      g_string_append_printf (filename, "battery-%s;", percentage);
+      g_string_append_printf (filename, "gpm-battery-%s;", percentage);
+    }
+
+  iconnames = g_strsplit (filename->str, ";", -1);
+  gicon = g_themed_icon_new_from_names (iconnames, -1);
+
+  g_strfreev (iconnames);
+  g_string_free (filename, TRUE);
+
+  return gicon;
+}
+
+static GIcon*
+get_device_icon (UpDeviceKind   kind,
+                 UpDeviceState  state,
+                 guint64        time_sec,
+                 const gchar   *device_icon)
+{
+  GIcon *gicon = NULL;
+
+  if (kind == UP_DEVICE_KIND_BATTERY &&
+      (state == UP_DEVICE_STATE_FULLY_CHARGED ||
+       state == UP_DEVICE_STATE_CHARGING ||
+       state == UP_DEVICE_STATE_DISCHARGING))
+    {
+      if (state == UP_DEVICE_STATE_FULLY_CHARGED ||
+          state == UP_DEVICE_STATE_CHARGING)
+        {
+          gicon = build_battery_icon (state, NULL);
+        }
+      else if (state == UP_DEVICE_STATE_DISCHARGING)
+        {
+          if ((time_sec > 60 * 30) && /* more than 30 minutes left */
+              (g_strrstr (device_icon, "000") ||
+               g_strrstr (device_icon, "020") ||
+               g_strrstr (device_icon, "caution"))) /* the icon is red */
+            {
+              gicon = build_battery_icon (state, "low");
+            }
+        }
+    }
+
+  if (gicon == NULL)
+    gicon = g_icon_new_for_string (device_icon, NULL);
+
+  return gicon;
+}
+#endif
+
+/***
+****  Instantiation
 ***/
 
 IndicatorPowerDevice *
diff --git a/src/device.h b/src/device.h
index e9de044..9b82830 100644
--- a/src/device.h
+++ b/src/device.h
@@ -94,6 +94,11 @@ const gchar * indicator_power_device_get_icon        (const IndicatorPowerDevice
 gdouble       indicator_power_device_get_percentage  (const IndicatorPowerDevice * device);
 time_t        indicator_power_device_get_time        (const IndicatorPowerDevice * device);
 
+GStrv         indicator_power_device_get_icon_names  (const IndicatorPowerDevice * device);
+GIcon       * indicator_power_device_get_gicon       (const IndicatorPowerDevice * device);
+
+
+
 
 G_END_DECLS
 
diff --git a/src/indicator-power.c b/src/indicator-power.c
index 7029739..96c0f1d 100644
--- a/src/indicator-power.c
+++ b/src/indicator-power.c
@@ -431,101 +431,6 @@ set_accessible_desc (IndicatorPower *self, const gchar *desc)
   }
 }
 
-static const gchar *
-get_icon_percentage_for_status (const gchar *status)
-{
-
-  if (g_strcmp0 (status, "caution") == 0)
-    return "000";
-  else if (g_strcmp0 (status, "low") == 0)
-    return "040";
-  else if (g_strcmp0 (status, "good") == 0)
-    return "080";
-  else
-    return "100";
-}
-
-static GIcon*
-build_battery_icon (UpDeviceState  state,
-                    gchar         *suffix_str)
-{
-  GIcon *gicon;
-
-  GString *filename;
-  gchar **iconnames;
-
-  filename = g_string_new (NULL);
-
-  if (state == UP_DEVICE_STATE_FULLY_CHARGED)
-    {
-      g_string_append (filename, "battery-charged;");
-      g_string_append (filename, "battery-full-charged-symbolic;");
-      g_string_append (filename, "battery-full-charged;");
-      g_string_append (filename, "gpm-battery-charged;");
-      g_string_append (filename, "gpm-battery-100-charging;");
-    }
-  else if (state == UP_DEVICE_STATE_CHARGING)
-    {
-      g_string_append (filename, "battery-000-charging;");
-      g_string_append (filename, "battery-caution-charging-symbolic;");
-      g_string_append (filename, "battery-caution-charging;");
-      g_string_append (filename, "gpm-battery-000-charging;");
-    }
-  else if (state == UP_DEVICE_STATE_DISCHARGING)
-    {
-      const gchar *percentage = get_icon_percentage_for_status (suffix_str);
-      g_string_append_printf (filename, "battery-%s;", suffix_str);
-      g_string_append_printf (filename, "battery-%s-symbolic;", suffix_str);
-      g_string_append_printf (filename, "battery-%s;", percentage);
-      g_string_append_printf (filename, "gpm-battery-%s;", percentage);
-    }
-
-  iconnames = g_strsplit (filename->str, ";", -1);
-  gicon = g_themed_icon_new_from_names (iconnames, -1);
-
-  g_strfreev (iconnames);
-  g_string_free (filename, TRUE);
-
-  return gicon;
-}
-
-static GIcon*
-get_device_icon (UpDeviceKind   kind,
-                 UpDeviceState  state,
-                 guint64        time_sec,
-                 const gchar   *device_icon)
-{
-  GIcon *gicon = NULL;
-
-  if (kind == UP_DEVICE_KIND_BATTERY &&
-      (state == UP_DEVICE_STATE_FULLY_CHARGED ||
-       state == UP_DEVICE_STATE_CHARGING ||
-       state == UP_DEVICE_STATE_DISCHARGING))
-    {
-      if (state == UP_DEVICE_STATE_FULLY_CHARGED ||
-          state == UP_DEVICE_STATE_CHARGING)
-        {
-          gicon = build_battery_icon (state, NULL);
-        }
-      else if (state == UP_DEVICE_STATE_DISCHARGING)
-        {
-          if ((time_sec > 60 * 30) && /* more than 30 minutes left */
-              (g_strrstr (device_icon, "000") ||
-               g_strrstr (device_icon, "020") ||
-               g_strrstr (device_icon, "caution"))) /* the icon is red */
-            {
-              gicon = build_battery_icon (state, "low");
-            }
-        }
-    }
-
-  if (gicon == NULL)
-    gicon = g_icon_new_for_string (device_icon, NULL);
-
-  return gicon;
-}
-
-
 static gboolean
 menu_add_device (GtkMenu * menu, const IndicatorPowerDevice * device)
 {
@@ -538,7 +443,7 @@ menu_add_device (GtkMenu * menu, const IndicatorPowerDevice * device)
     GtkWidget *item;
     GtkWidget *details_label;
     GtkWidget *grid;
-    GIcon *device_gicons;
+    GIcon *device_gicon;
     const gchar *device_name;
     gchar *short_details = NULL;
     gchar *details = NULL;
@@ -546,14 +451,12 @@ menu_add_device (GtkMenu * menu, const IndicatorPowerDevice * device)
     AtkObject *atk_object;
     const time_t time = indicator_power_device_get_time (device);
     const UpDeviceState state = indicator_power_device_get_state (device);
-    const gchar * device_icon = indicator_power_device_get_icon (device);
     const gdouble percentage  = indicator_power_device_get_percentage (device);
 
     /* Process the data */
-    device_gicons = get_device_icon (kind, state, time, device_icon);
-    icon = gtk_image_new_from_gicon (device_gicons,
-                                     GTK_ICON_SIZE_SMALL_TOOLBAR);
-    g_clear_object (&device_gicons);
+    device_gicon = indicator_power_device_get_gicon (device);
+    icon = gtk_image_new_from_gicon (device_gicon, GTK_ICON_SIZE_SMALL_TOOLBAR);
+    g_clear_object (&device_gicon);
 
     device_name = device_kind_to_localised_string (kind);
 
@@ -727,7 +630,7 @@ get_primary_device (GSList * devices)
 static void
 put_primary_device (IndicatorPower *self, IndicatorPowerDevice *device)
 {
-  GIcon *device_gicons;
+  GIcon *device_gicon;
   gchar *short_details = NULL;
   gchar *details = NULL;
   gchar *accessible_name = NULL;
@@ -736,15 +639,12 @@ put_primary_device (IndicatorPower *self, IndicatorPowerDevice *device)
   const time_t time = indicator_power_device_get_time (device);
   const UpDeviceKind kind = indicator_power_device_get_kind (device);
   const UpDeviceState state = indicator_power_device_get_state (device);
-  const gchar * device_icon = indicator_power_device_get_icon (device);
   const gdouble percentage  = indicator_power_device_get_percentage (device);
 
   /* set icon */
-  device_gicons = get_device_icon (kind, state, time, device_icon);
-  gtk_image_set_from_gicon (priv->status_image,
-                            device_gicons,
-                            GTK_ICON_SIZE_LARGE_TOOLBAR);
-  g_clear_object (&device_gicons);
+  device_gicon = indicator_power_device_get_gicon (device);
+  gtk_image_set_from_gicon (priv->status_image, device_gicon, GTK_ICON_SIZE_LARGE_TOOLBAR);
+  g_clear_object (&device_gicon);
   gtk_widget_show (GTK_WIDGET (priv->status_image));
 
 
diff --git a/tests/test-device.cc b/tests/test-device.cc
index 132d762..b11188c 100644
--- a/tests/test-device.cc
+++ b/tests/test-device.cc
@@ -34,11 +34,37 @@ namespace
   }
 }
 
+class DeviceTest : public ::testing::Test
+{
+  protected:
+
+    virtual void SetUp()
+    {
+      ensure_glib_initialized ();
+    }
+
+    virtual void TearDown()
+    {
+    }
+
+  protected:
+
+    void check_icon_names (const IndicatorPowerDevice * device, const char * expected)
+    {
+      char ** names = indicator_power_device_get_icon_names (device);
+      char * str = g_strjoinv (";", names);
+      ASSERT_STREQ (expected, str);
+      g_free (str);
+      g_strfreev (names);
+    }
+};
+
+
 /***
 ****
 ***/
 
-TEST(DeviceTest, GObjectNew)
+TEST_F(DeviceTest, GObjectNew)
 {
   ensure_glib_initialized ();
 
@@ -49,7 +75,7 @@ TEST(DeviceTest, GObjectNew)
   g_object_unref (o);
 }
 
-TEST(DeviceTest, Properties)
+TEST_F(DeviceTest, Properties)
 {
   int i;
   gdouble d;
@@ -108,7 +134,7 @@ TEST(DeviceTest, Properties)
   g_object_unref (o);
 }
 
-TEST(DeviceTest, New)
+TEST_F(DeviceTest, New)
 {
   ensure_glib_initialized ();
 
@@ -131,7 +157,7 @@ TEST(DeviceTest, New)
   g_object_unref (device);
 }
 
-TEST(DeviceTest, NewFromVariant)
+TEST_F(DeviceTest, NewFromVariant)
 {
   ensure_glib_initialized ();
 
@@ -158,7 +184,7 @@ TEST(DeviceTest, NewFromVariant)
   g_variant_unref (variant);
 }
 
-TEST(DeviceTest, BadAccessors)
+TEST_F(DeviceTest, BadAccessors)
 {
   ensure_glib_initialized ();
 
@@ -181,3 +207,168 @@ TEST(DeviceTest, BadAccessors)
   indicator_power_device_get_object_path (device);
   g_object_unref (device);
 }
+
+/***
+****
+***/
+
+TEST_F(DeviceTest, IconNames)
+{
+  IndicatorPowerDevice * device = INDICATOR_POWER_DEVICE (g_object_new (INDICATOR_POWER_DEVICE_TYPE, NULL));
+  GObject * o = G_OBJECT(device);
+
+  /* power */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_LINE_POWER,
+                   NULL);
+  check_icon_names (device, "ac-adapter-symbolic;"
+                            "ac-adapter;");
+
+  /* monitor */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_MONITOR,
+                   NULL);
+  check_icon_names (device, "gpm-monitor-symbolic;"
+                            "gpm-monitor;");
+
+  /* empty battery */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_EMPTY,
+                   NULL);
+  check_icon_names (device, "battery-empty-symbolic;"
+                            "gpm-battery-empty;"
+                            "gpm-battery-000;"
+                            "battery-empty;");
+
+  /* charged battery */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_FULLY_CHARGED,
+                   NULL);
+  check_icon_names (device, "battery-full-charged-symbolic;battery-full-charging-symbolic;"
+                            "gpm-battery-full;gpm-battery-100;"
+                            "battery-full-charged;battery-full-charging;");
+
+  /* charging battery, 95% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_CHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 95.0,
+                   NULL);
+  check_icon_names (device, "battery-caution-charging-symbolic;"
+                            "gpm-battery-000-charging;"
+                            "battery-caution-charging;");
+
+  /* charging battery, 85% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_CHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 85.0,
+                   NULL);
+  check_icon_names (device, "battery-caution-charging-symbolic;"
+                            "gpm-battery-000-charging;"
+                            "battery-caution-charging;");
+
+  /* charging battery, 50% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_CHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 50.0,
+                   NULL);
+  check_icon_names (device, "battery-caution-charging-symbolic;"
+                            "gpm-battery-000-charging;"
+                            "battery-caution-charging;");
+
+  /* charging battery, 25% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_CHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 25.0,
+                   NULL);
+  check_icon_names (device, "battery-caution-charging-symbolic;"
+                            "gpm-battery-000-charging;"
+                            "battery-caution-charging;");
+
+  /* charging battery, 5% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_CHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 5.0,
+                   NULL);
+  check_icon_names (device, "battery-caution-charging-symbolic;"
+                            "gpm-battery-000-charging;"
+                            "battery-caution-charging;");
+
+
+  /* discharging battery, 95% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 95.0,
+                   NULL);
+  check_icon_names (device, "battery-full-symbolic;"
+                            "gpm-battery-100;"
+                            "battery-full;");
+
+  /* discharging battery, 85% */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 85.0,
+                   NULL);
+  check_icon_names (device, "battery-full-symbolic;"
+                            "gpm-battery-080;"
+                            "battery-full;");
+
+  /* discharging battery, 50% -- 1 hour left */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 50.0,
+                   INDICATOR_POWER_DEVICE_TIME, (guint64)(60*60),
+                   NULL);
+  check_icon_names (device, "battery-good-symbolic;"
+                            "gpm-battery-060;"
+                            "battery-good;");
+
+  /* discharging battery, 25% -- 1 hour left */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 25.0,
+                   INDICATOR_POWER_DEVICE_TIME, (guint64)(60*60),
+                   NULL);
+  check_icon_names (device, "battery-good-symbolic;"
+                            "gpm-battery-040;"
+                            "battery-good;");
+
+  /* discharging battery, 25% -- 15 minutes left */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 25.0,
+                   INDICATOR_POWER_DEVICE_TIME, (guint64)(60*15),
+                   NULL);
+  check_icon_names (device, "battery-low-symbolic;"
+                            "gpm-battery-020;"
+                            "battery-low;");
+
+  /* discharging battery, 5% -- 1 hour left */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 5.0,
+                   INDICATOR_POWER_DEVICE_TIME, (guint64)(60*60),
+                   NULL);
+  check_icon_names (device, "battery-good-symbolic;"
+                            "gpm-battery-040;"
+                            "battery-good;");
+
+  /* discharging battery, 5% -- 15 minutes left */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_DISCHARGING,
+                   INDICATOR_POWER_DEVICE_PERCENTAGE, 5.0,
+                   INDICATOR_POWER_DEVICE_TIME, (guint64)(60*15),
+                   NULL);
+  check_icon_names (device, "battery-caution-symbolic;"
+                            "gpm-battery-000;"
+                            "battery-caution;");
+
+  /* state unknown */
+  g_object_set (o, INDICATOR_POWER_DEVICE_KIND, UP_DEVICE_KIND_BATTERY,
+                   INDICATOR_POWER_DEVICE_STATE, UP_DEVICE_STATE_UNKNOWN, 
+                   NULL);
+  check_icon_names (device, "battery-missing-symbolic;"
+                            "gpm-battery-missing;"
+                            "battery-missing;");
+
+  /* cleanup */
+  g_object_unref(o);
+}
+