Merge lp:~nick-dedekind/unity8/lp1385331.led into lp:unity8

[Nick Dedekind]

Unhook Lights interface from indicator widgets

Unhook Lights interface from indicator widgets

 * Are there any related MPs required for this MP to build/function as expected? Please list.
https://code.launchpad.net/~nick-dedekind/qmenumodel/lp1385331.led/+merge/241422

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

 * Did you make sure that your branch does not contain spurious tags?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1373
http://jenkins.qa.ubuntu.com/job/unity8-ci/4919/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/237/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/96/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/84/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/84/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/237/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/4919/rebuild

review: Needs Fixing (continuous-integration)

[lp:~nick-dedekind/unity8/lp1385331.led]

1374. By Nick Dedekind on 2014-11-24

removed author

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1374
http://jenkins.qa.ubuntu.com/job/unity8-ci/4960/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/328/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/137/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/125/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/125/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/328/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/4960/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

Text conflict in debian/control
1 conflicts encountered.

[lp:~nick-dedekind/unity8/lp1385331.led]

1375. By Nick Dedekind on 2014-11-28

merged with trunk

[Nick Dedekind (nick-dedekind)]

> Text conflict in debian/control
> 1 conflicts encountered.

Fixed.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1375
http://jenkins.qa.ubuntu.com/job/unity8-ci/4993/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/411/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/170/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/158/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/158/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/411/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/4993/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

Text conflict in debian/control
Text conflict in tests/mocks/CMakeLists.txt
Text conflict in tests/mocks/QMenuModel/QDBusActionGroup.qml
3 conflicts encountered.

review: Needs Fixing

[lp:~nick-dedekind/unity8/lp1385331.led]

1376. By Nick Dedekind on 2014-12-11

merged with trunk

1377. By Nick Dedekind on 2014-12-12

better lights init

[Albert Astals Cid (aacid)]

Is there an easy way to reproduce the problem this is fixing?

review: Needs Information

[Nick Dedekind (nick-dedekind)]

1) kill unity8 when the light is blinking (new message + screen off)
2) clear messages (I use a messaging test script to add the new messages)
3) when unity8 restarts light will remain be blinking

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1377
http://jenkins.qa.ubuntu.com/job/unity8-ci/5043/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/562/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/207/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/208/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/208/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/560/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/5043/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

/home/phablet/unity8-lp1385331.led/plugins/Unity/Indicators/modelactionrootstate.cpp: multiple new lines at end of file

review: Needs Fixing

[Albert Astals Cid (aacid)]

 * Start messages.py
 * Led blinks forever
 * Kill unity8 (restarts itself)
 * Led blinks just one time more

I guess it should be still blinking forever?

review: Needs Information

[Nick Dedekind (nick-dedekind)]

> * Start messages.py
> * Led blinks forever
> * Kill unity8 (restarts itself)
> * Led blinks just one time more
>
> I guess it should be still blinking forever?

hm. if the screen is off and there are messages, it should keep blinking. I'll have to check what's happening.

[Nick Dedekind (nick-dedekind)]

> > * Start messages.py
> > * Led blinks forever
> > * Kill unity8 (restarts itself)
> > * Led blinks just one time more
> >
> > I guess it should be still blinking forever?
>
> hm. if the screen is off and there are messages, it should keep blinking. I'll
> have to check what's happening.

Tested this and in my instance, this is because Powerd.status is coming up as "On" when unity8 is restarted from a kill (when the screen is off). :/
Needs some sorting out, but it's not related to this branch.

[lp:~nick-dedekind/unity8/lp1385331.led]

1378. By Nick Dedekind on 2014-12-17

removed whitespace

[Nick Dedekind (nick-dedekind)]

> /home/phablet/unity8-lp1385331.led/plugins/Unity/Indicators/modelactionrootsta
> te.cpp: multiple new lines at end of file

Fixed.

[Albert Astals Cid (aacid)]

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass?
It needs a dependency

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1378
http://jenkins.qa.ubuntu.com/job/unity8-ci/5056/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/603/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/220/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/221/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/221/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/601/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/5056/rebuild

review: Needs Fixing (continuous-integration)

=== modified file 'debian/control'
--- debian/control	2014-12-08 18:15:11 +0000
+++ debian/control	2014-12-17 11:19:14 +0000
@@ -22,7 +22,7 @@
                libpam0g-dev,
                libpay2-dev,
                libpulse-dev,
-               libqmenumodel-dev (>= 0.2.8),
+               libqmenumodel-dev (>= 0.2.9),
                libqt5xmlpatterns5-dev,
                libsystemsettings-dev,
                libunity-api-dev (>= 7.94),
@@ -61,7 +61,7 @@
 
 Package: indicators-client
 Architecture: amd64 armhf i386
-Depends: qmenumodel-qml (>= 0.2.8),
+Depends: qmenumodel-qml (>= 0.2.9),
          qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.1.1239) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.1.1239),
          unity8 (= ${binary:Version}),
          ${misc:Depends},
@@ -85,7 +85,7 @@
 Depends: gsettings-desktop-schemas,
          libcap2-bin,
          libglib2.0-bin,
-         qmenumodel-qml (>= 0.2.8),
+         qmenumodel-qml (>= 0.2.9),
          qml-module-qt-labs-folderlistmodel,
          qml-module-qtquick-xmllistmodel,
          qtdeclarative5-gsettings1.0,

=== modified file 'plugins/Lights/Lights.cpp'
--- plugins/Lights/Lights.cpp	2014-09-08 15:07:24 +0000
+++ plugins/Lights/Lights.cpp	2014-12-17 11:19:14 +0000
@@ -44,12 +44,20 @@
 
 void Lights::setState(Lights::State newState)
 {
+    if (!init()) {
+        qWarning() << "No lights device";
+        return;
+    }
+
     if (m_state != newState) {
         if (newState == Lights::On) {
             turnOn();
         } else {
             turnOff();
         }
+
+        m_state = newState;
+        Q_EMIT stateChanged(m_state);
     }
 }
 
@@ -115,6 +123,7 @@
         err = module->methods->open(module, LIGHT_ID_NOTIFICATIONS, &device);
         if (err == 0) {
             m_lightDevice = (light_device_t*)device;
+            turnOff();
             return true;
         } else {
             qWarning() << "Failed to access notification lights";
@@ -127,15 +136,6 @@
 
 void Lights::turnOn()
 {
-    if (!init()) {
-        qWarning() << "No lights device";
-        return;
-    }
-
-    if (m_state == Lights::On) {
-        return;
-    }
-
     // pulse
     light_state_t state;
     memset(&state, 0, sizeof(light_state_t));
@@ -147,23 +147,11 @@
 
     if (m_lightDevice->set_light(m_lightDevice, &state) != 0) {
          qWarning() << "Failed to turn the light off";
-    } else {
-        m_state = Lights::On;
-        Q_EMIT stateChanged(m_state);
     }
 }
 
 void Lights::turnOff()
 {
-    if (!init()) {
-        qWarning() << "No lights device";
-        return;
-    }
-
-    if (m_state == Lights::Off) {
-        return;
-    }
-
     light_state_t state;
     memset(&state, 0, sizeof(light_state_t));
     state.color = 0x00000000;
@@ -174,8 +162,5 @@
 
     if (m_lightDevice->set_light(m_lightDevice, &state) != 0) {
         qWarning() << "Failed to turn the light off";
-    } else {
-        m_state = Lights::Off;
-        Q_EMIT stateChanged(m_state);
     }
 }

=== modified file 'plugins/Unity/Indicators/CMakeLists.txt'
--- plugins/Unity/Indicators/CMakeLists.txt	2014-10-03 10:53:02 +0000
+++ plugins/Unity/Indicators/CMakeLists.txt	2014-12-17 11:19:14 +0000
@@ -16,14 +16,16 @@
 )
 
 set(IndicatorsQML_SOURCES
+    actionrootstate.cpp
     indicator.cpp
     indicators.h
     indicatorsmanager.cpp
     indicatorsmodel.cpp
     menucontentactivator.cpp
+    modelactionrootstate.cpp
     modelprinter.cpp
     plugin.cpp
-    rootactionstate.cpp
+    rootstateparser.cpp
     sharedunitymenumodel.cpp
     unitymenumodelcache.cpp
     unitymenumodelstack.cpp

=== modified file 'plugins/Unity/Indicators/Indicators.qmltypes'
--- plugins/Unity/Indicators/Indicators.qmltypes	2014-10-07 10:58:39 +0000
+++ plugins/Unity/Indicators/Indicators.qmltypes	2014-12-17 11:19:14 +0000
@@ -8,6 +8,14 @@
 
 Module {
     Component {
+        name: "ActionRootState"
+        prototype: "RootStateObject"
+        exports: ["Unity.Indicators/ActionRootState 0.1"]
+        exportMetaObjectRevisions: [0]
+        Property { name: "actionGroup"; type: "QDBusActionGroup"; isPointer: true }
+        Property { name: "actionName"; type: "string" }
+    }
+    Component {
         name: "ActionState"
         prototype: "QObject"
         exports: ["Unity.Indicators/ActionState 0.1"]
@@ -23,7 +31,6 @@
             }
         }
     }
