Merge lp:~charlesk/indicator-datetime/lp-1560960-use-dbusmock-timedated-template into lp:indicator-datetime/15.10

[Charles Kerr]

Use dbusmock's timedated template in our tests

Minor patch on top of https://code.launchpad.net/~charlesk/indicator-datetime/always-get-initial-tzid-from-timedate1/+merge/291421 .

That branch added a test fixture which uses dbusmock's timedated template. This branch promotes that up to a shared test header where it can be reused by test-live-actions, and removes test-live-actions' previous handrolled timedated mock.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:448
http://jenkins.qa.ubuntu.com/job/indicator-datetime-ci/372/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-wily-amd64-ci/62
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-wily-armhf-ci/62
        deb: http://jenkins.qa.ubuntu.com/job/indicator-datetime-wily-armhf-ci/62/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-datetime-ci/372/rebuild

review: Approve (continuous-integration)

[Renato Araujo Oliveira Filho (renatofilho)]

looks good.

review: Approve

[lp:~charlesk/indicator-datetime/lp-1560960-use-dbusmock-timedated-template]

449. By Charles Kerr on 2016-04-12

sync with lp:~charlesk/indicator-datetime/always-get-initial-tzid-from-timedate1/

[Charles Kerr (charlesk)]

re-approving because the r449 was already approved in prerequisite branch

[Charles Kerr (charlesk)]

review: Approve

=== modified file 'include/datetime/dbus-shared.h'
--- include/datetime/dbus-shared.h	2016-04-12 17:27:17 +0000
+++ include/datetime/dbus-shared.h	2016-04-12 17:27:17 +0000
@@ -28,10 +28,6 @@
 #define BUS_POWERD_PATH      "/com/canonical/powerd"
 #define BUS_POWERD_INTERFACE "com.canonical.powerd"
 
-namespace unity {
-namespace indicator {
-namespace datetime {
-
 namespace Bus
 {
     namespace Timedate1
@@ -67,8 +63,4 @@
     }
 }
 
-} // namespace datetime
-} // namespace indicator
-} // namespace unity
-
 #endif /* INDICATOR_DATETIME_DBUS_SHARED_H */
 
=== modified file 'src/actions-live.cpp'
--- src/actions-live.cpp	2016-04-05 21:00:09 +0000
+++ src/actions-live.cpp	2016-04-12 17:27:17 +0000
@@ -17,6 +17,7 @@
  *   Charles Kerr <[e-mail]>
  */
 
+#include <datetime/dbus-shared.h>
 #include <datetime/actions-live.h>
 
 #include <url-dispatcher.h>
@@ -252,7 +253,7 @@
   else
     {
       g_dbus_proxy_call(proxy,
-                        "SetTimezone",
+                        Bus::Timedate1::Methods::SET_TIMEZONE,
                         g_variant_new ("(sb)", data->tzid.c_str(), TRUE),
                         G_DBUS_CALL_FLAGS_NONE,
                         -1,
@@ -280,9 +281,9 @@
     g_dbus_proxy_new_for_bus (G_BUS_TYPE_SYSTEM,
                               G_DBUS_PROXY_FLAGS_NONE,
                               nullptr,
-                              "org.freedesktop.timedate1",
-                              "/org/freedesktop/timedate1",
-                              "org.freedesktop.timedate1",
+                              Bus::Timedate1::BUSNAME,
+                              Bus::Timedate1::ADDR,
+                              Bus::Timedate1::IFACE,
                               nullptr,
                               on_datetime1_proxy_ready,
                               data);
 
=== modified file 'tests/glib-fixture.h'
--- tests/glib-fixture.h	2016-02-10 20:47:39 +0000
+++ tests/glib-fixture.h	2016-04-12 17:27:17 +0000
@@ -125,10 +125,11 @@
       }
     }
 
