Merge lp:~xavi-garcia-mena/indicator-sound/bug-1213907-active-players-playback-control-unit-tests into lp:indicator-sound/15.10

[Xavi Garcia]

This branch adds extra integration tests to the recently landed branch https://code.launchpad.net/~xavi-garcia-mena/indicator-sound/bug-1213907-active-players-playback-control

It adds GSettings tests and checks the last-running-player property.

This branch adds extra integration tests to the recently landed branch https://code.launchpad.net/~xavi-garcia-mena/indicator-sound/bug-1213907-active-players-playback-control

It adds GSettings tests and checks the last-running-player property.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:528
http://jenkins.qa.ubuntu.com/job/indicator-sound-ci/310/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-sound-wily-amd64-ci/85/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-sound-wily-armhf-ci/86/console

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

review: Needs Fixing (continuous-integration)

[lp:~xavi-garcia-mena/indicator-sound/bug-1213907-active-players-playback-control-unit-tests]

529. By Xavi Garcia on 2016-02-17

Integration test moved

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:529
http://jenkins.qa.ubuntu.com/job/indicator-sound-ci/311/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-sound-wily-amd64-ci/86
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-sound-wily-armhf-ci/87
        deb: http://jenkins.qa.ubuntu.com/job/indicator-sound-wily-armhf-ci/87/artifact/work/output/*zip*/output.zip

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

review: Approve (continuous-integration)

[Charles Kerr (charlesk)]

Comments inline

review: Needs Information

Unmerged revisions

529. By Xavi Garcia on 2016-02-17

Integration test moved

528. By Xavi Garcia on 2016-02-16

Added integration tests

=== modified file 'tests/integration/CMakeLists.txt'
--- tests/integration/CMakeLists.txt	2015-12-23 13:35:46 +0000
+++ tests/integration/CMakeLists.txt	2016-02-17 09:23:14 +0000
@@ -51,6 +51,7 @@
     indicator-sound-test-base.cpp
     test-indicator.cpp
     utils/dbus-pulse-volume.cpp
+    utils/gsettings.cpp
     main.cpp
 )
 
 
=== modified file 'tests/integration/indicator-sound-test-base.cpp'
--- tests/integration/indicator-sound-test-base.cpp	2016-01-29 11:16:34 +0000
+++ tests/integration/indicator-sound-test-base.cpp	2016-02-17 09:23:14 +0000
@@ -380,6 +380,11 @@
     }
 }
 