-    Component { name: "ActionStateParser"; prototype: "QObject" }
     Component {
         name: "FlatMenuProxyModelRole"
         prototype: "QObject"
@@ -147,6 +154,14 @@
         Property { name: "active"; type: "bool"; isReadonly: true }
     }
     Component {
+        name: "ModelActionRootState"
+        prototype: "RootStateObject"
+        exports: ["Unity.Indicators/ModelActionRootState 0.1"]
+        exportMetaObjectRevisions: [0]
+        Property { name: "menu"; type: "UnityMenuModel"; isPointer: true }
+        Signal { name: "indexChanged" }
+    }
+    Component {
         name: "ModelPrinter"
         prototype: "QObject"
         exports: ["Unity.Indicators/ModelPrinter 0.1"]
@@ -192,11 +207,8 @@
     }
     Component { name: "QIdentityProxyModel"; prototype: "QAbstractProxyModel" }
     Component {
-        name: "RootActionState"
-        prototype: "ActionStateParser"
-        exports: ["Unity.Indicators/RootActionState 0.1"]
-        exportMetaObjectRevisions: [0]
-        Property { name: "menu"; type: "UnityMenuModel"; isPointer: true }
+        name: "RootStateObject"
+        prototype: "QObject"
         Property { name: "valid"; type: "bool"; isReadonly: true }
         Property { name: "title"; type: "string"; isReadonly: true }
         Property { name: "leftLabel"; type: "string"; isReadonly: true }
@@ -205,7 +217,6 @@
         Property { name: "accessibleName"; type: "string"; isReadonly: true }
         Property { name: "visible"; type: "bool"; isReadonly: true }
         Signal { name: "updated" }
-        Signal { name: "indexChanged" }
     }
     Component {
         name: "SharedUnityMenuModel"

=== added file 'plugins/Unity/Indicators/actionrootstate.cpp'
--- plugins/Unity/Indicators/actionrootstate.cpp	1970-01-01 00:00:00 +0000
+++ plugins/Unity/Indicators/actionrootstate.cpp	2014-12-17 11:19:14 +0000
@@ -0,0 +1,97 @@
+/*
+ * Copyright 2014 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 as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#include "actionrootstate.h"
+
+#include <qdbusactiongroup.h>
+#include <QDebug>
+
+ActionRootState::ActionRootState(QObject *parent)
+    : RootStateObject(parent)
+    , m_actionGroup(nullptr)
+{
+}
+
+QDBusActionGroup *ActionRootState::actionGroup() const
+{
+    return m_actionGroup;
+}
+
+void ActionRootState::setActionGroup(QDBusActionGroup *actionGroup)
+{
+    if (m_actionGroup != actionGroup) {
+        bool wasValid = valid();
+
+        if (m_actionGroup) {
+            disconnect(m_actionGroup, 0, this, 0);
+        }
+        m_actionGroup = actionGroup;
+
+        if (m_actionGroup) {
+            connect(m_actionGroup, &QDBusActionGroup::statusChanged, this, [&](bool) { updateActionState(); });
+            connect(m_actionGroup, &QDBusActionGroup::actionAppear, this, [&](const QString&) { updateActionState(); });
+            connect(m_actionGroup, &QDBusActionGroup::actionVanish, this, [&](const QString&) { updateActionState(); });
+            connect(m_actionGroup, &QDBusActionGroup::actionStateChanged, this, [&](QVariant) { updateActionState(); });
+
+            connect(m_actionGroup, &QObject::destroyed, this, [&](QObject*) { updateActionState(); });
+        }
+        updateActionState();
+        Q_EMIT actionGroupChanged();
+
+        if (wasValid != valid()) Q_EMIT validChanged();
+    }
+}
+
+QString ActionRootState::actionName() const
+{
+    return m_actionName;
+}
+
+void ActionRootState::setActionName(const QString &actionName)
+{
+    if (m_actionName != actionName) {
+        bool wasValid = valid();
+
+        m_actionName = actionName;
+        updateActionState();
+
+        Q_EMIT actionNameChanged();
+
+        if (wasValid != valid()) Q_EMIT validChanged();
+    }
+}
+
+bool ActionRootState::valid() const
+{
+    return m_actionGroup && m_actionGroup->status() == DBusEnums::Connected &&
+           !m_actionName.isEmpty() && m_actionGroup->hasAction(m_actionName);
+}
+
+void ActionRootState::updateActionState()
+{
+    if (valid()) {
+        ActionStateParser* oldParser = m_actionGroup->actionStateParser();
+        m_actionGroup->setActionStateParser(&m_parser);
+
+        QVariantMap state = m_actionGroup->actionState(m_actionName).toMap();
+
+        m_actionGroup->setActionStateParser(oldParser);
+
+        setCurrentState(state);
+    } else {
+        setCurrentState(QVariantMap());
+    }
+}

=== added file 'plugins/Unity/Indicators/actionrootstate.h'
--- plugins/Unity/Indicators/actionrootstate.h	1970-01-01 00:00:00 +0000
+++ plugins/Unity/Indicators/actionrootstate.h	2014-12-17 11:19:14 +0000
@@ -0,0 +1,55 @@
+/*
+ * Copyright 2014 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 as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#ifndef ACTIONROOTSTATE_H
+#define ACTIONROOTSTATE_H
+
+#include "unityindicatorsglobal.h"
+
+#include "rootstateparser.h"
+
+class QDBusActionGroup;
+
+class UNITYINDICATORS_EXPORT ActionRootState : public RootStateObject
+{
+    Q_OBJECT
+    Q_PROPERTY(QDBusActionGroup* actionGroup READ actionGroup WRITE setActionGroup NOTIFY actionGroupChanged)
+    Q_PROPERTY(QString actionName READ actionName WRITE setActionName NOTIFY actionNameChanged)
+
+public:
+    ActionRootState(QObject *parent = 0);
+
+    QDBusActionGroup *actionGroup() const;
+    void setActionGroup(QDBusActionGroup *actionGroup);
+
+    QString actionName() const;
+    void setActionName(const QString& actionName);
+
+    bool valid() const override;
+
+Q_SIGNALS:
+    void actionGroupChanged();
+    void actionNameChanged();
+
+private Q_SLOTS:
+    void updateActionState();
+
+private:
+    QDBusActionGroup* m_actionGroup;
+    QString m_actionName;
+};
+
+#endif // ACTIONROOTSTATE_H

=== added file 'plugins/Unity/Indicators/modelactionrootstate.cpp'
--- plugins/Unity/Indicators/modelactionrootstate.cpp	1970-01-01 00:00:00 +0000
+++ plugins/Unity/Indicators/modelactionrootstate.cpp	2014-12-17 11:19:14 +0000
@@ -0,0 +1,127 @@
+/*
+ * Copyright 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 as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ *
+ * Authors:
+ *      Nick Dedekind <[e-mail]>
+ */
+
+#include "modelactionrootstate.h"
+#include "indicators.h"
+
+#include <unitymenumodel.h>
+#include <QVariant>
+#include <QIcon>
+
+extern "C" {
+#include <glib.h>
+#include <gio/gio.h>
+}
+
+ModelActionRootState::ModelActionRootState(QObject *parent)
+    : RootStateObject(parent),
+      m_menu(nullptr)
+{
+}
+
+ModelActionRootState::~ModelActionRootState()
+{
+}
+
+UnityMenuModel* ModelActionRootState::menu() const
+{
+    return m_menu;
+}
+
+void ModelActionRootState::setMenu(UnityMenuModel* menu)
+{
+    if (m_menu != menu) {
+        bool wasValid = valid();
+
+        if (m_menu) {
+            m_menu->disconnect(this);
+        }
+        m_menu = menu;
+
+        if (m_menu) {
+            connect(m_menu, SIGNAL(rowsInserted(const QModelIndex&, int, int)), SLOT(onModelRowsAdded(const QModelIndex&, int, int)));
+            connect(m_menu, SIGNAL(rowsRemoved(const QModelIndex&, int, int)), SLOT(onModelRowsRemoved(const QModelIndex&, int, int)));
+            connect(m_menu, SIGNAL(dataChanged(const QModelIndex&, const QModelIndex&, const QVector<int>&)), SLOT(onModelDataChanged(const QModelIndex&, const QModelIndex&, const QVector<int>&)));
+
+            connect(m_menu, SIGNAL(destroyed()), SLOT(reset()));
+        }
+        updateActionState();
+        Q_EMIT menuChanged();
+
+        if (wasValid != valid())
+            Q_EMIT validChanged();
+    }
+}
+
+bool ModelActionRootState::valid() const
+{
+    return m_menu && m_menu->rowCount() > 0;
+}
+
+void ModelActionRootState::onModelRowsAdded(const QModelIndex& parent, int start, int end)
+{
+    Q_UNUSED(parent);
+    if (start == 0 && end >= 0) {
+        updateActionState();
+    }
+}
+
+void ModelActionRootState::onModelRowsRemoved(const QModelIndex& parent, int start, int end)
+{
+    Q_UNUSED(parent);
+    if (start == 0 && end >= 0) {
+        updateActionState();
+    }
+}
+
+void ModelActionRootState::onModelDataChanged(const QModelIndex& topLeft, const QModelIndex& bottomRight, const QVector<int>& roles)
+{
+    Q_UNUSED(roles);
+    if (!topLeft.isValid() || !bottomRight.isValid()) {
+        return;
+    }
+
+    if (topLeft.row() <= 0 && bottomRight.row() >= 0) {
+        updateActionState();
+    }
+}
+
+void ModelActionRootState::reset()
+{
+    m_menu = nullptr;
+
+    Q_EMIT menuChanged();
+    setCurrentState(QVariantMap());
+}
+
+void ModelActionRootState::updateActionState()
+{
+    if (m_menu && m_menu->rowCount() > 0) {
+        ActionStateParser* oldParser = m_menu->actionStateParser();
+        m_menu->setActionStateParser(&m_parser);
+
+        QVariantMap state = m_menu->get(0, "actionState").toMap();
+
+        m_menu->setActionStateParser(oldParser);
+
+        setCurrentState(state);
+    } else {
+        setCurrentState(QVariantMap());
+    }
+}

=== added file 'plugins/Unity/Indicators/modelactionrootstate.h'
--- plugins/Unity/Indicators/modelactionrootstate.h	1970-01-01 00:00:00 +0000
+++ plugins/Unity/Indicators/modelactionrootstate.h	2014-12-17 11:19:14 +0000
@@ -0,0 +1,61 @@
+/*
+ * Copyright 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 as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ *
+ * Authors:
+ *      Nick Dedekind <[e-mail]>
+ */
+
+#ifndef MODELACTIONROOTSTATE_H
+#define MODELACTIONROOTSTATE_H
+
+#include "unityindicatorsglobal.h"
+
+#include "rootstateparser.h"
+
+class UnityMenuModel;
+
+class UNITYINDICATORS_EXPORT ModelActionRootState : public RootStateObject
+{
+    Q_OBJECT
+    Q_PROPERTY(UnityMenuModel* menu READ menu WRITE setMenu NOTIFY menuChanged)
+public:
+    ModelActionRootState(QObject *parent = 0);
+    virtual ~ModelActionRootState();
+
+    UnityMenuModel* menu() const;
+    void setMenu(UnityMenuModel* menu);
+
+    int index() const;
+    void setIndex(int index);
+
+    bool valid() const override;
+
+Q_SIGNALS:
+    void menuChanged();
+    void indexChanged();
+
+private Q_SLOTS:
+    void onModelRowsAdded(const QModelIndex& parent, int start, int end);
+    void onModelRowsRemoved(const QModelIndex& parent, int start, int end);
+    void onModelDataChanged(const QModelIndex& topLeft, const QModelIndex& bottomRight, const QVector<int>&);
+    void reset();
+
+private:
+    void updateActionState();
+
+    UnityMenuModel* m_menu;
+};
+
+#endif // MODELACTIONROOTSTATE_H

=== modified file 'plugins/Unity/Indicators/plugin.cpp'
--- plugins/Unity/Indicators/plugin.cpp	2014-10-07 10:28:59 +0000
+++ plugins/Unity/Indicators/plugin.cpp	2014-12-17 11:19:14 +0000
@@ -24,12 +24,13 @@
 #include "plugin.h"
 
 // local
+#include "actionrootstate.h"
 #include "indicators.h"
 #include "indicatorsmanager.h"
 #include "indicatorsmodel.h"
 #include "menucontentactivator.h"
+#include "modelactionrootstate.h"
 #include "modelprinter.h"
-#include "rootactionstate.h"
 #include "sharedunitymenumodel.h"
 #include "unitymenumodelcache.h"
 #include "unitymenumodelstack.h"
@@ -51,7 +52,8 @@
     qmlRegisterType<IndicatorsModel>(uri, 0, 1, "IndicatorsModel");
     qmlRegisterType<MenuContentActivator>(uri, 0, 1, "MenuContentActivator");
     qmlRegisterType<UnityMenuModelStack>(uri, 0, 1, "UnityMenuModelStack");
-    qmlRegisterType<RootActionState>(uri, 0, 1, "RootActionState");
+    qmlRegisterType<ModelActionRootState>(uri, 0, 1, "ModelActionRootState");
+    qmlRegisterType<ActionRootState>(uri, 0, 1, "ActionRootState");
     qmlRegisterType<ModelPrinter>(uri, 0, 1, "ModelPrinter");
     qmlRegisterType<VisibleIndicatorsModel>(uri, 0, 1, "VisibleIndicatorsModel");
     qmlRegisterType<SharedUnityMenuModel>(uri, 0, 1, "SharedUnityMenuModel");

=== renamed file 'plugins/Unity/Indicators/rootactionstate.cpp' => 'plugins/Unity/Indicators/rootstateparser.cpp'
--- plugins/Unity/Indicators/rootactionstate.cpp	2014-10-20 12:38:16 +0000
+++ plugins/Unity/Indicators/rootstateparser.cpp	2014-12-17 11:19:14 +0000
@@ -12,155 +12,18 @@
  *
  * 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/>.
- *
- * Authors:
- *      Nick Dedekind <[e-mail]>
  */
 
-#include "rootactionstate.h"
-#include "indicators.h"
-
-#include <unitymenumodel.h>
-#include <QVariant>
-#include <QIcon>
+#include "rootstateparser.h"
 
 extern "C" {
 #include <glib.h>
 #include <gio/gio.h>
 }
 
-RootActionState::RootActionState(QObject *parent)
-    : ActionStateParser(parent),
-      m_menu(nullptr)
-{
-}
-
-RootActionState::~RootActionState()
-{
-}
-
-UnityMenuModel* RootActionState::menu() const
-{
-    return m_menu;
-}
-
-void RootActionState::setMenu(UnityMenuModel* menu)
-{
-    if (m_menu != menu) {
-        if (m_menu) {
-            m_menu->disconnect(this);
-        }
-        m_menu = menu;
-
-        if (m_menu) {
-            connect(m_menu, SIGNAL(rowsInserted(const QModelIndex&, int, int)), SLOT(onModelRowsAdded(const QModelIndex&, int, int)));
-            connect(m_menu, SIGNAL(rowsRemoved(const QModelIndex&, int, int)), SLOT(onModelRowsRemoved(const QModelIndex&, int, int)));
-            connect(m_menu, SIGNAL(dataChanged(const QModelIndex&, const QModelIndex&, const QVector<int>&)), SLOT(onModelDataChanged(const QModelIndex&, const QModelIndex&, const QVector<int>&)));
-
-            connect(m_menu, SIGNAL(destroyed()), SLOT(reset()));
-        }
-        updateActionState();
-        Q_EMIT menuChanged();
-    }
-}
-
-void RootActionState::onModelRowsAdded(const QModelIndex& parent, int start, int end)
-{
-    Q_UNUSED(parent);
-    if (start == 0 && end >= 0) {
-        updateActionState();
-    }
-}
-
-void RootActionState::onModelRowsRemoved(const QModelIndex& parent, int start, int end)
-{
-    Q_UNUSED(parent);
-    if (start == 0 && end >= 0) {
-        updateActionState();
-    }
-}
-
-void RootActionState::onModelDataChanged(const QModelIndex& topLeft, const QModelIndex& bottomRight, const QVector<int>& roles)
-{
-    Q_UNUSED(roles);
-    if (!topLeft.isValid() || !bottomRight.isValid()) {
-        return;
-    }
-
-    if (topLeft.row() <= 0 && bottomRight.row() >= 0) {
-        updateActionState();
-    }
-}
-
-void RootActionState::reset()
-{
-    m_cachedState.clear();
-    m_menu = nullptr;
-
-    Q_EMIT menuChanged();
-    Q_EMIT updated();
-}
-
-void RootActionState::updateActionState()
-{
-    if (m_menu && m_menu->rowCount() > 0) {
-        ActionStateParser* oldParser = m_menu->actionStateParser();
-        m_menu->setActionStateParser(this);
-
-        m_cachedState = m_menu->get(0, "actionState").toMap();
-
-        m_menu->setActionStateParser(oldParser);
-    } else {
-        m_cachedState.clear();
-    }
-    Q_EMIT updated();
-}
-
-bool RootActionState::isValid() const
-{
-    return m_menu && m_menu->rowCount() > 0;
-}
-
-QString RootActionState::title() const
-{
-    if (!isValid()) return QString();
-
-    return m_cachedState.value("title", QVariant::fromValue(QString())).toString();
-}
-
-QString RootActionState::leftLabel() const
-{
-    if (!isValid()) return QString();
-
-    return m_cachedState.value("pre-label", QVariant::fromValue(QString())).toString();
-}
-
-QString RootActionState::rightLabel() const
-{
-    if (!isValid()) return QString();
-
-    return m_cachedState.value("label", QVariant::fromValue(QString())).toString();
-}
-
-QStringList RootActionState::icons() const
-{
-    if (!isValid()) return QStringList();
-
-    return m_cachedState.value("icons", QVariant::fromValue(QStringList())).toStringList();
-}
-
-QString RootActionState::accessibleName() const
-{
-    if (!isValid()) return QString();
-
-    return m_cachedState.value("accessible-desc", QVariant::fromValue(QString())).toString();
-}
-
-bool RootActionState::indicatorVisible() const
-{
-    if (!isValid()) return false;
-
-    return m_cachedState.value("visible", QVariant::fromValue(true)).toBool();
+RootStateParser::RootStateParser(QObject* parent)
+    : ActionStateParser(parent)
+{
 }
 
 static QString iconUri(GIcon *icon)
@@ -209,7 +72,7 @@
     return uri;
 }
 
-QVariant RootActionState::toQVariant(GVariant* state) const
+QVariant RootStateParser::toQVariant(GVariant* state) const
 {
     if (!state) {
         return QVariant();
@@ -296,3 +159,73 @@
     }
     return ActionStateParser::toQVariant(state);
 }
+
+
+RootStateObject::RootStateObject(QObject* parent)
+    : QObject(parent)
+{
+}
+
+QString RootStateObject::title() const
+{
+    if (!valid()) return QString();
+
+    return m_currentState.value("title", QVariant::fromValue(QString())).toString();
+}
+
+QString RootStateObject::leftLabel() const
+{
+    if (!valid()) return QString();
+
+    return m_currentState.value("pre-label", QVariant::fromValue(QString())).toString();
+}
+
+QString RootStateObject::rightLabel() const
+{
+    if (!valid()) return QString();
+
+    return m_currentState.value("label", QVariant::fromValue(QString())).toString();
+}
+
+QStringList RootStateObject::icons() const
+{
+    if (!valid()) return QStringList();
+
+    return m_currentState.value("icons", QVariant::fromValue(QStringList())).toStringList();
+}
+
+QString RootStateObject::accessibleName() const
+{
+    if (!valid()) return QString();
+
+    return m_currentState.value("accessible-desc", QVariant::fromValue(QString())).toString();
+}
+
+bool RootStateObject::indicatorVisible() const
+{
+    if (!valid()) return false;
+
+    return m_currentState.value("visible", QVariant::fromValue(true)).toBool();
+}
+
+void RootStateObject::setCurrentState(const QVariantMap& newState)
+{
+    QString oldTitle = title();
+    QString oldLeftLabel = leftLabel();
+    QString oldRightLabel = rightLabel();
+    QStringList oldIcons = icons();
+    QString oldAccessibleName = accessibleName();
+    bool oldIndicatorVisible = indicatorVisible();
+
+    if (m_currentState != newState) {
+        m_currentState = newState;
+        Q_EMIT updated();
+
+        if (oldTitle != title()) Q_EMIT titleChanged();
+        if (oldLeftLabel != leftLabel()) Q_EMIT leftLabelChanged();
+        if (oldRightLabel != rightLabel()) Q_EMIT rightLabelChanged();
+        if (oldIcons != icons()) Q_EMIT iconsChanged();
+        if (oldAccessibleName != accessibleName()) Q_EMIT accessibleNameChanged();
+        if (oldIndicatorVisible != indicatorVisible()) Q_EMIT indicatorVisibleChanged();
+    }
+}

=== renamed file 'plugins/Unity/Indicators/rootactionstate.h' => 'plugins/Unity/Indicators/rootstateparser.h'
--- plugins/Unity/Indicators/rootactionstate.h	2014-10-20 12:38:16 +0000
+++ plugins/Unity/Indicators/rootstateparser.h	2014-12-17 11:19:14 +0000
@@ -12,26 +12,27 @@
  *
  * 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/>.
- *
- * Authors:
- *      Nick Dedekind <[e-mail]>
  */
 
-#ifndef ROOTACTIONSTATE_H
-#define ROOTACTIONSTATE_H
+#ifndef ROOTSTATEPARSER_H
+#define ROOTSTATEPARSER_H
 
 #include "unityindicatorsglobal.h"
 
 #include <actionstateparser.h>
 
-class UnityMenuModel;
+class UNITYINDICATORS_EXPORT RootStateParser : public ActionStateParser
+{
+public:
+    RootStateParser(QObject* parent = nullptr);
+    virtual QVariant toQVariant(GVariant* state) const override;
+};
 
-class UNITYINDICATORS_EXPORT RootActionState : public ActionStateParser
+class UNITYINDICATORS_EXPORT RootStateObject : public QObject
 {
     Q_OBJECT
-    Q_PROPERTY(UnityMenuModel* menu READ menu WRITE setMenu NOTIFY menuChanged)
 
-    Q_PROPERTY(bool valid READ isValid NOTIFY validChanged)
+    Q_PROPERTY(bool valid READ valid NOTIFY validChanged)
     Q_PROPERTY(QString title READ title NOTIFY titleChanged)
     Q_PROPERTY(QString leftLabel READ leftLabel NOTIFY leftLabelChanged)
     Q_PROPERTY(QString rightLabel READ rightLabel NOTIFY rightLabelChanged)
@@ -39,16 +40,10 @@
     Q_PROPERTY(QString accessibleName READ accessibleName NOTIFY accessibleNameChanged)
     Q_PROPERTY(bool indicatorVisible READ indicatorVisible NOTIFY indicatorVisibleChanged)
 public:
-    RootActionState(QObject *parent = 0);
-    virtual ~RootActionState();
-
-    UnityMenuModel* menu() const;
-    void setMenu(UnityMenuModel* menu);
-
-    int index() const;
-    void setIndex(int index);
-
-    bool isValid() const;
+    RootStateObject(QObject* parent = 0);
+
+    virtual bool valid() const = 0;
+
     QString title() const;
     QString leftLabel() const;
     QString rightLabel() const;
@@ -56,15 +51,12 @@
     QString accessibleName() const;
     bool indicatorVisible() const;
 
-    // from ActionStateParser
-    virtual QVariant toQVariant(GVariant* state) const override;
+    QVariantMap currentState() const { return m_currentState; }
+    void setCurrentState(const QVariantMap& currentState);
 
 Q_SIGNALS:
     void updated();
 
-    void menuChanged();
-    void indexChanged();
-
     void validChanged();
     void titleChanged();
     void leftLabelChanged();
@@ -73,17 +65,9 @@
     void accessibleNameChanged();
     void indicatorVisibleChanged();
 
-private Q_SLOTS:
-    void onModelRowsAdded(const QModelIndex& parent, int start, int end);
-    void onModelRowsRemoved(const QModelIndex& parent, int start, int end);
-    void onModelDataChanged(const QModelIndex& topLeft, const QModelIndex& bottomRight, const QVector<int>&);
-    void reset();
-
-private:
-    void updateActionState();
-
-    UnityMenuModel* m_menu;
-    QVariantMap m_cachedState;
+protected:
+    RootStateParser m_parser;
+    QVariantMap m_currentState;
 };
 
-#endif // ROOTACTIONSTATE_H
+#endif // ROOTSTATEPARSER_H

=== modified file 'qml/Greeter/Clock.qml'
--- qml/Greeter/Clock.qml	2014-11-21 19:23:57 +0000
+++ qml/Greeter/Clock.qml	2014-12-17 11:19:14 +0000
@@ -53,7 +53,7 @@
         menuObjectPath: clock.visible ? "/com/canonical/indicator/datetime/phone" : ""
     }
 