-    bool wait_for_name_owned(GDBusConnection* connection,
-                             const gchar* name,
-                             guint timeout_msec=1000,
-                             GBusNameWatcherFlags flags=G_BUS_NAME_WATCHER_FLAGS_AUTO_START)
+    bool wait_for_name_owned(
+        GDBusConnection* connection,
+        const gchar* name,
+        guint timeout_msec=1000,
+        GBusNameWatcherFlags flags=G_BUS_NAME_WATCHER_FLAGS_AUTO_START)
     {
       struct Data {
         GMainLoop* loop = nullptr;
@@ -139,8 +140,7 @@
       auto on_name_appeared = [](GDBusConnection* /*connection*/,
                                  const gchar* /*name_*/,
                                  const gchar* name_owner,
-                                 gpointer gdata)
-      {
+                                 gpointer gdata){
         if (name_owner == nullptr)
           return;
         auto tmp = static_cast<Data*>(gdata);
@@ -150,13 +150,16 @@
 
       const auto timeout_id = g_timeout_add(timeout_msec, wait_msec__timeout, loop);
       data.loop = loop;
-      const auto watch_id = g_bus_watch_name_on_connection(connection,
-                                                           name,
-                                                           flags,
-                                                           on_name_appeared,
-                                                           nullptr, /* name_vanished */
-                                                           &data,
-                                                           nullptr); /* user_data_free_func */
+      const auto watch_id = g_bus_watch_name_on_connection(
+          connection,
+          name,
+          flags,
+          on_name_appeared,
+          nullptr, // name_vanished
+          &data,
+          nullptr // user_data_free_func
+      );
+
       g_main_loop_run(loop);
 
       g_bus_unwatch_name(watch_id);
 
=== modified file 'tests/test-live-actions.cpp'
--- tests/test-live-actions.cpp	2016-03-21 17:32:39 +0000
+++ tests/test-live-actions.cpp	2016-04-12 17:27:17 +0000
@@ -17,18 +17,75 @@
  *   Charles Kerr <[e-mail]>
  */
 
+#include "state-mock.h"
 #include "timedated-fixture.h"
 
+#include <datetime/actions-live.h>
+
+using namespace unity::indicator::datetime;
+
+class MockLiveActions: public LiveActions
+{
+public:
+    std::string last_cmd;
+    std::string last_url;
+    explicit MockLiveActions(const std::shared_ptr<State>& state_in): LiveActions(state_in) {}
+    ~MockLiveActions() {}
+
+protected:
+    void dispatch_url(const std::string& url) override { last_url = url; }
+    void execute_command(const std::string& cmd) override { last_cmd = cmd; }
+};
+
+class TestLiveActionsFixture: public TimedatedFixture
+{
+private:
+
+    using super = TimedatedFixture;
+
+protected:
+
+    std::shared_ptr<MockState> m_mock_state;
+    std::shared_ptr<State> m_state;
+    std::shared_ptr<MockLiveActions> m_live_actions;
+    std::shared_ptr<Actions> m_actions;
+
+    void SetUp() override
+    {
+        super::SetUp();
+
+        // create the State and Actions
+        m_mock_state.reset(new MockState);
+        m_mock_state->settings.reset(new Settings);
+        m_state = std::dynamic_pointer_cast<State>(m_mock_state);
+        m_live_actions.reset(new MockLiveActions(m_state));
+        m_actions = std::dynamic_pointer_cast<Actions>(m_live_actions);
+
+        // start the timedate1 dbusmock
+        start_timedate1("Etc/Utc");
+    }
+
+    void TearDown() override
+    {
+        m_actions.reset();
+        m_live_actions.reset();
+        m_state.reset();
+        m_mock_state.reset();
+
+        super::TearDown();
+    }
+};
+
 /***
 ****
 ***/
 
-TEST_F(TimedateFixture, HelloWorld)
+TEST_F(TestLiveActionsFixture, HelloWorld)
 {
     EXPECT_TRUE(true);
 }
 
-TEST_F(TimedateFixture, SetLocation)
+TEST_F(TestLiveActionsFixture, SetLocation)
 {
     const std::string tzid = "America/Chicago";
     const std::string name = "Oklahoma City";
@@ -36,30 +93,31 @@
 
     EXPECT_NE(expected, m_state->settings->timezone_name.get());
 
+    std::string new_name;
+    m_state->settings->timezone_name.changed().connect(
+        [&new_name](const std::string& n){new_name = n;}
+    );
+
     m_actions->set_location(tzid, name);
-    m_state->settings->timezone_name.changed().connect(
-          [this](const std::string&){
-            g_main_loop_quit(loop);
-            });
-    g_main_loop_run(loop);
-    EXPECT_EQ(attempted_tzid, tzid);
-    wait_msec();
 
+    EXPECT_TRUE(wait_for([&new_name](){return !new_name.empty();}));
+    EXPECT_EQ(expected, new_name);
     EXPECT_EQ(expected, m_state->settings->timezone_name.get());
+    EXPECT_EQ(tzid, get_timedate1_timezone());
 }
 
 /***
 ****
 ***/
 
-TEST_F(TimedateFixture, DesktopOpenAlarmApp)
+TEST_F(TestLiveActionsFixture, DesktopOpenAlarmApp)
 {
     m_actions->desktop_open_alarm_app();
     const std::string expected = "evolution -c calendar";
     EXPECT_EQ(expected, m_live_actions->last_cmd);
 }
 
-TEST_F(TimedateFixture, DesktopOpenAppointment)
+TEST_F(TestLiveActionsFixture, DesktopOpenAppointment)
 {
     Appointment a;
     a.uid = "some-uid";
@@ -69,14 +127,14 @@
     EXPECT_NE(m_live_actions->last_cmd.find(expected_substr), std::string::npos);
 }
 
-TEST_F(TimedateFixture, DesktopOpenCalendarApp)
+TEST_F(TestLiveActionsFixture, DesktopOpenCalendarApp)
 {
     m_actions->desktop_open_calendar_app(DateTime::NowLocal());
     const std::string expected_substr = "evolution \"calendar:///?startdate=";
     EXPECT_NE(m_live_actions->last_cmd.find(expected_substr), std::string::npos);
 }
 
-TEST_F(TimedateFixture, DesktopOpenSettingsApp)
+TEST_F(TestLiveActionsFixture, DesktopOpenSettingsApp)
 {
     m_actions->desktop_open_settings_app();
     const std::string expected_substr = "control-center";
@@ -92,13 +150,13 @@
     const std::string clock_app_url = "appid://com.ubuntu.clock/clock/current-user-version";
 }
 
-TEST_F(TimedateFixture, PhoneOpenAlarmApp)
+TEST_F(TestLiveActionsFixture, PhoneOpenAlarmApp)
 {
     m_actions->phone_open_alarm_app();
     EXPECT_EQ(clock_app_url, m_live_actions->last_url);
 }
 
-TEST_F(TimedateFixture, PhoneOpenAppointment)
+TEST_F(TestLiveActionsFixture, PhoneOpenAppointment)
 {
     Appointment a;
 
@@ -116,7 +174,7 @@
     EXPECT_EQ(clock_app_url, m_live_actions->last_url);
 }
 
-TEST_F(TimedateFixture, PhoneOpenCalendarApp)
+TEST_F(TestLiveActionsFixture, PhoneOpenCalendarApp)
 {
     auto now = DateTime::NowLocal();
     m_actions->phone_open_calendar_app(now);
@@ -125,7 +183,7 @@
 }
 
 
-TEST_F(TimedateFixture, PhoneOpenSettingsApp)
+TEST_F(TestLiveActionsFixture, PhoneOpenSettingsApp)
 {
     m_actions->phone_open_settings_app();
     const std::string expected = "settings:///system/time-date";
@@ -136,7 +194,7 @@
 ****
 ***/
 
-TEST_F(TimedateFixture, CalendarState)
+TEST_F(TestLiveActionsFixture, CalendarState)
 {
     // init the clock
     auto now = DateTime::Local(2014, 1, 1, 0, 0, 0);
 
=== modified file 'tests/test-timezone-timedated.cpp'
--- tests/test-timezone-timedated.cpp	2016-04-12 17:27:17 +0000
+++ tests/test-timezone-timedated.cpp	2016-04-12 17:27:17 +0000
@@ -18,119 +18,26 @@
  *   Ted Gould <[e-mail]>
  */
 
-#include "glib-fixture.h"
+#include "timedated-fixture.h"
 
-#include <datetime/dbus-shared.h>
 #include <datetime/timezone-timedated.h>
 
-#include <gio/gio.h>
-
-
 using namespace unity::indicator::datetime;
 
-
-struct Timedate1Fixture: public GlibFixture
-{
-private:
-
-    typedef GlibFixture super;
-
-protected:
-
-    GDBusConnection* m_bus {};
-    GTestDBus* m_test_bus {};
-
-    void SetUp() override
-    {
-        super::SetUp();
-
-        // use a fake bus
-        m_test_bus = g_test_dbus_new(G_TEST_DBUS_NONE);
-        g_test_dbus_up(m_test_bus);
-        const char * address = g_test_dbus_get_bus_address(m_test_bus);
-        g_setenv("DBUS_SYSTEM_BUS_ADDRESS", address, true);
-        g_setenv("DBUS_SESSION_BUS_ADDRESS", address, true);
-        g_debug("test_dbus's address is %s", address);
-
-        // get the bus
-        m_bus = g_bus_get_sync(G_BUS_TYPE_SESSION, nullptr, nullptr);
-        g_dbus_connection_set_exit_on_close(m_bus, FALSE);
-    }
-
-    void TearDown() override
-    {
-        // tear down the bus
-        bool bus_finished = false;
-        g_object_weak_ref(
-            G_OBJECT(m_bus),
-            [](gpointer gbus_finished, GObject*){*static_cast<bool*>(gbus_finished) = true;},
-            &bus_finished
-        );
-        g_clear_object(&m_bus);
-        EXPECT_TRUE(wait_for([&bus_finished](){return bus_finished;}));
-
-        // tear down test bus
-        g_clear_object(&m_test_bus);
-
-        super::TearDown();
-    }
-
-    void start_timedate1(const std::string& tzid)
-    {
-        // start dbusmock with the timedated template
-        auto json_parameters = g_strdup_printf("{\"Timezone\": \"%s\"}", tzid.c_str());
-        const gchar* child_argv[] = { "python3", "-m", "dbusmock", "--template", "timedated", "--parameters", json_parameters, nullptr };
-        GError* error = nullptr;
-        g_spawn_async(nullptr, (gchar**)child_argv, nullptr, G_SPAWN_SEARCH_PATH, nullptr, nullptr, nullptr, &error);
-        g_assert_no_error(error);
-        g_free(json_parameters);
-
-        // wait for it to appear on the bus
-        wait_for_name_owned(m_bus, Bus::Timedate1::BUSNAME);
-    }
-
-    bool wait_for_tzid(const std::string& tzid, Timezone& tz)
-    {
-        return wait_for([&tzid, &tz](){return tzid == tz.timezone.get();});
-    }
-
-    void set_timedate1_timezone(const std::string& tzid)
-    {
-        GError* error {};
-        auto v = g_dbus_connection_call_sync(
-            m_bus,
-            Bus::Timedate1::BUSNAME,
-            Bus::Timedate1::ADDR,
-            Bus::Timedate1::IFACE,
-            Bus::Timedate1::Methods::SET_TIMEZONE,
-            g_variant_new("(sb)", tzid.c_str(), FALSE),
-            nullptr,
-            G_DBUS_CALL_FLAGS_NONE,
-            -1,
-            nullptr,
-            &error);
-        g_clear_pointer(&v, g_variant_unref);
-        g_assert_no_error(error);
-    }
-};
-
-#define EXPECT_TZID(expected_tzid, tmp) \
-    EXPECT_TRUE(wait_for_tzid(expected_tzid, tmp)) \
-        << "expected " << expected_tzid \
-        << " got " << tmp.timezone.get();
+using TestTimedatedFixture = TimedatedFixture;
 
 /***
 ****
 ***/
 
-TEST_F(Timedate1Fixture, HelloWorld)
+TEST_F(TestTimedatedFixture, HelloWorld)
 {
 }
 
 /**
  * Test that the tzid is right if timedated isn't available
  */
-TEST_F(Timedate1Fixture, DefaultTimezone)
+TEST_F(TestTimedatedFixture, DefaultTimezone)
 {
     const std::string expected_tzid{"Etc/Utc"};
 
@@ -141,7 +48,7 @@
 /**
  * Test that the tzid is right if timedated shows BEFORE we start
  */
-TEST_F(Timedate1Fixture, Timedate1First)
+TEST_F(TestTimedatedFixture, Timedate1First)
 {
     const std::string expected_tzid{"America/Chicago"};
 
@@ -153,7 +60,7 @@
 /**
  * Test that the tzid is right if timedated shows AFTER we start
  */
-TEST_F(Timedate1Fixture, Timedate1Last)
+TEST_F(TestTimedatedFixture, Timedate1Last)
 {
     const std::string expected_tzid("America/Los_Angeles");
 
@@ -165,7 +72,7 @@
 /**
  * Test that the tzid is right if timedated's property changes
  */
-TEST_F(Timedate1Fixture, TimezoneChange)
+TEST_F(TestTimedatedFixture, TimezoneChange)
 {
     const std::vector<std::string> expected_tzids{"America/Los_Angeles", "America/Chicago", "Etc/Utc"};
 
 
=== modified file 'tests/timedated-fixture.h'
--- tests/timedated-fixture.h	2015-09-03 09:54:20 +0000
+++ tests/timedated-fixture.h	2016-04-12 17:27:17 +0000
@@ -17,285 +17,142 @@
  *   Charles Kerr <[e-mail]>
  */
 
-#ifndef INDICATOR_DATETIME_TESTS_TIMEDATED_FIXTURE_H
-#define INDICATOR_DATETIME_TESTS_TIMEDATED_FIXTURE_H
+#pragma once
 
 #include <datetime/actions-live.h>
 
-#include "state-mock.h"
 #include "glib-fixture.h"
 
-using namespace unity::indicator::datetime;
-
-class MockLiveActions: public LiveActions
-{
-public:
-    std::string last_cmd;
-    std::string last_url;
-    explicit MockLiveActions(const std::shared_ptr<State>& state_in): LiveActions(state_in) {}
-    ~MockLiveActions() {}
-
-protected:
-    void dispatch_url(const std::string& url) override { last_url = url; }
-    void execute_command(const std::string& cmd) override { last_cmd = cmd; }
-};
+#include <datetime/dbus-shared.h>
+#include <datetime/timezone.h>
 
 /***
 ****
 ***/
 
-using namespace unity::indicator::datetime;
-
-class TimedateFixture: public GlibFixture
+struct TimedatedFixture: public GlibFixture
 {
 private:
 
-    typedef GlibFixture super;
-
-    static GVariant * timedate1_get_properties (GDBusConnection * /*connection*/ ,
-                                                const gchar * /*sender*/,
-                                                const gchar * /*object_path*/,
-                                                const gchar * /*interface_name*/,
-                                                const gchar *property_name,
-                                                GError ** /*error*/,
-                                                gpointer gself)
-
-    {
-        auto self = static_cast<TimedateFixture*>(gself);
-        g_debug("get_properties called");
-        if (g_strcmp0(property_name, "Timezone") == 0)
-        {
-            g_debug("timezone requested, giving '%s'",
-                    self->attempted_tzid.c_str());
-            return g_variant_new_string(self->attempted_tzid.c_str());
-        }
-        return nullptr;
-    }
-
-
-    static void on_bus_acquired(GDBusConnection* conn,
-                                const gchar* name,
-                                gpointer gself)
-    {
-        auto self = static_cast<TimedateFixture*>(gself);
-        g_debug("bus acquired: %s, connection is %p", name, conn);
-
-        /* Set up a fake timedated which handles setting and getting the
-         ** timezone
-         */
-        static const GDBusInterfaceVTable vtable = {
-            timedate1_handle_method_call,
-            timedate1_get_properties, /* GetProperty */
-            nullptr, /* SetProperty */
-        };
-
-        self->connection = G_DBUS_CONNECTION(g_object_ref(G_OBJECT(conn)));
-
-        GError* error = nullptr;
-        self->object_register_id = g_dbus_connection_register_object(
-            conn,
-            "/org/freedesktop/timedate1",
-            self->node_info->interfaces[0],
-            &vtable,
-            self,
-            nullptr,
-            &error);
-        g_assert_no_error(error);
-    }
-
-    static void on_name_acquired(GDBusConnection* conn,
-                                 const gchar*     name,
-                                 gpointer         gself)
-    {
-        g_debug("on_name_acquired");
-        auto self = static_cast<TimedateFixture*>(gself);
-        self->name_acquired = true;
-        self->proxy = g_dbus_proxy_new_sync(conn,
-                G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES,
-                nullptr,
-                name,
-                "/org/freedesktop/timedate1",
-                "org.freedesktop.timedate1",
-                nullptr,
-                nullptr);
-        g_main_loop_quit(self->loop);
-    }
-
-    static void on_name_lost(GDBusConnection* /*conn*/,
-                             const gchar*     /*name*/,
-                             gpointer           gself)
-    {
-        g_debug("on_name_lost");
-        auto self = static_cast<TimedateFixture*>(gself);
-        self->name_acquired = false;
-    }
-
-    static void on_bus_closed(GObject*       /*object*/,
-                              GAsyncResult*    res,
-                              gpointer         gself)
-    {
-        g_debug("on_bus_closed");
-        auto self = static_cast<TimedateFixture*>(gself);
-        GError* err = nullptr;
-        g_dbus_connection_close_finish(self->connection, res, &err);
-        g_assert_no_error(err);
-        g_main_loop_quit(self->loop);
-    }
-
-    static void
-    timedate1_handle_method_call(GDBusConnection       *   connection,
-                                 const gchar           * /*sender*/,
-                                 const gchar           *   object_path,
-                                 const gchar           *   interface_name,
-                                 const gchar           *   method_name,
-                                 GVariant              *   parameters,
-                                 GDBusMethodInvocation *   invocation,
-                                 gpointer                  gself)
-    {
-        g_assert(!g_strcmp0(method_name, "SetTimezone"));
-        g_assert(g_variant_is_of_type(parameters, G_VARIANT_TYPE_TUPLE));
-        g_assert(2 == g_variant_n_children(parameters));
-
-        auto child = g_variant_get_child_value(parameters, 0);
-        g_assert(g_variant_is_of_type(child, G_VARIANT_TYPE_STRING));
-        auto self = static_cast<TimedateFixture*>(gself);
-        self->attempted_tzid = g_variant_get_string(child, nullptr);
-        g_debug("set tz (dbus side): '%s'", self->attempted_tzid.c_str());
-        g_dbus_method_invocation_return_value(invocation, nullptr);
-
-        /* Send PropertiesChanged */
-        GError * local_error = nullptr;
-        auto builder = g_variant_builder_new (G_VARIANT_TYPE_ARRAY);
-        g_variant_builder_add (builder,
-                               "{sv}",
-                               "Timezone",
-                               g_variant_new_string(
-                                   self->attempted_tzid.c_str()));
-        g_dbus_connection_emit_signal (connection,
-                                       NULL,
-                                       object_path,
-                                       "org.freedesktop.DBus.Properties",
-                                       "PropertiesChanged",
-                                       g_variant_new ("(sa{sv}as)",
-                                                      interface_name,
-                                                      builder,
-                                                      NULL),
-                                       &local_error);
-        g_assert_no_error (local_error);
-        g_variant_unref(child);
-    }
+    using super = GlibFixture;
 
 protected:
 
-    std::shared_ptr<MockState> m_mock_state;
-    std::shared_ptr<State> m_state;
-    std::shared_ptr<MockLiveActions> m_live_actions;
-    std::shared_ptr<Actions> m_actions;
-
-    bool name_acquired;
-    std::string attempted_tzid;
-
-    GTestDBus* bus;
-    guint own_name;
-    GDBusConnection* connection;
-    GDBusNodeInfo* node_info;
-    int object_register_id;
-    GDBusProxy *proxy;
-
-    void SetUp()
+    GDBusConnection* m_bus {};
+    GTestDBus* m_test_bus {};
+
+    virtual void SetUp() override
     {
         super::SetUp();
-        g_debug("SetUp");
-
-        name_acquired = false;
-        attempted_tzid.clear();
-        connection = nullptr;
-        node_info = nullptr;
-        object_register_id = 0;
-        own_name = 0;
-        proxy = nullptr;
-
-        // bring up the test bus
-        bus = g_test_dbus_new(G_TEST_DBUS_NONE);
-        g_test_dbus_up(bus);
-        const auto address = g_test_dbus_get_bus_address(bus);
+
+        // use a fake bus
+        m_test_bus = g_test_dbus_new(G_TEST_DBUS_NONE);
+        g_test_dbus_up(m_test_bus);
+        const char * address = g_test_dbus_get_bus_address(m_test_bus);
         g_setenv("DBUS_SYSTEM_BUS_ADDRESS", address, true);
         g_setenv("DBUS_SESSION_BUS_ADDRESS", address, true);
         g_debug("test_dbus's address is %s", address);
 
-        // parse the org.freedesktop.timedate1 interface
-        const gchar introspection_xml[] =
-            "<node>"
-            "  <interface name='org.freedesktop.timedate1'>"
-            "  <property name='Timezone' type='s' access='read' />"
-            "    <method name='SetTimezone'>"
-            "      <arg name='timezone' type='s' direction='in'/>"
-            "      <arg name='user_interaction' type='b' direction='in'/>"
-            "    </method>"
-            "  </interface>"
-            "</node>";
-        node_info = g_dbus_node_info_new_for_xml(introspection_xml, nullptr);
-        ASSERT_TRUE(node_info != nullptr);
-        ASSERT_TRUE(node_info->interfaces != nullptr);
-        ASSERT_TRUE(node_info->interfaces[0] != nullptr);
-        ASSERT_TRUE(node_info->interfaces[1] == nullptr);
-        ASSERT_STREQ("org.freedesktop.timedate1", node_info->interfaces[0]->name);
-
-        // own the bus
-        own_name = g_bus_own_name(G_BUS_TYPE_SYSTEM,
-                                  "org.freedesktop.timedate1",
-                                  G_BUS_NAME_OWNER_FLAGS_NONE,
-                                  on_bus_acquired, on_name_acquired, on_name_lost,
-                                  this, nullptr);
-        ASSERT_TRUE(object_register_id == 0);
-        ASSERT_FALSE(name_acquired);
-        ASSERT_TRUE(connection == nullptr);
-        g_main_loop_run(loop);
-        ASSERT_TRUE(object_register_id != 0);
-        ASSERT_TRUE(name_acquired);
-        ASSERT_TRUE(G_IS_DBUS_CONNECTION(connection));
-
-        // create the State and Actions
-        m_mock_state.reset(new MockState);
-        m_mock_state->settings.reset(new Settings);
-        m_state = std::dynamic_pointer_cast<State>(m_mock_state);
-        m_live_actions.reset(new MockLiveActions(m_state));
-        m_actions = std::dynamic_pointer_cast<Actions>(m_live_actions);
+        // get the bus
+        m_bus = g_bus_get_sync(G_BUS_TYPE_SESSION, nullptr, nullptr);
+        g_dbus_connection_set_exit_on_close(m_bus, FALSE);
+        g_object_add_weak_pointer(G_OBJECT(m_bus), (gpointer*)&m_bus);
     }
 
-    void TearDown()
+    virtual void TearDown() override
     {
-        g_debug("TearDown");
-        m_actions.reset();
-        m_live_actions.reset();
-        m_state.reset();
-        m_mock_state.reset();
-        g_dbus_connection_unregister_object(connection, object_register_id);
-        g_object_unref(proxy);
-        g_dbus_node_info_unref(node_info);
-        g_bus_unown_name(own_name);
-        g_dbus_connection_close(connection, nullptr, on_bus_closed, this);
-        g_main_loop_run(loop);
-        g_clear_object(&connection);
-        g_test_dbus_down(bus);
-        g_clear_object(&bus);
+        // take down the bus
+        bool bus_finished = false;
+        g_object_weak_ref(
+            G_OBJECT(m_bus),
+            [](gpointer gbus_finished, GObject*){*static_cast<bool*>(gbus_finished) = true;},
+            &bus_finished
+        );
+        g_clear_object(&m_bus);
+        EXPECT_TRUE(wait_for([&bus_finished](){return bus_finished;}));
+
+        // take down the GTestBus
+        g_clear_object(&m_test_bus);
 
         super::TearDown();
     }
-public:
-    void set_timezone(std::string tz)
-    {
-        g_debug("set_timezone: '%s'", tz.c_str());
-        g_dbus_proxy_call_sync(proxy,
-                "SetTimezone",
-                g_variant_new("(sb)",
-                    tz.c_str(),
-                    FALSE),
-                G_DBUS_CALL_FLAGS_NONE,
-                500,
-                nullptr,
-                nullptr);
+
+    void start_timedate1(const std::string& tzid)
+    {
+        // start dbusmock with the timedated template
+        auto json_parameters = g_strdup_printf("{\"Timezone\": \"%s\"}", tzid.c_str());
+        const gchar* child_argv[] = {
+            "python3", "-m", "dbusmock",
+            "--template", "timedated",
+            "--parameters", json_parameters,
+            nullptr
+        };
+        GError* error = nullptr;
+        g_spawn_async(nullptr, (gchar**)child_argv, nullptr, G_SPAWN_SEARCH_PATH, nullptr, nullptr, nullptr, &error);
+        g_assert_no_error(error);
+        g_free(json_parameters);
+
+        // wait for it to appear on the bus
+        wait_for_name_owned(m_bus, Bus::Timedate1::BUSNAME);
+    }
+
+    bool wait_for_tzid(const std::string& tzid, unity::indicator::datetime::Timezone& tz)
+    {
+        return wait_for([&tzid, &tz](){return tzid == tz.timezone.get();});
+    }
+
+    void set_timedate1_timezone(const std::string& tzid)
+    {
+        GError* error {};
+        auto v = g_dbus_connection_call_sync(
+            m_bus,
+            Bus::Timedate1::BUSNAME,
+            Bus::Timedate1::ADDR,
+            Bus::Timedate1::IFACE,
+            Bus::Timedate1::Methods::SET_TIMEZONE,
+            g_variant_new("(sb)", tzid.c_str(), FALSE),
+            nullptr,
+            G_DBUS_CALL_FLAGS_NONE,
+            -1,
+            nullptr,
+            &error);
+        g_assert_no_error(error);
+
+        g_clear_pointer(&v, g_variant_unref);
+    }
+
+    std::string get_timedate1_timezone()
+    {
+        GError* error {};
+        auto v = g_dbus_connection_call_sync(
+            m_bus,
+            Bus::Timedate1::BUSNAME,
+            Bus::Timedate1::ADDR,
+            Bus::Properties::IFACE,
+            Bus::Properties::Methods::GET,
+            g_variant_new("(ss)", Bus::Timedate1::IFACE, Bus::Timedate1::Properties::TIMEZONE),
+            G_VARIANT_TYPE("(v)"),
+            G_DBUS_CALL_FLAGS_NONE,
+            -1,
+            nullptr,
+            &error);
+        g_assert_no_error(error);
+
+        GVariant* tzv {}; 
+        g_variant_get(v, "(v)", &tzv); 
+        std::string tzid;
+        const char* tz = g_variant_get_string(tzv, nullptr); 
+        if (tz != nullptr)
+            tzid = tz;
+
+        g_clear_pointer(&tzv, g_variant_unref);
+        g_clear_pointer(&v, g_variant_unref);
+        return tzid;
     }
 };
 
-#endif
+#define EXPECT_TZID(expected_tzid, tmp) \
+    EXPECT_TRUE(wait_for_tzid(expected_tzid, tmp)) \
+        << "expected " << expected_tzid \
+        << " got " << tmp.timezone.get();
+