Merge lp:~cimi/unity8/fix-open-new-scope-from-tmp into lp:unity8

[Andrea Cimitan]

Fix temp scopes opening temp scopes, correctly close previously opened temp scope with its preview

* Are there any related MPs required for this MP to build/function as expected? Please list.
https://code.launchpad.net/~stolowski/unity-scopes-shell/fix-temp-scopes/+merge/248235
https://code.launchpad.net/~stolowski/unity-api/scopes-close-scope/+merge/248725
 * 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

[Albert Astals Cid (aacid)]

I don't understand the MR, you're adding to fake_scopes three methods that you don't use and you're removing the code in closeScopes that checks that the scope you closed is indeed a scope that was opened.

Also what's the point of having the scopeThatOpenedScope variable now that you can simply use scopes.closeScope?

review: Needs Information

[Andrea Cimitan (cimi)]

> I don't understand the MR, you're adding to fake_scopes three methods that you
> don't use

future proof, if we wanted to test those, since they added API for this branch

and you're removing the code in closeScopes that checks that the
> scope you closed is indeed a scope that was opened.
>
that code is broken... m_openScope is never updated

> Also what's the point of having the scopeThatOpenedScope variable now that you
> can simply use scopes.closeScope?

I can switch to that too... but didn't want to move far from the previous implementation

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1552
http://jenkins.qa.ubuntu.com/job/unity8-ci/5235/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1161
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/399
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/400
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/400
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/1024
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1159
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1159/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/17754

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

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

> > I don't understand the MR, you're adding to fake_scopes three methods that
> you
> > don't use
>
> future proof, if we wanted to test those, since they added API for this branch

Hmmm, no, the only added api for the branch is
Q_INVOKABLE void closeScope(unity::shell::scopes::ScopeInterface* scope);
the others are private methods you'll never see

>
> and you're removing the code in closeScopes that checks that the
> > scope you closed is indeed a scope that was opened.
> >
> that code is broken... m_openScope is never updated

Broken where? after your changes? may be, then what about fixing it?

> > Also what's the point of having the scopeThatOpenedScope variable now that
> you
> > can simply use scopes.closeScope?
>
> I can switch to that too... but didn't want to move far from the previous
> implementation

Well, you made Pawel to add this new method and now you're not using it?

Also, why are you using
  dashTempScopeItem.scope.openScope(scopes.getScopeFromAll("MockScope9"));
?
Can you please make it work with a click that calls activate in the scope and thus ends up in Scope::activate that emits openScope?
That way we're properly exercising all the parts of the code

[Michał Sawicz (saviq)]

You need to depend on the new unity-api version, bump the API requirement in tests/mocks/Unity/CMakeLists.txt and bump unity-scopes-impl dep in debian/control.

review: Needs Fixing

[Albert Astals Cid (aacid)]

Please drop scopeThatOpenedScope since we don't need it anymore now that we can just close directly from Scopes

review: Needs Fixing

[Albert Astals Cid (aacid)]

Seems like you don't need to store m_scopes in PreviewStack and you can just pass it down to the PreviewModel in the constructor, no?

review: Needs Fixing

[Albert Astals Cid (aacid)]

You have declared
  unity::shell::scopes::ScopeInterface* findTempScope(QString const& id) const;
but not implemented it, just kill it?

review: Needs Fixing

[Albert Astals Cid (aacid)]

Please mark closeScope in Scopes with override to mark it implements the declaration of the same function in the internface

review: Needs Fixing

[Andrea Cimitan (cimi)]

Done

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1557
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/fix-open-new-scope-from-tmp/+merge/249471/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/5303/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1323/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/467/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/468/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/468/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1321/console

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1558
http://jenkins.qa.ubuntu.com/job/unity8-ci/5305/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1328/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/469/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/470/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/470/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1326/console

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

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

Can you please turn

if (!m_tempScopes.contains(scope)) {
   m_tempScopes.insert(scope);
 }

into just
   m_tempScopes.insert(scope);

It's a QSet after all, inserting won't produce a duplicate anyway

review: Needs Fixing

[Albert Astals Cid (aacid)]

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

 * Did CI run pass?
No the jenkins one because it needs new unity-api, ran locally and found nothing.

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

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1559
http://jenkins.qa.ubuntu.com/job/unity8-ci/5308/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1333/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/472/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/473/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/473/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1331/console

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

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

Somehow i missed a qml test failure, testPreview segfaults, please fix