+void IndicatorSoundTestBase::stopIndicator()
+{
+    indicator.reset();
+}
+
 mh::MenuMatcher::Parameters IndicatorSoundTestBase::desktopParameters()
 {
     return mh::MenuMatcher::Parameters(
 
=== modified file 'tests/integration/indicator-sound-test-base.h'
--- tests/integration/indicator-sound-test-base.h	2016-01-29 11:16:34 +0000
+++ tests/integration/indicator-sound-test-base.h	2016-02-17 09:23:14 +0000
@@ -72,6 +72,7 @@
     void TearDown() override;
 
     void startIndicator();
+    void stopIndicator();
     void startPulseDesktop(DevicePortType speakerPort=WIRED, DevicePortType headphonesPort=WIRED);
     void startPulsePhone(DevicePortType speakerPort=WIRED, DevicePortType headphonesPort=WIRED);
     void startAccountsService();
 
=== modified file 'tests/integration/test-indicator.cpp'
--- tests/integration/test-indicator.cpp	2016-02-10 13:08:49 +0000
+++ tests/integration/test-indicator.cpp	2016-02-17 09:23:14 +0000
@@ -17,11 +17,15 @@
  */
 
 #include <indicator-sound-test-base.h>
+#include <utils/gsettings.h>
 
 #include <QDebug>
 #include <QTestEventLoop>
 #include <QSignalSpy>
 
+#include <chrono>
+#include <thread>
+
 using namespace std;
 using namespace testing;
 namespace mh = unity::gmenuharness;
@@ -1553,4 +1557,274 @@
     checkPortDevicesLabels(HDMI, HDMI);
 }
 
+TEST_F(TestIndicator, DesktopMprisPlayersPlaybackControlsActive)
+{
+    double INITIAL_VOLUME = 0.0;
+
+    ASSERT_NO_THROW(startAccountsService());
+    EXPECT_TRUE(clearGSettingsPlayers());
+    ASSERT_NO_THROW(startPulseDesktop());
+
+    // initialize volumes in pulseaudio
+    EXPECT_FALSE(setStreamRestoreVolume("alert", INITIAL_VOLUME));
+    EXPECT_TRUE(setSinkVolume(INITIAL_VOLUME));
+
+    // start the test player
+    EXPECT_TRUE(startTestMprisPlayer("testplayer1"));
+
+    // start now the indicator, so it picks the new volumes
+    ASSERT_NO_THROW(startIndicator());
+
+    // check that the player is added
+    EXPECT_MATCHRESULT(mh::MenuMatcher(desktopParameters())
+        .item(mh::MenuItemMatcher()
+            .action("indicator.root")
+            .string_attribute("x-canonical-type", "com.canonical.indicator.root")
+            .string_attribute("x-canonical-secondary-action", "indicator.mute")
+            .mode(mh::MenuItemMatcher::Mode::all)
+            .submenu()
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher().checkbox()
+                    .label("Mute")
+                )
+                .item(volumeSlider(INITIAL_VOLUME, "Volume"))
+            )
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher()
+                    .action("indicator.testplayer1.desktop")
+                    .label("TestPlayer1")
+                    .themed_icon("icon", {"testplayer"})
+                    .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                )
+                .item(mh::MenuItemMatcher()
+                    .string_attribute("x-canonical-previous-action","indicator.previous.testplayer1.desktop")
+                    .string_attribute("x-canonical-play-action","indicator.play.testplayer1.desktop")
+                    .string_attribute("x-canonical-next-action","indicator.next.testplayer1.desktop")
+                    .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                )
+            )
+            .item(mh::MenuItemMatcher()
+                            .label("Sound Settings…")
+             )
+        ).match());
+
+    // start the second test player
+    EXPECT_TRUE(startTestMprisPlayer("testplayer2"));
+
+    // check that the player is added
+    EXPECT_MATCHRESULT(mh::MenuMatcher(desktopParameters())
+        .item(mh::MenuItemMatcher()
+            .action("indicator.root")
+            .string_attribute("x-canonical-type", "com.canonical.indicator.root")
+            .string_attribute("x-canonical-secondary-action", "indicator.mute")
+            .mode(mh::MenuItemMatcher::Mode::all)
+            .submenu()
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher().checkbox()
+                    .label("Mute")
+                )
+                .item(volumeSlider(INITIAL_VOLUME, "Volume"))
+            )
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher()
+                    .action("indicator.testplayer1.desktop")
+                    .label("TestPlayer1")
+                    .themed_icon("icon", {"testplayer"})
+                    .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                )
+                .item(mh::MenuItemMatcher()
+                    .string_attribute("x-canonical-previous-action","indicator.previous.testplayer1.desktop")
+                    .string_attribute("x-canonical-play-action","indicator.play.testplayer1.desktop")
+                    .string_attribute("x-canonical-next-action","indicator.next.testplayer1.desktop")
+                    .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                )
+            )
+            .item(mh::MenuItemMatcher()
+                        .section()
+                        .item(mh::MenuItemMatcher()
+                            .action("indicator.testplayer2.desktop")
+                            .label("TestPlayer2")
+                            .themed_icon("icon", {"testplayer"})
+                            .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                        )
+                        .item(mh::MenuItemMatcher()
+                            .string_attribute("x-canonical-previous-action","indicator.previous.testplayer2.desktop")
+                            .string_attribute("x-canonical-play-action","indicator.play.testplayer2.desktop")
+                            .string_attribute("x-canonical-next-action","indicator.next.testplayer2.desktop")
+                            .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                        )
+                    )
+            .item(mh::MenuItemMatcher()
+                            .label("Sound Settings…")
+             )
+        ).match());
+
+    // stop the first player
+    EXPECT_TRUE(stopTestMprisPlayer("testplayer1"));
+
+    // check that the player's playback controls are removed
+    EXPECT_MATCHRESULT(mh::MenuMatcher(desktopParameters())
+        .item(mh::MenuItemMatcher()
+            .action("indicator.root")
+            .string_attribute("x-canonical-type", "com.canonical.indicator.root")
+            .string_attribute("x-canonical-secondary-action", "indicator.mute")
+            .mode(mh::MenuItemMatcher::Mode::all)
+            .submenu()
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher().checkbox()
+                    .label("Mute")
+                )
+                .item(volumeSlider(INITIAL_VOLUME, "Volume"))
+            )
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher()
+                    .action("indicator.testplayer1.desktop")
+                    .label("TestPlayer1")
+                    .themed_icon("icon", {"testplayer"})
+                    .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                )
+            )
+            .item(mh::MenuItemMatcher()
+                        .section()
+                        .item(mh::MenuItemMatcher()
+                            .action("indicator.testplayer2.desktop")
+                            .label("TestPlayer2")
+                            .themed_icon("icon", {"testplayer"})
+                            .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                        )
+                        .item(mh::MenuItemMatcher()
+                            .string_attribute("x-canonical-previous-action","indicator.previous.testplayer2.desktop")
+                            .string_attribute("x-canonical-play-action","indicator.play.testplayer2.desktop")
+                            .string_attribute("x-canonical-next-action","indicator.next.testplayer2.desktop")
+                            .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                        )
+                    )
+            .item(mh::MenuItemMatcher()
+                            .label("Sound Settings…")
+             )
+        ).match());
+
+    // stop the second player (it should be the last running one)
+    EXPECT_TRUE(stopTestMprisPlayer("testplayer2"));
+
+    // verify the players with gsettings
+    Settings settings;
+
+    int timeout = 1000;
+    while (settings.get_last_running_player() != "testplayer2.desktop" && timeout)
+    {
+        std::this_thread::sleep_for(std::chrono::milliseconds(500));
+        timeout -= 500;
+    }
+    EXPECT_EQ("testplayer2.desktop", settings.get_last_running_player());
+
+    timeout = 1000;
+    while (settings.get_interested_media_players() != "[[ testplayer1.desktop ][ testplayer2.desktop ]]" && timeout)
+    {
+        std::this_thread::sleep_for(std::chrono::milliseconds(500));
+        timeout -= 500;
+    }
+    EXPECT_EQ("[[ testplayer1.desktop ][ testplayer2.desktop ]]", settings.get_interested_media_players());
+
+    // restart the indicator
+    // we should get the player 2 as the one with playback constrols.
+    // In this case we only have the play button as the player is not running yet.
+    startIndicator();
+
+    // check that the player 2 has the playback controls with play set.
+    EXPECT_MATCHRESULT(mh::MenuMatcher(desktopParameters())
+        .item(mh::MenuItemMatcher()
+            .action("indicator.root")
+            .string_attribute("x-canonical-type", "com.canonical.indicator.root")
+            .string_attribute("x-canonical-secondary-action", "indicator.mute")
+            .mode(mh::MenuItemMatcher::Mode::all)
+            .submenu()
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher().checkbox()
+                    .label("Mute")
+                )
+                .item(volumeSlider(INITIAL_VOLUME, "Volume"))
+            )
+            .item(mh::MenuItemMatcher()
+                .section()
+                .item(mh::MenuItemMatcher()
+                    .action("indicator.testplayer1.desktop")
+                    .label("TestPlayer1")
+                    .themed_icon("icon", {"testplayer"})
+                    .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                )
+            )
+            .item(mh::MenuItemMatcher()
+                        .section()
+                        .item(mh::MenuItemMatcher()
+                            .action("indicator.testplayer2.desktop")
+                            .label("TestPlayer2")
+                            .themed_icon("icon", {"testplayer"})
+                            .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                        )
+                        .item(mh::MenuItemMatcher()
+                            .string_attribute("x-canonical-play-action","indicator.play.testplayer2.desktop")
+                            .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                        )
+                    )
+            .item(mh::MenuItemMatcher()
+                            .label("Sound Settings…")
+             )
+        ).match());
+
+    // now start the other player.
+    // that should change the playback controls shown from player 2 to player 1
+    EXPECT_TRUE(startTestMprisPlayer("testplayer1"));
+
+    EXPECT_MATCHRESULT(mh::MenuMatcher(desktopParameters())
+            .item(mh::MenuItemMatcher()
+                .action("indicator.root")
+                .string_attribute("x-canonical-type", "com.canonical.indicator.root")
+                .string_attribute("x-canonical-secondary-action", "indicator.mute")
+                .mode(mh::MenuItemMatcher::Mode::all)
+                .submenu()
+                .item(mh::MenuItemMatcher()
+                    .section()
+                    .item(mh::MenuItemMatcher().checkbox()
+                        .label("Mute")
+                    )
+                    .item(volumeSlider(INITIAL_VOLUME, "Volume"))
+                )
+                .item(mh::MenuItemMatcher()
+                    .section()
+                    .item(mh::MenuItemMatcher()
+                        .action("indicator.testplayer1.desktop")
+                        .label("TestPlayer1")
+                        .themed_icon("icon", {"testplayer"})
+                        .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                    )
+                    .item(mh::MenuItemMatcher()
+                        .string_attribute("x-canonical-previous-action","indicator.previous.testplayer1.desktop")
+                        .string_attribute("x-canonical-play-action","indicator.play.testplayer1.desktop")
+                        .string_attribute("x-canonical-next-action","indicator.next.testplayer1.desktop")
+                        .string_attribute("x-canonical-type","com.canonical.unity.playback-item")
+                    )
+                )
+                .item(mh::MenuItemMatcher()
+                            .section()
+                            .item(mh::MenuItemMatcher()
+                                .action("indicator.testplayer2.desktop")
+                                .label("TestPlayer2")
+                                .themed_icon("icon", {"testplayer"})
+                                .string_attribute("x-canonical-type", "com.canonical.unity.media-player")
+                            )
+                        )
+                .item(mh::MenuItemMatcher()
+                                .label("Sound Settings…")
+                 )
+            ).match());
+}
+
 } // namespace
 