-    Indicators.RootActionState {
+    Indicators.ModelActionRootState {
         menu: timeModel.model
         onUpdated: {
             if (timeLabel.text != rightLabel) {

=== modified file 'qml/Panel/IndicatorItem.qml'
--- qml/Panel/IndicatorItem.qml	2014-11-21 15:01:59 +0000
+++ qml/Panel/IndicatorItem.qml	2014-12-17 11:19:14 +0000
@@ -45,14 +45,6 @@
         onClicked: parent.clicked()
     }
 
-    // FIXME: For now we will enable led indicator support only for messaging indicator
-    // in the future we should export a led API insted of doing that,
-    Loader {
-        id: indicatorLed
-        // only load source Component if the icons contains the new message icon
-        source: (root.icons && (String(root.icons).indexOf("indicator-messages-new") != -1)) ? Qt.resolvedUrl("Indicators/IndicatorsLight.qml") : ""
-    }
-
     Item {
         id: mainItems
         anchors.centerIn: parent

=== modified file 'qml/Panel/Indicators/IndicatorBase.qml'
--- qml/Panel/Indicators/IndicatorBase.qml	2014-10-17 14:55:35 +0000
+++ qml/Panel/Indicators/IndicatorBase.qml	2014-12-17 11:19:14 +0000
@@ -40,7 +40,7 @@
         menuObjectPath: indicatorItem.menuObjectPath
     }
 
-    RootActionState {
+    ModelActionRootState {
         id: rootAction
         menu: menuModel ? menuModel : null
         onUpdated: indicatorItem.rootActionStateChanged()

=== modified file 'qml/Panel/Indicators/IndicatorsLight.qml'
--- qml/Panel/Indicators/IndicatorsLight.qml	2014-08-05 14:31:12 +0000
+++ qml/Panel/Indicators/IndicatorsLight.qml	2014-12-17 11:19:14 +0000
@@ -20,16 +20,34 @@
 import QtQuick 2.0
 import Powerd 0.1
 import Lights 0.1
+import QMenuModel 0.1 as QMenuModel
+import Unity.Indicators 0.1 as Indicators
 
 QtObject {
     id: root
 
+    property var _actionGroup: QMenuModel.QDBusActionGroup {
+        busType: 1
+        busName: "com.canonical.indicator.messages"
+        objectPath: "/com/canonical/indicator/messages"
+    }
+
+    property var _rootState: Indicators.ActionRootState {
+        actionGroup: _actionGroup
+        actionName: "messages"
+        Component.onCompleted: actionGroup.start()
+
+        property bool hasMessages: valid && (String(icons).indexOf("indicator-messages-new") != -1)
+    }
+
     Component.onDestruction: Lights.state = Lights.Off
 
     // QtObject does not have children
     property var _binding: Binding {
         target: Lights
         property: "state"
-        value: (Powerd.status === Powerd.Off) ? Lights.On : Lights.Off
+        value: {
+            return (Powerd.status === Powerd.Off && _rootState.hasMessages) ? Lights.On : Lights.Off
+        }
     }
 }

=== modified file 'qml/Panel/IndicatorsMenu.qml'
--- qml/Panel/IndicatorsMenu.qml	2014-10-21 19:32:03 +0000
+++ qml/Panel/IndicatorsMenu.qml	2014-12-17 11:19:14 +0000
@@ -18,6 +18,7 @@
 import Ubuntu.Components 1.1
 import Ubuntu.Gestures 0.1
 import "../Components"
+import "Indicators"
 
 Showable {
     id: root
@@ -60,6 +61,10 @@
     onUnitProgressChanged: d.updateState()
     clip: root.partiallyOpened
 
+    IndicatorsLight {
+        id: indicatorLights
+    }
+
     // eater
     MouseArea {
         anchors.fill: parent

=== modified file 'tests/mocks/CMakeLists.txt'
--- tests/mocks/CMakeLists.txt	2014-11-21 19:28:16 +0000
+++ tests/mocks/CMakeLists.txt	2014-12-17 11:19:14 +0000
@@ -32,6 +32,7 @@
 add_subdirectory(GSettings.1.0)
 add_subdirectory(libusermetrics)
 add_subdirectory(LightDM)
+add_subdirectory(Lights)
 add_subdirectory(MeeGo)
 add_subdirectory(Powerd)
 add_subdirectory(QMenuModel)

=== added directory 'tests/mocks/Lights'
=== added file 'tests/mocks/Lights/CMakeLists.txt'
--- tests/mocks/Lights/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/CMakeLists.txt	2014-12-17 11:19:14 +0000
@@ -0,0 +1,8 @@
+add_library(MockLights-qml MODULE
+    plugin.cpp
+    Lights.cpp
+    )
+
+qt5_use_modules(MockLights-qml Qml Gui)
+
+add_unity8_mock(Lights 0.1 Lights TARGETS MockLights-qml)

=== added file 'tests/mocks/Lights/Lights.cpp'
--- tests/mocks/Lights/Lights.cpp	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/Lights.cpp	2014-12-17 11:19:14 +0000
@@ -0,0 +1,84 @@
+/*
+ * Copyright (C) 2014 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#include "Lights.h"
+
+#include <QDebug>
+
+Lights::Lights(QObject* parent)
+  : QObject(parent),
+    m_color("blue"),
+    m_state(Lights::Off),
+    m_onMs(1000),
+    m_offMs(3000)
+{
+}
+
+Lights::~Lights()
+{
+}
+
+void Lights::setState(Lights::State newState)
+{
+    if (m_state != newState) {
+        m_state = newState;
+        Q_EMIT stateChanged(m_state);
+    }
+}
+
+Lights::State Lights::state() const
+{
+    return m_state;
+}
+
+void Lights::setColor(const QColor &color)
+{
+    if (m_color != color) {
+        m_color = color;
+        Q_EMIT colorChanged(m_color);
+    }
+}
+
+QColor Lights::color() const
+{
+    return m_color;
+}
+
+int Lights::onMillisec() const
+{
+    return m_onMs;
+}
+
+void Lights::setOnMillisec(int onMs)
+{
+    if (m_onMs != onMs) {
+        m_onMs = onMs;
+        Q_EMIT onMillisecChanged(m_onMs);
+    }
+}
+
+int Lights::offMillisec() const
+{
+    return m_offMs;
+}
+
+void Lights::setOffMillisec(int offMs)
+{
+    if (m_offMs != offMs) {
+        m_offMs = offMs;
+        Q_EMIT offMillisecChanged(m_offMs);
+    }
+}

=== added file 'tests/mocks/Lights/Lights.h'
--- tests/mocks/Lights/Lights.h	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/Lights.h	2014-12-17 11:19:14 +0000
@@ -0,0 +1,66 @@
+/*
+ * Copyright (C) 2014 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#ifndef UNITY_MOCK_LIGHTS_H
+#define UNITY_MOCK_LIGHTS_H
+
+#include <QtCore/QObject>
+#include <QtGui/QColor>
+
+class Lights: public QObject
+{
+    Q_OBJECT
+    Q_ENUMS(State)
+    Q_PROPERTY(State state READ state  WRITE setState NOTIFY stateChanged)
+    Q_PROPERTY(QColor color READ color WRITE setColor NOTIFY colorChanged)
+    Q_PROPERTY(int onMillisec READ onMillisec WRITE setOnMillisec NOTIFY onMillisecChanged)
+    Q_PROPERTY(int offMillisec READ offMillisec WRITE setOffMillisec NOTIFY offMillisecChanged)
+
+public:
+    enum State {
+        Off,
+        On,
+    };
+
+    explicit Lights(QObject *parent = 0);
+    ~Lights();
+
+    void setState(State newState);
+    State state() const;
+
+    void setColor(const QColor &color);
+    QColor color() const;
+
+    int onMillisec() const;
+    void setOnMillisec(int onMs);
+
+    int offMillisec() const;
+    void setOffMillisec(int offMs);
+
+Q_SIGNALS:
+    void stateChanged(State newState);
+    void colorChanged(const QColor &color);
+    void onMillisecChanged(int onMs);
+    void offMillisecChanged(int offMs);
+
+private:
+    QColor m_color;
+    State m_state;
+    int m_onMs;
+    int m_offMs;
+};
+
+#endif

=== added file 'tests/mocks/Lights/Lights.qmltypes'
--- tests/mocks/Lights/Lights.qmltypes	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/Lights.qmltypes	2014-12-17 11:19:14 +0000
@@ -0,0 +1,45 @@
+import QtQuick.tooling 1.1
+
+// This file describes the plugin-supplied types contained in the library.
+// It is used for QML tooling purposes only.
+//
+// This file was auto-generated by:
+// 'qmlplugindump -notrelocatable Lights 0.1 tests/mocks'
+
+Module {
+    Component {
+        name: "Lights"
+        prototype: "QObject"
+        exports: ["Lights/Lights 0.1"]
+        isCreatable: false
+        isSingleton: true
+        exportMetaObjectRevisions: [0]
+        Enum {
+            name: "State"
+            values: {
+                "Off": 0,
+                "On": 1
+            }
+        }
+        Property { name: "state"; type: "State" }
+        Property { name: "color"; type: "QColor" }
+        Property { name: "onMillisec"; type: "int" }
+        Property { name: "offMillisec"; type: "int" }
+        Signal {
+            name: "stateChanged"
+            Parameter { name: "newState"; type: "State" }
+        }
+        Signal {
+            name: "colorChanged"
+            Parameter { name: "color"; type: "QColor" }
+        }
+        Signal {
+            name: "onMillisecChanged"
+            Parameter { name: "onMs"; type: "int" }
+        }
+        Signal {
+            name: "offMillisecChanged"
+            Parameter { name: "offMs"; type: "int" }
+        }
+    }
+}

=== added file 'tests/mocks/Lights/plugin.cpp'
--- tests/mocks/Lights/plugin.cpp	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/plugin.cpp	2014-12-17 11:19:14 +0000
@@ -0,0 +1,33 @@
+/*
+ * Copyright (C) 2012,2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#include "plugin.h"
+#include "Lights.h"
+
+#include <QtQml/qqml.h>
+
+static QObject *lights_provider(QQmlEngine *engine, QJSEngine *scriptEngine)
+{
+    Q_UNUSED(engine)
+    Q_UNUSED(scriptEngine)
+    return new Lights();
+}
+
+void PowerdPlugin::registerTypes(const char *uri)
+{
+    Q_ASSERT(uri == QLatin1String("Lights"));
+    qmlRegisterSingletonType<Lights>(uri, 0, 1, "Lights", lights_provider);
+}

=== added file 'tests/mocks/Lights/plugin.h'
--- tests/mocks/Lights/plugin.h	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/plugin.h	2014-12-17 11:19:14 +0000
@@ -0,0 +1,32 @@
+/*
+ * Copyright (C) 2012,2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+#ifndef MOCK_POWER_PLUGIN_H
+#define MOCK_POWER_PLUGIN_H
+
+#include <QtQml/QQmlEngine>
+#include <QtQml/QQmlExtensionPlugin>
+
+class PowerdPlugin : public QQmlExtensionPlugin
+{
+    Q_OBJECT
+    Q_PLUGIN_METADATA(IID "org.qt-project.Qt.QQmlExtensionInterface")
+
+public:
+    void registerTypes(const char *uri);
+};
+
+#endif

=== added file 'tests/mocks/Lights/qmldir'
--- tests/mocks/Lights/qmldir	1970-01-01 00:00:00 +0000
+++ tests/mocks/Lights/qmldir	2014-12-17 11:19:14 +0000
@@ -0,0 +1,3 @@
+module Lights
+plugin MockLights-qml
+typeinfo Lights.qmltypes

=== modified file 'tests/mocks/QMenuModel/CMakeLists.txt'
--- tests/mocks/QMenuModel/CMakeLists.txt	2014-11-14 17:47:00 +0000
+++ tests/mocks/QMenuModel/CMakeLists.txt	2014-12-17 11:19:14 +0000
@@ -4,6 +4,7 @@
 )
 
 set(QMenuModelQml_SOURCES
+  actiondata.h
   actionstateparser.cpp
   dbus-enums.h
   plugin.cpp

=== modified file 'tests/mocks/QMenuModel/QDBusActionGroup.qml'
--- tests/mocks/QMenuModel/QDBusActionGroup.qml	2014-11-21 19:28:16 +0000
+++ tests/mocks/QMenuModel/QDBusActionGroup.qml	2014-12-17 11:19:14 +0000
@@ -19,81 +19,34 @@
 
 import QtQuick 2.0
 import Ubuntu.Settings.Menus 0.1 as Menus
+import QMenuModel 0.1
 
 QtObject {
     id: actionGroup
     property int busType
     property string busName
     property string objectPath
+    property var actions: ActionData ? ActionData.data : undefined
+
+    signal dataChanged
+
     function start() {}
+
     function action(actionName) {
-        switch (actionName) {
-            case "transfer-state.queued":
-                return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Queued,
-                        'percent': 0.0
-                    }
-            }
-            case "transfer-state.running":
-                return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Running,
-                        'seconds-left': 100,
-                        'percent': 0.1
-                    }
-            }
-            case "transfer-state.paused":
-                return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Paused,
-                        'seconds-left': 100,
-                        'percent': 0.5
-                    }
-            }
-            case "transfer-state.cancelled":
-                return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Canceled,
-                        'percent': 0.4
-                    }
-            }
-            case "transfer-state.finished": return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Finished,
-                        'seconds-left': 0,
-                        'percent': 1.0
-                    }
-            }
-            case "transfer-state.error":
-                return {
-                    'valid': true,
-                    'state': {
-                        'state': Menus.TransferState.Error,
-                        'seconds-left': 100,
-                        'percent': 0.0
-                    }
-            }
-            case "gps-detection-enabled":
-            case "location-detection-enabled":
-                return Qt.createQmlObject("
-import QtQuick 2.3
-QtObject {
-    property bool valid: true
-    property bool state: false
-    function activate() {
-        state = !state;
-    }
-}", actionGroup);
-            default:
-                break;
-        }
-
-        return null;
+        return Qt.createQmlObject("
+            import QtQuick 2.3
+            import QMenuModel 0.1
+
+            QtObject {
+                signal activated
+
+                property string actionName: \"" + actionName + "\"
+                property bool valid: ActionData.data[actionName] != undefined ? ActionData.data[actionName].valid : false
+                property var state: ActionData.data[actionName] != undefined ? ActionData.data[actionName].state : undefined
+
+                function activate() {
+                    activated();
+                }
+            }", actionGroup);
     }
 }

=== added file 'tests/mocks/QMenuModel/actiondata.h'
--- tests/mocks/QMenuModel/actiondata.h	1970-01-01 00:00:00 +0000
+++ tests/mocks/QMenuModel/actiondata.h	2014-12-17 11:19:14 +0000
@@ -0,0 +1,52 @@
+/*
+ * Copyright (C) 2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ *
+ * Authors: Nick Dedekind <[e-mail]>
+ */
+
+#ifndef ACTIONDATA_H
+#define ACTIONDATA_H
+
+#include <QObject>
+#include <QVariant>
+
+typedef struct _GVariant GVariant;
+
+class Q_DECL_EXPORT ActionData : public QObject
+{
+    Q_OBJECT
+    Q_PROPERTY(QVariant data READ data WRITE setData NOTIFY dataChanged)
+public:
+    ActionData(QObject* parent = 0)
+        : QObject(parent)
+    {}
+
+    QVariant data() const { return m_data; }
+    void setData(const QVariant& data)
+    {
+        if (m_data != data) {
+            m_data = data;
+            Q_EMIT dataChanged();
+        }
+    }
+
+Q_SIGNALS:
+    void dataChanged();
+
+private:
+    QVariant m_data;
+};
+
+#endif // ACTIONDATA_H

=== modified file 'tests/mocks/QMenuModel/plugin.cpp'
--- tests/mocks/QMenuModel/plugin.cpp	2014-11-14 17:47:00 +0000
+++ tests/mocks/QMenuModel/plugin.cpp	2014-12-17 11:19:14 +0000
@@ -18,11 +18,19 @@
 
 #include "plugin.h"
 #include "unitymenumodel.h"
+#include "actiondata.h"
 #include "actionstateparser.h"
 #include "dbus-enums.h"
 
 #include <QtQml/qqml.h>
 
+static QObject* actionDataSingleton(QQmlEngine* engine, QJSEngine* scriptEngine)
+{
+  Q_UNUSED(engine);
+  Q_UNUSED(scriptEngine);
+  return new ActionData;
+}
+
 void QMenuModelPlugin::registerTypes(const char *uri)
 {
     Q_ASSERT(uri == QLatin1String("QMenuModel"));
@@ -30,4 +38,6 @@
                                           "DBus is only a namespace");
     qmlRegisterType<UnityMenuModel>(uri, 0, 1, "UnityMenuModel");
     qmlRegisterType<ActionStateParser>(uri, 0, 1, "ActionStateParser");
+
+    qmlRegisterSingletonType<ActionData>(uri, 0, 1, "ActionData", actionDataSingleton);
 }

=== added file 'tests/mocks/Unity/Indicators/ActionRootState.qml'
--- tests/mocks/Unity/Indicators/ActionRootState.qml	1970-01-01 00:00:00 +0000
+++ tests/mocks/Unity/Indicators/ActionRootState.qml	2014-12-17 11:19:14 +0000
@@ -0,0 +1,46 @@
+/*
+ * Copyright 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 as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ *
+ * Authors:
+ *      Nick Dedekind <[e-mail]>
+ */
+
+import QtQuick 2.0
+
+QtObject {
+    property var actionGroup
+    property var actionName
+
+    property var action: actionGroup ? actionGroup.actions[actionName] : undefined
+    property var state: action && action.hasOwnProperty("state") ? action.state : undefined
+
+    property bool valid: action && action.hasOwnProperty("valid") ? action.valid : false
+    property string title: state && state.hasOwnProperty("title") ? state["title"] : ""
+    property string leftLabel: state && state.hasOwnProperty("pre-label") ? state["pre-label"] :     ""
+    property string rightLabel: state && state.hasOwnProperty("label") ? state["label"] : ""
+    property var icons: state && state.hasOwnProperty("icons") ? state["icons"] : []
+    property string accessibleName: state && state.hasOwnProperty("accessible-desc") ? state["accessible-desc"] : ""
+    property bool visible: state && state.hasOwnProperty("visible") ? state["visible"] : true
+
+    signal updated
+
+    onValidChanged: updated()
+    onTitleChanged: updated()
+    onLeftLabelChanged: updated()
+    onRightLabelChanged: updated()
+    onIconsChanged: updated()
+    onAccessibleNameChanged: updated()
+    onVisibleChanged: updated()
+}

=== renamed file 'tests/mocks/Unity/Indicators/RootActionState.qml' => 'tests/mocks/Unity/Indicators/ModelActionRootState.qml'
=== modified file 'tests/mocks/Unity/Indicators/qmldir'
--- tests/mocks/Unity/Indicators/qmldir	2014-10-07 10:28:59 +0000
+++ tests/mocks/Unity/Indicators/qmldir	2014-12-17 11:19:14 +0000
@@ -2,5 +2,6 @@
 plugin IndicatorsFakeQml
 typeinfo Indicators.qmltypes
 
+ActionRootState 0.1 ActionRootState.qml
 IndicatorsModel 0.1 IndicatorsModel.qml
-RootActionState 0.1 RootActionState.qml
+ModelActionRootState 0.1 ModelActionRootState.qml

=== modified file 'tests/plugins/Unity/Indicators/rootactionstatetest.cpp'
--- tests/plugins/Unity/Indicators/rootactionstatetest.cpp	2014-08-26 08:14:44 +0000
+++ tests/plugins/Unity/Indicators/rootactionstatetest.cpp	2014-12-17 11:19:14 +0000
@@ -17,7 +17,7 @@
  *      Nick Dedekind <[e-mail]>
  */
 
-#include "rootactionstate.h"
+#include "modelactionrootstate.h"
 
 #include <unitymenumodel.h>
 #include <QtTest>
@@ -32,7 +32,7 @@
     {
         UnityMenuModel* menuModel = new UnityMenuModel();
         ActionStateParser* originalParser = menuModel->actionStateParser();
-        RootActionState* rootState = new RootActionState();
+        ModelActionRootState* rootState = new ModelActionRootState();
 
         rootState->setMenu(menuModel);
 
@@ -44,7 +44,7 @@
     void testDeleteUnityMenuModel()
     {
         UnityMenuModel* menuModel = new UnityMenuModel();
-        RootActionState* rootState = new RootActionState();
+        ModelActionRootState* rootState = new ModelActionRootState();
 
         rootState->setMenu(menuModel);
 
@@ -77,7 +77,7 @@
 
         GVariant* params = g_variant_builder_end (&builderParams);
 
-        RootActionState rootState;
+        RootStateParser rootState;
         QVariant result = rootState.toQVariant(params);
         g_variant_unref(params);
 

=== modified file 'tests/qmltests/CMakeLists.txt'
--- tests/qmltests/CMakeLists.txt	2014-12-08 13:39:01 +0000
+++ tests/qmltests/CMakeLists.txt	2014-12-17 11:19:14 +0000
@@ -79,6 +79,7 @@
 add_qml_test(Panel MenuContent ENVIRONMENT "LD_LIBRARY_PATH=${CMAKE_BINARY_DIR}/tests/mocks/QMenuModel")
 add_qml_test(Panel Panel ENVIRONMENT "LD_LIBRARY_PATH=${CMAKE_BINARY_DIR}/tests/mocks/QMenuModel")
 add_qml_test(Panel SearchIndicator)
+add_qml_test(Panel/Indicators IndicatorsLight ENVIRONMENT "LD_LIBRARY_PATH=${CMAKE_BINARY_DIR}/tests/mocks/QMenuModel")
 # These MenuItemFactory tests need the test/mocks/ to come before plugins/
 add_qml_test(Panel/Indicators MenuItemFactory IMPORT_PATHS ${CMAKE_BINARY_DIR}/tests/mocks ${qmltest_DEFAULT_IMPORT_PATHS})
 add_qml_test(Panel/Indicators MessageMenuItemFactory IMPORT_PATHS ${CMAKE_BINARY_DIR}/tests/mocks ${qmltest_DEFAULT_IMPORT_PATHS})

=== added file 'tests/qmltests/Panel/Indicators/tst_IndicatorsLight.qml'
--- tests/qmltests/Panel/Indicators/tst_IndicatorsLight.qml	1970-01-01 00:00:00 +0000
+++ tests/qmltests/Panel/Indicators/tst_IndicatorsLight.qml	2014-12-17 11:19:14 +0000
@@ -0,0 +1,129 @@
+/*
+ * Copyright 2014 Canonical Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * 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/>.
+ */
+
+import QtQuick 2.0
+import QtQuick.Layouts 1.1
+import Unity.Test 0.1 as UT
+import Unity.Indicators 0.1 as Indicators
+import Ubuntu.Components 1.1
+import Powerd 0.1
+import Lights 0.1
+import QMenuModel 0.1
+import "../../../../qml/Panel/Indicators"
+
+Item {
+    id: root
+    width: units.gu(30)
+    height: units.gu(30)
+
+    property var newMessage: {
+        "messages" : {
+            'valid': true,
+            'state': {
+                'icons': [ 'indicator-messages-new' ]
+            }
+        }
+    };
+    property var noNewMessage: {
+        "messages" : {
+            'valid': true,
+            'state': {
+                'icons': [ 'indicator-messages' ]
+            }
+        }
+    };
+
+    Component {
+        id: light
+        IndicatorsLight {}
+    }
+
+    Loader {
+        id: loader
+        sourceComponent: light
+    }
+
+    Component.onCompleted: {
+        ActionData.data = newMessage;
+        Powerd.status = Powerd.On
+    }
+
+    RowLayout {
+        anchors.fill: parent
+        anchors.margins: units.gu(1)
+
+        Item {
+            Layout.fillWidth: true
+            Layout.fillHeight: true
+
+            Rectangle {
+                width: units.gu(4)
+                height: width
+                radius: width / 2
+                anchors.centerIn: parent
+
+                color: Lights.state === Lights.On ? Lights.color : "transparent"
+
+                border.color: "black"
+                border.width: 1
+            }
+        }
+
+        ColumnLayout {
+            Layout.alignment: Qt.AlignTop
+            Layout.fillWidth: false
+            Layout.preferredWidth: units.gu(15)
+
+            Button {
+                Layout.fillWidth: true
+                text: Powerd.status === Powerd.On ? "Power Off" : "Power On"
+                onClicked: {
+                    if (Powerd.status === Powerd.On) Powerd.status = Powerd.Off;
+                    else Powerd.status = Powerd.On;
+                }
+            }
+        }
+    }
+
+    UT.UnityTestCase {
+        name: "IndicatorsLight"
+        when: windowShown
+
+        function init() {
+            // reload
+            ActionData.data = noNewMessage;
+            loader.sourceComponent = undefined;
+            loader.sourceComponent = light;
+            Powerd.status = Powerd.On
+        }
+
+        function test_LightsStatus_data() {
+            return [
+                { tag: "Powerd.On with No Message", powerd: Powerd.On, actionData: noNewMessage, expected: Lights.Off },
+                { tag: "Powerd.Off with No Message", powerd: Powerd.Off, actionData: noNewMessage, expected: Lights.Off },
+                { tag: "Powerd.On with New Message", powerd: Powerd.On, actionData: newMessage, expected: Lights.Off },
+                { tag: "Powerd.Off with New Message", powerd: Powerd.Off, actionData: newMessage, expected: Lights.On },
+            ]
+        }
+
+        function test_LightsStatus(data) {
+            Powerd.status = data.powerd;
+            ActionData.data = data.actionData;
+
+            compare(Lights.state, data.expected, "Light does not match expected value");
+        }
+    }
+}

=== modified file 'tests/qmltests/Panel/Indicators/tst_MenuItemFactory.qml'
--- tests/qmltests/Panel/Indicators/tst_MenuItemFactory.qml	2014-10-09 17:06:00 +0000
+++ tests/qmltests/Panel/Indicators/tst_MenuItemFactory.qml	2014-12-17 11:19:14 +0000
@@ -17,6 +17,7 @@
 import QtQuick 2.0
 import QtTest 1.0
 import Unity.Test 0.1 as UT
+import Ubuntu.Settings.Menus 0.1 as Menus
 import QMenuModel 0.1
 import Utils 0.1 as Utils
 import "../../../../qml/Panel/Indicators"
@@ -448,9 +449,59 @@
         }
 
         function test_create_transferMenu(data) {
+            ActionData.data = {
+                "transfer-state.queued": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Queued,
+                        'percent': data.progress
+                    }
+                },
+                "transfer-state.running": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Running,
+                        'seconds-left': 100,
+                        'percent': data.progress
+                    }
+                },
+                "transfer-state.paused": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Paused,
+                        'seconds-left': 100,
+                        'percent': data.progress
+                    }
+                },
+                "transfer-state.cancelled": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Canceled,
+                        'percent': data.progress
+                    }
+                },
+                "transfer-state.finished": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Finished,
+                        'seconds-left': 0,
+                        'percent': data.progress
+                    }
+                },
+                "transfer-state.error": {
+                    'valid': true,
+                    'state': {
+                        'state': Menus.TransferState.Error,
+                        'seconds-left': 100,
+                        'percent': data.progress
+                    }
+                }
+            };
+
             menuData.type = "com.canonical.indicator.transfer";
             menuData.label = data.label;
             menuData.sensitive = data.enabled;