review: Needs Fixing

[lp:~cimi/unity8/fix-open-new-scope-from-tmp]

1563. By Andrea Cimitan on 2015-02-16

Fix testPreview

[Albert Astals Cid (aacid)]

It's good again

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1563
http://jenkins.qa.ubuntu.com/job/unity8-ci/5323/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1371/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/487/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/488/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/488/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1369/console

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

review: Needs Fixing (continuous-integration)

=== modified file 'debian/control'
--- debian/control	2015-02-11 17:12:22 +0000
+++ debian/control	2015-02-16 12:09:57 +0000
@@ -27,7 +27,7 @@
                libqmenumodel-dev (>= 0.2.9),
                libqt5xmlpatterns5-dev,
                libsystemsettings-dev,
-               libunity-api-dev (>= 7.95),
+               libunity-api-dev (>= 7.96),
                libusermetricsoutput1-dev,
                libxcb1-dev,
                pkg-config,
@@ -126,7 +126,7 @@
          unity-application-impl-4,
          unity-notifications-impl-3,
          unity-plugin-scopes | unity-scopes-impl,
-         unity-scopes-impl-4,
+         unity-scopes-impl-6,
          unity8-fake-env | unity-application-impl,
          ${misc:Depends},
 Breaks: unity8 (<< 7.86),

=== modified file 'qml/Dash/Dash.qml'
--- qml/Dash/Dash.qml	2015-01-12 08:31:10 +0000
+++ qml/Dash/Dash.qml	2015-02-16 12:09:57 +0000
@@ -122,7 +122,6 @@
             dash.setCurrentScope(scopeId, true, false);
         }
         onOpenScope: {
-            scopeItem.scopeThatOpenedScope = currentScope;
             scopeItem.scope = scope;
             x = -width;
         }
@@ -130,7 +129,7 @@
             UbuntuNumberAnimation {
                 onRunningChanged: {
                     if (!running && dashContent.x == 0) {
-                        scopeItem.scopeThatOpenedScope.closeScope(scopeItem.scope);
+                        scopes.closeScope(scopeItem.scope);
                         scopeItem.scope = null;
                     }
                 }
@@ -187,7 +186,6 @@
             onOpenScope: {
                 bottomEdgeController.enableAnimation = true;
                 bottomEdgeController.progress = 0;
-                scopeItem.scopeThatOpenedScope = scopesList.scope;
                 scopeItem.scope = scope;
                 dashContent.x = -dashContent.width;
             }
@@ -199,8 +197,7 @@
         }
     }
 
-    DashBackground
-    {
+    DashBackground {
         anchors.fill: scopeItem
         visible: scopeItem.visible
     }
@@ -209,8 +206,6 @@
         id: scopeItem
         objectName: "dashTempScopeItem"
 
-        property var scopeThatOpenedScope: null
-
         x: dashContent.x + width
         y: dashContent.y
         width: parent.width
@@ -229,7 +224,10 @@
                 dashContent.gotoScope(scopeId);
             }
             onOpenScope: {
-                dashContent.openScope(scope);
+                scopeItem.closePreview();
+                var oldScope = scopeItem.scope;
+                scopeItem.scope = scope;
+                scopes.closeScope(oldScope);
             }
         }
     }

=== modified file 'qml/Dash/DashContent.qml'
--- qml/Dash/DashContent.qml	2015-01-05 15:08:04 +0000
+++ qml/Dash/DashContent.qml	2015-02-16 12:09:57 +0000
@@ -84,8 +84,7 @@
 
         set_current_index = undefined;
 
