From 271b0fbf8b14a4f7a8f47de0e3a8751bd50676c3 Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles.kerr@canonical.com>
Date: Thu, 30 Jan 2014 18:40:13 -0600
Subject: copyediting: remove dead '#if 0''ed code

---
 tests/test-timezone-geoclue.cpp | 59 ++++-------------------------------------
 1 file changed, 5 insertions(+), 54 deletions(-)

(limited to 'tests/test-timezone-geoclue.cpp')

diff --git a/tests/test-timezone-geoclue.cpp b/tests/test-timezone-geoclue.cpp
index 4bf08a7..3cc1393 100644
--- a/tests/test-timezone-geoclue.cpp
+++ b/tests/test-timezone-geoclue.cpp
@@ -21,64 +21,17 @@
 
 #include <datetime/timezone-geoclue.h>
 
-//#include <libdbustest/dbus-test.h>
-
 using unity::indicator::datetime::GeoclueTimezone;
 
-/***
-****
-***/
-
-class TimezoneGeoclueFixture : public GeoclueFixture
-{
-};
-
-#if 0
-namespace
+// This test looks small because the interesting
+// work is all happening in GeoclueFixture...
+TEST_F(GeoclueFixture, ChangeDetected)
 {
-  struct EmitAddressChangedData
-  {
-    DbusTestDbusMock * mock = nullptr;
-    DbusTestDbusMockObject * obj_client = nullptr;
-    std::string timezone;
-    EmitAddressChangedData(DbusTestDbusMock * mock_,
-                           DbusTestDbusMockObject * obj_client_,
-                           const std::string& timezone_): mock(mock_), obj_client(obj_client_), timezone(timezone_) {}
-  };
-
-  gboolean emit_address_changed_idle(gpointer gdata)
-  {
-    auto data = static_cast<EmitAddressChangedData*>(gdata);
-
-    GError * error = nullptr;
-    dbus_test_dbus_mock_object_emit_signal(data->mock, data->obj_client,
-                                            "org.freedesktop.Geoclue.Address",
-                                            "AddressChanged",
-                                            G_VARIANT_TYPE("(ia{ss}(idd))"),
-                                            g_variant_new_parsed("(1385238033, {'timezone': 'America/Chicago'}, (3, 0.0, 0.0))"),
-                                            &error);
-    if (error)
-      {
-        g_warning("%s: %s", G_STRFUNC, error->message);
-        g_error_free(error);
-      }
-
-    delete data;
-    return G_SOURCE_REMOVE;
-  }
-}
-#endif
-
-TEST_F(TimezoneGeoclueFixture, ChangeDetected)
-{
-//  const std::string timezone_1 = "America/Denver";
-  const std::string timezone_2 = "America/Chicago";
-
   GeoclueTimezone tz;
   wait_msec(500); // wait for the bus to get set up
   EXPECT_EQ(timezone_1, tz.timezone.get());
 
-  // start listening for a timezone change, then change the timezone
+  // Start listening for a timezone change, then change the timezone.
 
   bool changed = false;
   auto connection = tz.timezone.changed().connect(
@@ -88,10 +41,8 @@ TEST_F(TimezoneGeoclueFixture, ChangeDetected)
           g_main_loop_quit(loop);
         });
 
+  const std::string timezone_2 = "America/Chicago";
   setGeoclueTimezoneOnIdle(timezone_2);
-  //g_timeout_add(50, emit_address_changed_idle, new EmitAddressChangedData(mock, obj_client, timezone_2.c_str()));
   g_main_loop_run(loop);
   EXPECT_EQ(timezone_2, tz.timezone.get());
 }
-
-
-- 
cgit v1.2.3