+            menuData.isToggled = data.active;
             menuData.icon = data.icon;
             menuData.ext = {
                 'xCanonicalUid': data.tag
@@ -464,7 +515,7 @@
             compare(loader.item.text, data.label, "Label does not match data");
             compare(loader.item.iconSource, data.icon, "Icon does not match data");
             compare(loader.item.enabled, data.enabled, "Enabled does not match data");
-            compare(loader.item.progress, data.progress, "Icon does not match data");
+            compare(loader.item.progress, data.progress, "Progress does not match data");
             compare(loader.item.active, data.active, "Active does not match data");
             compare(loader.item.stateText, data.stateText, "State text does not match data");
         }

=== modified file 'tests/qmltests/Wizard/tst_Wizard.qml'
--- tests/qmltests/Wizard/tst_Wizard.qml	2014-12-02 17:43:25 +0000
+++ tests/qmltests/Wizard/tst_Wizard.qml	2014-12-17 11:19:14 +0000
@@ -18,6 +18,7 @@
 import QtTest 1.0
 import AccountsService 0.1
 import MeeGo.QOfono 0.2
+import QMenuModel 0.1
 import Ubuntu.Components 1.1
 import Ubuntu.SystemSettings.SecurityPrivacy 1.0
 import Unity.Test 0.1 as UT