-        if (dashContentList.count > index)
-        {
+        if (dashContentList.count > index) {
             dashContentList.currentIndex = index
 
             if (reset) {

=== modified file 'tests/mocks/Unity/CMakeLists.txt'
--- tests/mocks/Unity/CMakeLists.txt	2015-02-04 13:22:27 +0000
+++ tests/mocks/Unity/CMakeLists.txt	2015-02-16 12:09:57 +0000
@@ -6,7 +6,7 @@
 add_subdirectory(DashCommunicator)
 
 pkg_search_module(GOBJECT gobject-2.0 REQUIRED)
-pkg_check_modules(SCOPES_API REQUIRED unity-shell-scopes=5)
+pkg_check_modules(SCOPES_API REQUIRED unity-shell-scopes=6)
 
 include_directories(
   ${CMAKE_CURRENT_BINARY_DIR}

=== modified file 'tests/mocks/Unity/fake_previewmodel.cpp'
--- tests/mocks/Unity/fake_previewmodel.cpp	2014-08-11 09:57:29 +0000
+++ tests/mocks/Unity/fake_previewmodel.cpp	2015-02-16 12:09:57 +0000
@@ -22,18 +22,23 @@
 #include "fake_previewmodel.h"
 
 // local
+#include "fake_scope.h"
+#include "fake_scopes.h"
 #include "fake_previewwidgetmodel.h"
 
 // Qt
 #include <QDebug>
 
-PreviewModel::PreviewModel(QObject* parent)
+PreviewModel::PreviewModel(QObject* parent, Scope* scope)
  : unity::shell::scopes::PreviewModelInterface(parent)
  , m_loaded(true)
+ , m_scope(scope)
 {
     // we have one column by default
     PreviewWidgetModel* columnModel = new PreviewWidgetModel(this);
     m_previewWidgetModels.append(columnModel);
+    connect(this, SIGNAL(triggered(QString const&, QString const&, QVariantMap const&)),
+            this, SLOT(triggeredSlot(QString const&, QString const&, QVariantMap const&)));
 }
 
 void PreviewModel::setWidgetColumnCount(int count)
@@ -80,3 +85,12 @@
         Q_EMIT loadedChanged();
     }
 }
+
+void PreviewModel::triggeredSlot(QString const&, QString const&, QVariantMap const&) {
+    if (m_scope) {
+        Scopes *scopes = dynamic_cast<Scopes*>(m_scope->parent());
+        Scope* scope = scopes->getScopeFromAll("MockScope9");
+        scopes->addTempScope(scope);
+        Q_EMIT m_scope->openScope(scope);
+    }
+}

=== modified file 'tests/mocks/Unity/fake_previewmodel.h'
--- tests/mocks/Unity/fake_previewmodel.h	2014-08-11 09:57:29 +0000
+++ tests/mocks/Unity/fake_previewmodel.h	2015-02-16 12:09:57 +0000
@@ -25,12 +25,14 @@
 
 class PreviewWidgetModel;
 
+class Scope;
+
 class PreviewModel : public unity::shell::scopes::PreviewModelInterface
 {
     Q_OBJECT
 
 public:
-    explicit PreviewModel(QObject* parent = 0);
+    explicit PreviewModel(QObject* parent = 0, Scope* scope = 0);
 
     QVariant data(const QModelIndex& index, int role = Qt::DisplayRole) const override;
     int rowCount(const QModelIndex& parent = QModelIndex()) const override;
@@ -42,9 +44,13 @@
 
     Q_INVOKABLE void setLoaded(bool); // Only available for testing
 
+private Q_SLOTS:
+    void triggeredSlot(QString const&, QString const&, QVariantMap const&);
+
 private:
     QList<PreviewWidgetModel*> m_previewWidgetModels;
     bool m_loaded;
+    Scope* m_scope;
 };
 
 Q_DECLARE_METATYPE(PreviewModel*)

=== modified file 'tests/mocks/Unity/fake_previewstack.cpp'
--- tests/mocks/Unity/fake_previewstack.cpp	2014-05-19 11:10:58 +0000
+++ tests/mocks/Unity/fake_previewstack.cpp	2015-02-16 12:09:57 +0000
@@ -21,10 +21,10 @@
 #include "fake_previewmodel.h"
 #include "fake_scope.h"
 
-PreviewStack::PreviewStack(QObject* parent)
+PreviewStack::PreviewStack(QObject* parent, Scope *scope)
  : unity::shell::scopes::PreviewStackInterface(parent)
 {
-    m_previews << new PreviewModel(this);
+    m_previews << new PreviewModel(this, scope);
 }
 
 PreviewStack::~PreviewStack()

=== modified file 'tests/mocks/Unity/fake_previewstack.h'
--- tests/mocks/Unity/fake_previewstack.h	2014-05-19 11:10:58 +0000
+++ tests/mocks/Unity/fake_previewstack.h	2015-02-16 12:09:57 +0000
@@ -25,12 +25,14 @@
 
 class PreviewModel;
 
+class Scope;
+
 class PreviewStack : public unity::shell::scopes::PreviewStackInterface
 {
     Q_OBJECT
 
 public:
-    explicit PreviewStack(QObject* parent = 0);
+    explicit PreviewStack(QObject* parent = 0, Scope* scope = 0);
     virtual ~PreviewStack();
 
     QVariant data(const QModelIndex& index, int role = Qt::DisplayRole) const override;

=== modified file 'tests/mocks/Unity/fake_previewwidgetmodel.cpp'
--- tests/mocks/Unity/fake_previewwidgetmodel.cpp	2015-02-16 12:09:57 +0000
+++ tests/mocks/Unity/fake_previewwidgetmodel.cpp	2015-02-16 12:09:57 +0000
@@ -57,6 +57,7 @@
     QVariantMap attributes;
     QVariantMap buttonData;
     buttonData["label"] = "Button";
+    buttonData["id"] = "open_click";
     QVariantList buttons;
     buttons << buttonData << buttonData << buttonData;
     attributes["actions"] = QVariant::fromValue(buttons);

=== modified file 'tests/mocks/Unity/fake_scope.cpp'
--- tests/mocks/Unity/fake_scope.cpp	2014-12-15 12:39:00 +0000
+++ tests/mocks/Unity/fake_scope.cpp	2015-02-16 12:09:57 +0000
@@ -185,6 +185,7 @@
     if (result.toString() == "Result.2.2") {
         Scopes *scopes = dynamic_cast<Scopes*>(parent());
         m_openScope = scopes->getScopeFromAll("MockScope9");
+        scopes->addTempScope(m_openScope);
         Q_EMIT openScope(m_openScope);
     }
 }
@@ -198,7 +199,7 @@
     } else {
         // This probably leaks, do we don't care
         // it's a  test after all
-        return new PreviewStack;
+        return new PreviewStack(this, this);
     }
 }
 