=== added file 'tests/integration/utils/gobj_memory.h'
--- tests/integration/utils/gobj_memory.h	1970-01-01 00:00:00 +0000
+++ tests/integration/utils/gobj_memory.h	2016-02-17 09:23:14 +0000
@@ -0,0 +1,200 @@
+/*
+ * Copyright (C) 2013 Canonical Ltd.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by: Jussi Pakkanen <[e-mail]>
+ */
+
+#pragma once
+
+#include <stdexcept>
+
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#pragma GCC diagnostic ignored "-Wcast-qual"
+#include <glib-object.h>
+
+#include <iostream>
+
+/**
+ * This class is meant for automatically managing the lifetime of C objects derived
+ * from gobject. Its API perfectly mirrors the API of unique_ptr except that you
+ * can't define your own deleter function as it is always g_object_unref.
+ *
+ * API/ABI stability is not guaranteed. If you need to pass the object across an ABI
+ * boundary, pass the plain gobject.
+ *
+ * This is how you would use gobj_ptr 99% of the time:
+ *
+ * gobj_ptr<GSomeType> o(g_some_type_new(...));
+ *
+ * More specifically, the object will decrement the gobject reference count
+ * of the object it points to when it goes out of scope. It will never increment it.
+ * Thus you should only assign to it when already holding a reference. gobj_ptr
+ * will then take ownership of that particular reference.
+ *
+ * Floating gobjects can not be put in this container as they are meant to be put
+ * into native gobject aware containers immediately upon construction. Trying to insert
+ * a floating gobject into a gobj_ptr will throw an invalid_argument exception. To
+ * prevent accidental memory leaks, the floating gobject is unreffed in this case.
+ */
+template <typename T>
+class gobj_ptr final
+{
+private:
+    T* u;
+
+    void validate_float(T* t)
+    {
+        if (t != nullptr && g_object_is_floating(G_OBJECT(t)))
+        {
+            // LCOV_EXCL_START  // False negative from gcovr.
+            throw std::invalid_argument("Tried to add a floating gobject into a gobj_ptr.");
+            // LCOV_EXCL_STOP
+        }
+    }
+
+public:
+    typedef T element_type;
+    typedef T* pointer;
+    typedef decltype(g_object_unref) deleter_type;
+
+    constexpr gobj_ptr() noexcept : u(nullptr)
+    {
+    }
+    explicit gobj_ptr(T* t)
+        : u(t)
+    {
+        // What should we do if validate throws? Unreffing unknown objs
+        // is dodgy but not unreffing runs the risk of
+        // memory leaks. Currently unrefs as u is destroyed
+        // when this exception is thrown.
+        validate_float(t);
+    }
+    constexpr gobj_ptr(std::nullptr_t) noexcept : u(nullptr){};
+    gobj_ptr(gobj_ptr&& o) noexcept
+    {
+        u = o.u;
+        o.u = nullptr;
+    }
+    gobj_ptr(const gobj_ptr& o)
+        : u(nullptr)
+    {
+        *this = o;
+    }
+    gobj_ptr& operator=(const gobj_ptr& o)
+    {
+        if (o.u != nullptr)
+        {
+            g_object_ref(o.u);
+        }
+        reset(o.u);
+        return *this;
+    }
+    ~gobj_ptr()
+    {
+        reset();
+    }
+
+    deleter_type& get_deleter() noexcept
+    {
+        return g_object_unref;
+    }
+    deleter_type& get_deleter() const noexcept
+    {
+        return g_object_unref;
+    }
+
+    void swap(gobj_ptr<T>& o) noexcept
+    {
+        T* tmp = u;
+        u = o.u;
+        o.u = tmp;
+    }
+    void reset(pointer p = pointer())
+    {
+        if (u != nullptr)
+        {
+            g_object_unref(G_OBJECT(u));
+            u = nullptr;
+        }
+        // Same throw dilemma as in pointer constructor.
+        u = p;
+        validate_float(p);
+    }
+
+    T* release() noexcept
+    {
+        T* r = u;
+        u = nullptr;
+        return r;
+    }
+    T* get() const noexcept
+    {
+        return u;
+    }
+
+    T& operator*() const
+    {
+        return *u;
+    }
+    T* operator->() const noexcept
+    {
+        return u;
+    }
+    explicit operator bool() const noexcept
+    {
+        return u != nullptr;
+    }
+
+    gobj_ptr& operator=(gobj_ptr&& o) noexcept
+    {
+        reset();
+        u = o.u;
+        o.u = nullptr;
+        return *this;
+    }
+    gobj_ptr& operator=(std::nullptr_t) noexcept
+    {
+        reset();
+        return *this;
+    }
+    bool operator==(const gobj_ptr<T>& o) const noexcept
+    {
+        return u == o.u;
+    }
+    bool operator!=(const gobj_ptr<T>& o) const noexcept
+    {
+        return u != o.u;
+    }
+    bool operator<(const gobj_ptr<T>& o) const noexcept
+    {
+        return u < o.u;
+    }
+    bool operator<=(const gobj_ptr<T>& o) const noexcept
+    {
+        return u <= o.u;
+    }
+    bool operator>(const gobj_ptr<T>& o) const noexcept
+    {
+        return u > o.u;
+    }
+    bool operator>=(const gobj_ptr<T>& o) const noexcept
+    {
+        return u >= o.u;
+    }
+};
+
+#pragma GCC diagnostic pop
+
 
