From c4530fde870f6c7838d3d5dee2a63a885d8f896a Mon Sep 17 00:00:00 2001
From: Ted Gould <ted@gould.cx>
Date: Tue, 4 Nov 2014 18:21:05 -0600
Subject: Pulling out waiting for items into a helper function

---
 tests/indicator-fixture.h | 45 ++++++++++++++++++++++++++-------------------
 1 file changed, 26 insertions(+), 19 deletions(-)

(limited to 'tests/indicator-fixture.h')

diff --git a/tests/indicator-fixture.h b/tests/indicator-fixture.h
index 75ad0f6..f2039f7 100644
--- a/tests/indicator-fixture.h
+++ b/tests/indicator-fixture.h
@@ -30,7 +30,6 @@ class IndicatorFixture : public ::testing::Test
 	private:
 		std::string _indicatorPath;
 		std::string _indicatorAddress;
-		GMainLoop * _loop;
 		std::shared_ptr<GMenuModel>  _menu;
 		DbusTestService * _test_service;
 		DbusTestTask * _test_indicator;
@@ -44,7 +43,6 @@ class IndicatorFixture : public ::testing::Test
 				const std::string& addr)
 			: _indicatorPath(path)
 			, _indicatorAddress(addr)
-			, _loop(nullptr)
 			, _menu(nullptr)
 			, _session(nullptr)
 		{
@@ -54,7 +52,6 @@ class IndicatorFixture : public ::testing::Test
 	protected:
 		virtual void SetUp() override
 		{
-			_loop = g_main_loop_new(nullptr, FALSE);
 
 			_test_service = dbus_test_service_new(nullptr);
 
@@ -93,9 +90,6 @@ class IndicatorFixture : public ::testing::Test
 				g_dbus_connection_close_sync(_session, nullptr, nullptr);
 			}
 			g_clear_object(&_session);
-
-			/* Dropping temp loop */
-			g_main_loop_unref(_loop);
 		}
 
 	private:
@@ -110,27 +104,40 @@ class IndicatorFixture : public ::testing::Test
 			return G_SOURCE_CONTINUE;
 		}
 
-	protected:
-		void setMenu (const std::string& path) {
-			_menu.reset();
+		void menuWaitForItems (const std::shared_ptr<GMenuModel>& menu) {
+			auto count = g_menu_model_get_n_items(menu.get());
+			
+			if (count != 0)
+				return;
 
-			g_debug("Getting Menu: %s:%s", _indicatorAddress.c_str(), path.c_str());
-			_menu = std::shared_ptr<GMenuModel>(G_MENU_MODEL(g_dbus_menu_model_get(_session, _indicatorAddress.c_str(), path.c_str())), [](GMenuModel * modelptr) {
-				g_clear_object(&modelptr);
-			});
+			auto loop = g_main_loop_new(nullptr, FALSE);
 
 			/* Our two exit criteria */
-			gulong signal = g_signal_connect(G_OBJECT(_menu.get()), "items-changed", G_CALLBACK(_changed_quit), _loop);
-			guint timer = g_timeout_add_seconds(5, _loop_quit, _loop);
+			gulong signal = g_signal_connect(G_OBJECT(menu.get()), "items-changed", G_CALLBACK(_changed_quit), loop);
+			guint timer = g_timeout_add_seconds(5, _loop_quit, loop);
 
-			g_menu_model_get_n_items(_menu.get());
+			g_menu_model_get_n_items(menu.get());
 
 			/* Wait for sync */
-			g_main_loop_run(_loop);
+			g_main_loop_run(loop);
 
 			/* Clean up */
 			g_source_remove(timer);
-			g_signal_handler_disconnect(G_OBJECT(_menu.get()), signal);
+			g_signal_handler_disconnect(G_OBJECT(menu.get()), signal);
+
+			g_main_loop_unref(loop);
+		}
+
+	protected:
+		void setMenu (const std::string& path) {
+			_menu.reset();
+
+			g_debug("Getting Menu: %s:%s", _indicatorAddress.c_str(), path.c_str());
+			_menu = std::shared_ptr<GMenuModel>(G_MENU_MODEL(g_dbus_menu_model_get(_session, _indicatorAddress.c_str(), path.c_str())), [](GMenuModel * modelptr) {
+				g_clear_object(&modelptr);
+			});
+
+			menuWaitForItems(_menu);
 		}
 
 		void expectActionExists (const std::string& name) {
@@ -176,7 +183,7 @@ class IndicatorFixture : public ::testing::Test
 			if (submenu == nullptr)
 				return nullptr;
 
-			g_menu_model_get_n_items(submenu.get());
+			menuWaitForItems(submenu);
 
 			return getMenuAttributeRecurse(menuLocation + 1, menuEnd, attribute, value, submenu);
 		}
-- 
cgit v1.2.3