@@ -208,11 +209,10 @@
 
 void Scope::closeScope(unity::shell::scopes::ScopeInterface* scope)
 {
-    if (scope != m_openScope) {
-        qDebug() << scope << m_openScope;
-        qFatal("Scope::closeScope got wrong scope in closeScope");
+    Scopes *scopes = dynamic_cast<Scopes*>(parent());
+    if (scopes) {
+        return scopes->closeScope(scope);
     }
-    m_openScope = nullptr;
 }
 
 QString Scope::currentNavigationId() const

=== modified file 'tests/mocks/Unity/fake_scopes.cpp'
--- tests/mocks/Unity/fake_scopes.cpp	2014-12-16 09:03:37 +0000
+++ tests/mocks/Unity/fake_scopes.cpp	2015-02-16 12:09:57 +0000
@@ -23,6 +23,7 @@
 // TODO: Implement remaining pieces, like Categories (i.e. LensView now gives warnings)
 
 // Qt
+#include <QDebug>
 #include <QTimer>
 
 Scopes::Scopes(QObject *parent)
@@ -177,6 +178,17 @@
     return m_scopesOverview;
 }
 
+void Scopes::addTempScope(unity::shell::scopes::ScopeInterface* scope)
+{
+    m_tempScopes.insert(scope);
+}
+
+void Scopes::closeScope(unity::shell::scopes::ScopeInterface* scope)
+{
+    Q_ASSERT(m_tempScopes.contains(scope));
+    m_tempScopes.remove(scope);
+}
+
 void Scopes::setFavorite(const QString& scopeId, bool favorite)
 {
     if (favorite) {

=== modified file 'tests/mocks/Unity/fake_scopes.h'
--- tests/mocks/Unity/fake_scopes.h	2014-12-10 09:07:14 +0000
+++ tests/mocks/Unity/fake_scopes.h	2015-02-16 12:09:57 +0000
@@ -26,6 +26,7 @@
 
 // Qt
 #include <QList>
+#include <QSet>
 #include <QTimer>
 
 class ScopesOverview;
@@ -59,6 +60,9 @@
     Q_INVOKABLE void setFavorite(const QString& scopeId, bool favorite) override;
     Q_INVOKABLE void moveFavoriteTo(const QString& scopeId, int index) override;
 
+    void addTempScope(unity::shell::scopes::ScopeInterface* scope);
+    Q_INVOKABLE void closeScope(unity::shell::scopes::ScopeInterface* scope) override;
+
     // This is used as part of implementation of the other C++ code, not API
     QList<Scope*> favScopes() const;
     QList<Scope*> nonFavScopes() const;
@@ -70,6 +74,7 @@
 private:
     QList<Scope*> m_scopes; // the favorite ones
     QList<Scope*> m_allScopes;
+    QSet<unity::shell::scopes::ScopeInterface*> m_tempScopes;
     ScopesOverview *m_scopesOverview;
     bool m_loaded;
     QTimer timer;

=== modified file 'tests/mocks/Unity/fake_scopesoverview.cpp'
--- tests/mocks/Unity/fake_scopesoverview.cpp	2014-12-16 09:03:37 +0000
+++ tests/mocks/Unity/fake_scopesoverview.cpp	2015-02-16 12:09:57 +0000
@@ -48,6 +48,7 @@
         Q_EMIT gotoScope(result.toString());
     } else {
         m_openScope = scopes->getScopeFromAll(result.toString());
+        scopes->addTempScope(m_openScope);
         Q_EMIT openScope(m_openScope);
     }
 }

