From 9af35685d7fb590ca4c43131cc129632ec8e6f98 Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles.kerr@canonical.com>
Date: Wed, 19 Mar 2014 10:46:44 -0500
Subject: copy newer GLibTestFixture wait_for_signal() and wait_msec() methods
 from indicator-datetime's tests to fix noisy g_warnings

---
 tests/gtest-dbus-fixture.h | 55 +++++++++++++++++++++++++++-------------------
 1 file changed, 32 insertions(+), 23 deletions(-)

(limited to 'tests')

diff --git a/tests/gtest-dbus-fixture.h b/tests/gtest-dbus-fixture.h
index e6cd9c7..dedb727 100644
--- a/tests/gtest-dbus-fixture.h
+++ b/tests/gtest-dbus-fixture.h
@@ -54,13 +54,6 @@ class GTestDBusFixture : public ::testing::Test
       g_main_loop_quit (self->loop);
     }
 
-    static gboolean
-    wait_for_signal__timeout (gpointer name)
-    {
-      g_error ("%s: timed out waiting for signal '%s'", G_STRLOC, (char*)name);
-      return G_SOURCE_REMOVE;
-    }
-
   protected:
 
     virtual void SetUp ()
@@ -103,29 +96,45 @@ class GTestDBusFixture : public ::testing::Test
       g_clear_pointer (&loop, g_main_loop_unref);
     }
 
-    /* convenience func to loop while waiting for a GObject's signal */
-    void wait_for_signal (gpointer o, const gchar * signal)
+  private:
+
+    static gboolean
+    wait_for_signal__timeout(gpointer name)
     {
-      const int timeout_seconds = 5; // arbitrary
+      g_error("%s: timed out waiting for signal '%s'", G_STRLOC, (char*)name);
+      return G_SOURCE_REMOVE;
+    }
 
+    static gboolean
+    wait_msec__timeout(gpointer loop)
+    {
+      g_main_loop_quit(static_cast<GMainLoop*>(loop));
+      return G_SOURCE_CONTINUE;
+    }
+
+  protected:
+
+    /* convenience func to loop while waiting for a GObject's signal */
+    void wait_for_signal(gpointer o, const gchar * signal, const int timeout_seconds=5)
+    {
       // wait for the signal or for timeout, whichever comes first
-      guint handler_id = g_signal_connect_swapped (o, signal,
-                                                   G_CALLBACK(g_main_loop_quit),
-                                                   loop);
-      gulong timeout_id = g_timeout_add_seconds (timeout_seconds,
-                                                 wait_for_signal__timeout,
-                                                 loop);
-      g_main_loop_run (loop);
-      g_source_remove (timeout_id);
-      g_signal_handler_disconnect (o, handler_id);
+      const auto handler_id = g_signal_connect_swapped(o, signal,
+                                                       G_CALLBACK(g_main_loop_quit),
+                                                       loop);
+      const auto timeout_id = g_timeout_add_seconds(timeout_seconds,
+                                                    wait_for_signal__timeout,
+                                                    loop);
+      g_main_loop_run(loop);
+      g_source_remove(timeout_id);
+      g_signal_handler_disconnect(o, handler_id);
     }
 
     /* convenience func to loop for N msec */
-    void wait_msec (int msec)
+    void wait_msec(int msec=50)
     {
-      guint id = g_timeout_add (msec, (GSourceFunc)g_main_loop_quit, loop);
-      g_main_loop_run (loop);
-      g_source_remove (id);
+      const auto id = g_timeout_add(msec, wait_msec__timeout, loop);
+      g_main_loop_run(loop);
+      g_source_remove(id);
     }
 
     GMainLoop * loop;
-- 
cgit v1.2.3


From e5f30358325683fe338137fc0cc71541e73c81be Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles.kerr@canonical.com>
Date: Wed, 19 Mar 2014 10:48:12 -0500
Subject: sync tests to the red-icon-removal change in r436

---
 tests/test-service.cc | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

(limited to 'tests')

diff --git a/tests/test-service.cc b/tests/test-service.cc
index f14fe61..0815d19 100644
--- a/tests/test-service.cc
+++ b/tests/test-service.cc
@@ -616,7 +616,7 @@ TEST_F (ServiceTest, OnlineAccountError)
   check_last_command_is ("online-accounts");
 
   // check that the header's icon and a11y adjusted to the error state
-  check_header ("", "system-devices-panel-alert", "System (Attention Required)");
+  check_header ("", "system-devices-panel", "System");
 
   // cleanup
   g_settings_reset (mock_settings, error_key);
@@ -756,12 +756,7 @@ TEST_F (ServiceTest, User)
   g_settings_set_boolean (indicator_settings, show_name_key, true);
   wait_for_signal (action_group, "action-state-changed");
   check_header ("Third Doctor", "system-devices-panel", "System, Third Doctor");
-  g_settings_set_boolean (mock_settings, error_key, true);
-  wait_for_signal (action_group, "action-state-changed");
-  check_header ("Third Doctor", "system-devices-panel-alert", "System, Third Doctor (Attention Required)");
-  g_settings_reset (mock_settings, error_key);
   g_settings_reset (indicator_settings, show_name_key);
-  wait_for_menu_resync ();
 
   // try setting the max user count to 2...
   // since troughton has the fewest logins, he should get culled
-- 
cgit v1.2.3