=== added file 'tests/integration/utils/gsettings.cpp'
--- tests/integration/utils/gsettings.cpp	1970-01-01 00:00:00 +0000
+++ tests/integration/utils/gsettings.cpp	2016-02-17 09:23:14 +0000
@@ -0,0 +1,149 @@
+/*
+ * Copyright (C) 2016 Canonical Ltd.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by: Xavi Garcia <[e-mail]>
+ */
+
+#include "gsettings.h"
+
+//#include "settings-defaults.h"
+
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#pragma GCC diagnostic ignored "-Wcast-qual"
+#include <gio/gio.h>
+#pragma GCC diagnostic pop
+#include <QDebug>
+
+#include <memory>
+
+using namespace std;
+
+Settings::Settings()
+    : Settings("com.canonical.indicator.sound")
+{
+}
+
+Settings::Settings(string const& schema_name)
+    : schema_(nullptr, &g_settings_schema_unref)
+    , schema_name_(schema_name)
+{
+    GSettingsSchemaSource* src = g_settings_schema_source_get_default();
+    schema_.reset(g_settings_schema_source_lookup(src, schema_name.c_str(), true));
+    if (schema_)
+    {
+        settings_.reset(g_settings_new(schema_name.c_str()));
+    }
+    else
+    {
+        qCritical() << "The schema" << schema_name.c_str() << "is missing";
+    }
+}
+
+Settings::~Settings() = default;
+
+string Settings::get_last_running_player() const
+{
+    return get_string("last-running-player", "");
+}
+
+string Settings::get_interested_media_players() const
+{
+    return get_string_array_as_string("interested-media-players", "");
+}
+
+string Settings::get_string(char const* key, string const& default_value) const
+{
+    if (!settings_ || !g_settings_schema_has_key(schema_.get(), key))
+    {
+        return default_value;
+    }
+
+    char *value = g_settings_get_string(settings_.get(), key);
+    if (value)
+    {
+        string result = value;
+        g_free(value);
+        return result;
+    }
+    return default_value; // LCOV_EXCL_LINE
+}
+
+int Settings::get_positive_int(char const* key, int default_value) const
+{
+    int i = get_int(key, default_value);
+    if (i <= 0)
+    {
+        throw domain_error(string("Settings::get_positive_int(): invalid zero or negative value for ")
+                           + key + ": " + to_string(i) + " in schema " + schema_name_);
+    }
+    return i;
+}
+
+int Settings::get_positive_or_zero_int(char const* key, int default_value) const
+{
+    int i = get_int(key, default_value);
+    if (i < 0)
+    {
+        throw domain_error(string("Settings::get_positive_or_zero_int(): invalid negative value for ")
+                           + key + ": " + to_string(i) + " in schema " + schema_name_);
+    }
+    return i;
+}
+
+int Settings::get_int(char const* key, int default_value) const
+{
+    if (!settings_ || !g_settings_schema_has_key(schema_.get(), key))
+    {
+        return default_value;
+    }
+
+    return g_settings_get_int(settings_.get(), key);
+}
+
+bool Settings::get_bool(char const* key, bool default_value) const
+{
+    if (!settings_ || !g_settings_schema_has_key(schema_.get(), key))
+    {
+        return default_value;
+    }
+
+    return g_settings_get_boolean(settings_.get(), key);
+}
+
+std::string Settings::get_string_array_as_string(char const* key, std::string const& default_value) const
+{
+    if (!settings_ || !g_settings_schema_has_key(schema_.get(), key))
+    {
+        return default_value;
+    }
+    gchar **values;
+    values = g_settings_get_strv(settings_.get(), key);
+    if (!values)
+    {
+        return default_value;
+    }
+    int length = 0;
+    std::string res = "[";
+    while (((gpointer*) values)[length])
+    {
+        std::string test(values[length]);
+        length++;
+        res += "[ " + test + " ]";
+    }
+    res += "]";
+
+    return res;
+}
 