=== modified file 'tests/qmltests/Dash/tst_Dash.qml'
--- tests/qmltests/Dash/tst_Dash.qml	2015-01-09 10:42:42 +0000
+++ tests/qmltests/Dash/tst_Dash.qml	2015-02-16 12:09:57 +0000
@@ -428,5 +428,74 @@
             tryCompare(bottomEdgeController, "progress", 0);
             tryCompare(dashContentList, "currentIndex", 1)
         }
+
+        function test_close_temp_scope_preview_opening_scope() {
+            // Show the manage dash
+            touchFlick(dash, dash.width / 2, dash.height - 1, dash.width / 2, units.gu(2));
+            var bottomEdgeController = findInvisibleChild(dash, "bottomEdgeController");
+            tryCompare(bottomEdgeController, "progress", 1);
+
+            // Make sure stuff is loaded
+            var nonfavScopesListCategory = findChild(dash, "scopesListCategoryother");
+            var nonfavScopesListCategoryList = findChild(nonfavScopesListCategory, "scopesListCategoryInnerList");
+            tryCompare(nonfavScopesListCategoryList, "currentIndex", 0);
+
+            // Click on a non favorite scope
+            mouseClick(nonfavScopesListCategoryList.currentItem);
+
+            // Check the bottom edge (manage dash) is disabled from temp scope
+            var overviewDragHandle = findChild(dash, "overviewDragHandle");
+            compare(overviewDragHandle.enabled, false);
+
+            // Check temp scope is there
+            var dashTempScopeItem = findChild(dash, "dashTempScopeItem");
+            tryCompare(dashTempScopeItem, "x", 0);
+            tryCompare(dashTempScopeItem, "visible", true);
+
+            // Check the manage dash is gone
+            tryCompare(bottomEdgeController, "progress", 0);
+
+            // Open preview
+            var categoryListView = findChild(dashTempScopeItem, "categoryListView");
+            categoryListView.positionAtBeginning();
+            tryCompareFunction(function() {
+                                    var cardGrid = findChild(dashTempScopeItem, "dashCategory0");
+                                    if (cardGrid != null) {
+                                        var tile = findChild(cardGrid, "delegate0");
+                                        return tile != null;
+                                    }
+                                    return false;
+                                },
+                                true);
+            var tile = findChild(findChild(dashTempScopeItem, "dashCategory0"), "delegate0");
+            waitForRendering(tile);
+            mouseClick(tile);
+            var subPageLoader = findChild(dashTempScopeItem, "subPageLoader");
+            tryCompare(subPageLoader, "open", true);
+            tryCompare(subPageLoader, "x", 0);
+            tryCompare(findChild(dashTempScopeItem, "categoryListView"), "visible", false);
+            var previewListRow0 = findChild(subPageLoader, "previewListRow0");
+            touchFlick(previewListRow0, previewListRow0.width / 2, units.gu(20), previewListRow0.width / 2, units.gu(1));
+            tryCompare(previewListRow0, "atYEnd", true);
+            tryCompare(previewListRow0, "moving", false);
+            var widget = findChild(subPageLoader, "widget-21");
+            var initialWidgetHeight = widget.height;
+            var openButton = findChild(widget, "buttonopen_click");
+            mouseClick(openButton);
+
+            tryCompare(subPageLoader, "open", false);
+            tryCompare(subPageLoader, "x", subPageLoader.width);
+
+            compare(dashTempScopeItem.scope.id, "MockScope9");
+
+            // Go back
+            var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader");
+            var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton");
+            mouseClick(backButton);
+
+            // Check temp scope is gone
+            tryCompare(dashTempScopeItem, "x", dash.width);
+            tryCompare(dashTempScopeItem, "visible", false);
+        }
     }
 }