@@ -58,6 +59,16 @@
         signalName: "setSecurityCalled"
     }
 
+    SignalSpy {
+        id: activateLocationSpy
+        signalName: "activated"
+    }
+
+    SignalSpy {
+        id: activateGPSSpy
+        signalName: "activated"
+    }
+
     function setup() {
         AccountsService.hereEnabled = false;
         AccountsService.hereLicensePath = Qt.resolvedUrl("licenses");
@@ -68,6 +79,19 @@
 
         updateSessionLanguageSpy.clear();
         setSecuritySpy.clear();
+        activateLocationSpy.clear();
+        activateGPSSpy.clear();
+
+        ActionData.data = {
+            "location-detection-enabled": {
+                'valid': true,
+                'state': false
+            },
+            "gps-detection-enabled": {
+                'valid': true,
+                'state': false
+            }
+        };
     }
 
     Component.onCompleted: {
@@ -360,8 +384,8 @@
             var nopeCheck = findChild(page, "nopeCheck");
 
             var locationActionGroup = findInvisibleChild(page, "locationActionGroup");
-            tryCompare(locationActionGroup.location, "state", false);
-            tryCompare(locationActionGroup.gps, "state", false);
+            activateLocationSpy.target = locationActionGroup.location;
+            activateGPSSpy.target = locationActionGroup.gps;
 
             tap(gpsCheck);
             tryCompare(gpsCheck, "checked", true);
@@ -370,8 +394,8 @@
 
             tap(findChild(page, "forwardButton"));
             tryCompare(AccountsService, "hereEnabled", false);
-            tryCompare(locationActionGroup.location, "state", true);
-            tryCompare(locationActionGroup.gps, "state", true);
+            tryCompare(activateLocationSpy, "count", 1)
+            tryCompare(activateGPSSpy, "count", 1)
         }
 
         function test_locationNope() {
@@ -381,8 +405,8 @@
             var nopeCheck = findChild(page, "nopeCheck");
 
             var locationActionGroup = findInvisibleChild(page, "locationActionGroup");
-            tryCompare(locationActionGroup.location, "state", false);
-            tryCompare(locationActionGroup.gps, "state", false);
+            activateLocationSpy.target = locationActionGroup.location;
+            activateGPSSpy.target = locationActionGroup.gps;
 
             tap(nopeCheck);
             tryCompare(gpsCheck, "checked", false);
@@ -391,8 +415,8 @@
 
             tap(findChild(page, "forwardButton"));
             tryCompare(AccountsService, "hereEnabled", false);
-            tryCompare(locationActionGroup.location, "state", false);
-            tryCompare(locationActionGroup.gps, "state", false);
+            tryCompare(activateLocationSpy, "count", 0)
+            tryCompare(activateGPSSpy, "count", 0)
         }
 
         function test_locationHere() {
@@ -402,8 +426,8 @@
             var nopeCheck = findChild(page, "nopeCheck");
 
             var locationActionGroup = findInvisibleChild(page, "locationActionGroup");
-            tryCompare(locationActionGroup.location, "state", false);
-            tryCompare(locationActionGroup.gps, "state", false);
+            activateLocationSpy.target = locationActionGroup.location;
+            activateGPSSpy.target = locationActionGroup.gps;
 
             // no tap because HERE is the default
             tryCompare(gpsCheck, "checked", false);
@@ -412,8 +436,8 @@
 
             tap(findChild(page, "forwardButton"));
             tryCompare(AccountsService, "hereEnabled", true);
-            tryCompare(locationActionGroup.location, "state", true);
-            tryCompare(locationActionGroup.gps, "state", true);
+            tryCompare(activateLocationSpy, "count", 1)
+            tryCompare(activateGPSSpy, "count", 1)
         }
 
         function test_locationHereTerms() {