=== added file 'tests/integration/utils/gsettings.h'
--- tests/integration/utils/gsettings.h	1970-01-01 00:00:00 +0000
+++ tests/integration/utils/gsettings.h	2016-02-17 09:23:14 +0000
@@ -0,0 +1,52 @@
+/*
+ * Copyright (C) 2016 Canonical Ltd.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by: Xavi Garcia <[e-mail]>
+ */
+
+#pragma once
+
+#include "gobj_memory.h"
+
+#include <memory>
+#include <string>
+
+typedef struct _GSettings GSettings;
+typedef struct _GSettingsSchema GSettingsSchema;
+
+class Settings
+{
+public:
+    Settings();
+    // This constructor is only for use in the tests.
+    explicit Settings(std::string const& schema_name);
+    ~Settings();
+
+    std::string get_last_running_player() const;
+
+    std::string get_interested_media_players() const;
+
+private:
+    std::string get_string(char const* key, std::string const& default_value) const;
+    int get_positive_int(char const* key, int default_value) const;
+    int get_positive_or_zero_int(char const* key, int default_value) const;
+    int get_int(char const* key, int default_value) const;
+    bool get_bool(char const* key, bool default_value) const;
+    std::string get_string_array_as_string(char const* key, std::string const& default_value) const;
+
+    std::unique_ptr<GSettingsSchema, void(*)(GSettingsSchema*)> schema_;
+    std::string schema_name_;
+    gobj_ptr<GSettings> settings_;
+};