Merge lp:~dobey/unity8/no-more-click into lp:unity8

Proposed by dobey
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 2778
Merged at revision: 2801
Proposed branch: lp:~dobey/unity8/no-more-click
Merge into: lp:unity8
Diff against target: 393 lines (+16/-172)
11 files modified
debian/control (+0/-2)
qml/Dash/CardGrid.qml (+2/-3)
qml/Dash/CardTool.qml (+2/-26)
qml/Dash/GenericScopeView.qml (+1/-1)
qml/Dash/ScopesListCategory.qml (+1/-1)
tests/autopilot/unity8/dash.py (+1/-17)
tests/autopilot/unity8/shell/tests/test_helpers.py (+0/-49)
tests/mocks/Unity/fake_scopes.cpp (+0/-1)
tests/qmltests/Dash/tst_Dash.qml (+1/-48)
tests/qmltests/Dash/tst_DashContent.qml (+3/-3)
tests/qmltests/Dash/tst_GenericScopeView.qml (+5/-21)
To merge this branch: bzr merge lp:~dobey/unity8/no-more-click
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Approve
Albert Astals Cid (community) Approve
Ubuntu Unity PS integration team Pending
Review via email: mp+315412@code.launchpad.net

Commit message

Remove dependencies on clickscope.

Description of the change

 * Are there any related MPs required for this MP to build/function as expected?

No. But other changes in silo 2272 go along nicely with this change.

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

Yes. Uninstalling unity-scope-click in a VM with unity8 behaves fine.

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

Yes.

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

App drawer replacement for click scope is already in unity8, so N/A.

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2771
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3002/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/3907/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3935
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3780
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3780/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3780
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3780/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3780
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3780/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3780/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3780
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3780/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3780
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3780/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3002/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2771
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3006/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3911
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2291
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2291
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3939
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3783/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3783/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3783/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3783/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3783/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3783
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3783/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3006/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Please remove this mentions of the clickscope

./Dash/GenericScopeView.qml:585:
favoriteEnabled: scopeView.scope && scopeView.scope.id !== "clickscope"

./Dash/ScopesListCategory.qml:92:
showStar: model.scopeId !== "clickscope" && (root.isFavoritesFeed || root.isAlsoInstalled)

./Dash/CardTool.qml:80:
readonly property bool isAppLikeScope: scopeId === "clickscope" || scopeId === "libertine-scope.ubuntu_libertine-scope"

./Dash/CardTool.qml:81:
readonly property bool isAppLikeScopeAppCategory: ((scopeId === "clickscope" && (categoryId === "predefined" || categoryId === "local"))

for isAppLikeScopeAppCategory pleae remove it (since it'll always be false together with the ifs) that depend on it

review: Needs Fixing
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2771
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3023/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3933
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2305
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2305
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3961
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3806/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3806/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3806/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3806/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3806/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3806
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3806/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3023/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
dobey (dobey) wrote :

On Wed, 2017-01-25 at 16:51 +0000, Albert Astals Cid wrote:
> for isAppLikeScopeAppCategory pleae remove it (since it'll always be
> false together with the ifs) that depend on it

I left this value in, as it seems to have a check for libertine scope
as well, but removed the clickscope reference here, as well as the
others you mentioned.

lp:~dobey/unity8/no-more-click updated
2772. By dobey

Fixes per review.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Actually on a second thought I'm not sure that last change i suggested is a good idea.

Will the user still be able to install the click scope even if we don't ship it by default?

If so I guess we want the CardTool.qml changes back in

Revision history for this message
dobey (dobey) wrote :

Yes it will still be installable.

On Thu, 2017-01-26 at 08:27 +0000, Albert Astals Cid wrote:
> Actually on a second thought I'm not sure that last change i
> suggested is a good idea.
>
> Will the user still be able to install the click scope even if we
> don't ship it by default?
>
> If so I guess we want the CardTool.qml changes back in

lp:~dobey/unity8/no-more-click updated
2773. By dobey

Strip the remaining special casing of libertine scope as well.

2774. By dobey

Remove some tests that are no longer relevant.

2775. By dobey

Update scope load checks in init.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2773
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3069/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3992
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2337
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2337
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4020
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3864/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3864/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3864/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3864/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3864/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3864
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3864/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3069/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2775
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3070/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3993
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2338
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2338
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4021
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3865/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3865/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3865/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3865/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3865/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3865
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3865/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3070/rebuild

review: Needs Fixing (continuous-integration)
lp:~dobey/unity8/no-more-click updated
2776. By dobey

Fix the after reset test.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2776
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3071/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/3994/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4022
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3866
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3866/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3866
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3866/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3866
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3866/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3866/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3866
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3866/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3866
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3866/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3071/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2776
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3072/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3995
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2339
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2339
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4023
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3867/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3867/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3867/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3867/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3867/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3867
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3867/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3072/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Yeah i forgot about the genericscope tests :D

You'll need http://paste.ubuntu.com/23892663/ to make them pass.

lp:~dobey/unity8/no-more-click updated
2777. By Albert Astals Cid

More test fixes.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2777
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3080/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/4003/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4031
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3875
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3875/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3875
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3875/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3875/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3875
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3875/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3875
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3875/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3875
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3875/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3080/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2777
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3083/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4007
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2348
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2348
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4035
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3879/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3879/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3879/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3879/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3879/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3879
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3879/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3083/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2777
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3087/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4014
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2351
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2351
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4042
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3886/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3886/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3886/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3886/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3886/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3886
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3886/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3087/rebuild

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Wops, one more thing, can you remove the

// ↓ This is the ubuntu store icon

comment from CardGrid.qml, makes no sense anymore :D

review: Needs Fixing
lp:~dobey/unity8/no-more-click updated
2778. By dobey

Removed comment.

Revision history for this message
Albert Astals Cid (aacid) wrote :

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

 * Did CI run pass? If not, please explain why.
Yes

review: Approve
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2778
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3094/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/4022/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4050
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3894
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3894/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3894
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3894/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3894
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3894/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3894/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3894
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3894/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3894
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3894/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3094/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2778
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3098/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4026
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/2359
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=zesty,testname=qmluitests.sh/2359
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4054
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/3898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/3898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/3898/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/3098/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2017-01-24 07:38:04 +0000
+++ debian/control 2017-01-31 14:31:04 +0000
@@ -141,7 +141,6 @@
141Recommends: indicator-keyboard,141Recommends: indicator-keyboard,
142 indicator-session,142 indicator-session,
143 unity-greeter-session-broadcast,143 unity-greeter-session-broadcast,
144 unity-scope-click,
145Suggests: unity-scope-mediascanner2,144Suggests: unity-scope-mediascanner2,
146Breaks: indicator-network (<< 0.5.1+14.10.20141014),145Breaks: indicator-network (<< 0.5.1+14.10.20141014),
147 ubuntu-touch-session (<< 0.107),146 ubuntu-touch-session (<< 0.107),
@@ -191,7 +190,6 @@
191 python3-gi,190 python3-gi,
192 qttestability-autopilot (>= 1.4),191 qttestability-autopilot (>= 1.4),
193 ubuntu-ui-toolkit-autopilot (>= 1.3.2030),192 ubuntu-ui-toolkit-autopilot (>= 1.3.2030),
194 unity-scope-click,
195 unity8 (= ${source:Version}),193 unity8 (= ${source:Version}),
196 unity8-tests (= ${source:Version}),194 unity8-tests (= ${source:Version}),
197 url-dispatcher-tools,195 url-dispatcher-tools,
198196
=== modified file 'qml/Dash/CardGrid.qml'
--- qml/Dash/CardGrid.qml 2016-10-12 14:29:02 +0000
+++ qml/Dash/CardGrid.qml 2017-01-31 14:31:04 +0000
@@ -25,8 +25,7 @@
25 return cardTool.template["collapsed-rows"];25 return cardTool.template["collapsed-rows"];
26 }26 }
2727
28 // ↓ This is the ubuntu store icon28 readonly property string backgroundShapeStyle: "flat"
29 readonly property string backgroundShapeStyle: cardTool.isAppLikeScope && !cardTool.isAppLikeScopeAppCategory ? "shadow" : "flat"
3029
31 expandedHeight: grid.totalContentHeight30 expandedHeight: grid.totalContentHeight
32 collapsedHeight: Math.min(grid.contentHeightForRows(collapsedRows, grid.cellHeight), expandedHeight)31 collapsedHeight: Math.min(grid.contentHeightForRows(collapsedRows, grid.cellHeight), expandedHeight)
@@ -46,7 +45,7 @@
46 ResponsiveGridView {45 ResponsiveGridView {
47 id: grid46 id: grid
48 anchors.fill: parent47 anchors.fill: parent
49 minimumHorizontalSpacing: (cardTool.isAppLikeScopeAppCategory && cardTool.isWideView) ? units.gu(5) : units.gu(1)48 minimumHorizontalSpacing: cardTool.isWideView ? units.gu(5) : units.gu(1)
50 delegateWidth: cardTool.cardWidth49 delegateWidth: cardTool.cardWidth
51 delegateHeight: cardTool.cardHeight50 delegateHeight: cardTool.cardHeight
52 verticalSpacing: units.gu(1)51 verticalSpacing: units.gu(1)
5352
=== modified file 'qml/Dash/CardTool.qml'
--- qml/Dash/CardTool.qml 2016-10-12 14:29:02 +0000
+++ qml/Dash/CardTool.qml 2017-01-31 14:31:04 +0000
@@ -77,16 +77,8 @@
77 return layout;77 return layout;
78 }78 }
7979
80 readonly property bool isAppLikeScope: scopeId === "clickscope" || scopeId === "libertine-scope.ubuntu_libertine-scope"
81 readonly property bool isAppLikeScopeAppCategory: ((scopeId === "clickscope" && (categoryId === "predefined" || categoryId === "local"))
82 || (scopeId === "libertine-scope.ubuntu_libertine-scope" && categoryId !== "hint"))
83
84 readonly property string artShapeStyle: {80 readonly property string artShapeStyle: {
85 if (isAppLikeScope) {81 return categoryLayout === "carousel" ? "shadow" : "flat"
86 return isAppLikeScopeAppCategory ? "icon" : "flat";
87 } else {
88 return categoryLayout === "carousel" ? "shadow" : "flat"
89 }
90 }82 }
9183
92 // FIXME ? This seems like it should not be needed, but on Qt 5.4 + phone84 // FIXME ? This seems like it should not be needed, but on Qt 5.4 + phone
@@ -121,18 +113,6 @@
121 If -1, should use implicit width of the actual card.113 If -1, should use implicit width of the actual card.
122 */114 */
123 readonly property real cardWidth: {115 readonly property real cardWidth: {
124 if (isAppLikeScopeAppCategory) {
125 if (!isNarrowView) {
126 if (isWideView) {
127 return units.gu(11);
128 } else {
129 return units.gu(10);
130 }
131 } else {
132 return units.gu(12);
133 }
134 }
135
136 switch (categoryLayout) {116 switch (categoryLayout) {
137 case "grid":117 case "grid":
138 case "vertical-journal":118 case "vertical-journal":
@@ -191,11 +171,7 @@
191 readonly property int headerHeight: cardLoader.item ? cardLoader.item.headerHeight : 0171 readonly property int headerHeight: cardLoader.item ? cardLoader.item.headerHeight : 0
192172
193 readonly property size artShapeSize: {173 readonly property size artShapeSize: {
194 if (isAppLikeScopeAppCategory) {174 return cardLoader.item ? cardLoader.item.artShapeSize : Qt.size(0, 0)
195 return Qt.size(units.gu(8), units.gu(7.5));
196 } else {
197 return cardLoader.item ? cardLoader.item.artShapeSize : Qt.size(0, 0)
198 }
199 }175 }
200176
201 QtObject {177 QtObject {
202178
=== modified file 'qml/Dash/GenericScopeView.qml'
--- qml/Dash/GenericScopeView.qml 2016-11-29 09:37:25 +0000
+++ qml/Dash/GenericScopeView.qml 2017-01-31 14:31:04 +0000
@@ -582,7 +582,7 @@
582 showSignatureLine: !showBackButton582 showSignatureLine: !showBackButton
583 searchEntryEnabled: true583 searchEntryEnabled: true
584 settingsEnabled: scopeView.scope && scopeView.scope.settings && scopeView.scope.settings.count > 0 || false584 settingsEnabled: scopeView.scope && scopeView.scope.settings && scopeView.scope.settings.count > 0 || false
585 favoriteEnabled: scopeView.scope && scopeView.scope.id !== "clickscope"585 favoriteEnabled: scopeView.scope
586 favorite: scopeView.scope && scopeView.scope.favorite586 favorite: scopeView.scope && scopeView.scope.favorite
587 navigationTag: scopeView.scope ? scopeView.scope.primaryNavigationTag : ""587 navigationTag: scopeView.scope ? scopeView.scope.primaryNavigationTag : ""
588 scopeStyle: scopeView.scopeStyle588 scopeStyle: scopeView.scopeStyle
589589
=== modified file 'qml/Dash/ScopesListCategory.qml'
--- qml/Dash/ScopesListCategory.qml 2016-11-17 20:58:14 +0000
+++ qml/Dash/ScopesListCategory.qml 2017-01-31 14:31:04 +0000
@@ -89,7 +89,7 @@
89 icon: model.art || model.mascot || ""89 icon: model.art || model.mascot || ""
90 text: model.title || ""90 text: model.title || ""
91 subtext: model.subtitle || ""91 subtext: model.subtitle || ""
92 showStar: model.scopeId !== "clickscope" && (root.isFavoritesFeed || root.isAlsoInstalled)92 showStar: root.isFavoritesFeed || root.isAlsoInstalled
93 isFavorite: root.isFavoritesFeed93 isFavorite: root.isFavoritesFeed
9494
95 hideChildren: dragItem.loaderToShrink == loader95 hideChildren: dragItem.loaderToShrink == loader
9696
=== modified file 'tests/autopilot/unity8/dash.py'
--- tests/autopilot/unity8/dash.py 2016-10-25 09:28:16 +0000
+++ tests/autopilot/unity8/dash.py 2017-01-31 14:31:04 +0000
@@ -48,23 +48,7 @@
48 self.dash_content_list = self.wait_select_single(48 self.dash_content_list = self.wait_select_single(
49 'QQuickListView', objectName='dashContentList')49 'QQuickListView', objectName='dashContentList')
5050
51 def get_applications_grid(self):51 def get_scope(self, scope_name):
52 get_grid = self.get_scope('clickscope').wait_select_single(
53 'CardGrid', objectName='local')
54 return get_grid
55
56 def get_application_icon(self, text):
57 """Returns a 'Tile' icon that has the text 'text' from the application
58 grid.
59
60 :param text: String containing the text of the icon to search for.
61
62 """
63 app_grid = self.get_applications_grid()
64 resp_grid = app_grid.wait_select_single('ResponsiveGridView')
65 return resp_grid.select_single('Tile', text=text)
66
67 def get_scope(self, scope_name='clickscope'):
68 return self.dash_content_list.wait_select_single(52 return self.dash_content_list.wait_select_single(
69 'QQuickLoader', scopeId=scope_name)53 'QQuickLoader', scopeId=scope_name)
7054
7155
=== modified file 'tests/autopilot/unity8/shell/tests/test_helpers.py'
--- tests/autopilot/unity8/shell/tests/test_helpers.py 2016-10-25 09:11:13 +0000
+++ tests/autopilot/unity8/shell/tests/test_helpers.py 2017-01-31 14:31:04 +0000
@@ -99,12 +99,6 @@
99 self._assert_scope_is_opened(scope, scope_id)99 self._assert_scope_is_opened(scope, scope_id)
100 self.assertIsInstance(scope, dash_helpers.GenericScopeView)100 self.assertIsInstance(scope, dash_helpers.GenericScopeView)
101101
102 def test_open_applications_scope(self):
103 scope_id = 'clickscope'
104 scope = self.dash.open_scope(scope_id)
105 self._assert_scope_is_opened(scope, scope_id)
106 self.assertIsInstance(scope, dash_helpers.GenericScopeView)
107
108102
109class GenericScopeViewHelperTestCase(tests.DashBaseTestCase):103class GenericScopeViewHelperTestCase(tests.DashBaseTestCase):
110104
@@ -122,46 +116,3 @@
122 """Open an item that requires swiping to make it visible."""116 """Open an item that requires swiping to make it visible."""
123 preview = self.generic_scope.open_preview('2', 'Title.2.0')117 preview = self.generic_scope.open_preview('2', 'Title.2.0')
124 self.assertIsInstance(preview, dash_helpers.Preview)118 self.assertIsInstance(preview, dash_helpers.Preview)
125
126
127class DashAppsHelperTestCase(tests.DashBaseTestCase):
128
129 available_applications = [
130 'Title.2.0', 'Title.2.1', 'Title.2.2', 'Title.2.3', 'Title.2.4',
131 'Title.2.5', 'Title.2.6', 'Title.2.7', 'Title.2.8', 'Title.2.9',
132 'Title.2.10', 'Title.2.11', 'Title.2.12', 'Title.2.13', 'Title.2.14',
133 'Title.2.15', 'Title.2.16', 'Title.2.17', 'Title.2.18']
134
135 def setUp(self):
136 # Set up the fake scopes before launching unity.
137 self.useFixture(fixture_setup.FakeScopes())
138 super().setUp()
139 self.applications_scope = self.dash.open_scope('clickscope')
140
141 def test_get_applications_should_return_correct_applications(self):
142 category = '2'
143 category_element = self.applications_scope._get_category_element(
144 category)
145 list_view = self.dash.get_scope('clickscope')\
146 .select_single(dash_helpers.ListViewWithPageHeader)
147 expected_apps_count = self._get_number_of_application_slots(category)
148 expected_applications = self.available_applications[
149 :expected_apps_count]
150 x_center = list_view.globalRect.x + list_view.width / 2
151 y_center = list_view.globalRect.y + list_view.height / 2
152 y_diff = (
153 category_element.y - list_view.height + category_element.height
154 )
155 list_view._slow_drag(x_center, x_center, y_center, y_center - y_diff)
156 applications = self.applications_scope.get_applications(category)
157 self.assertEqual(expected_applications, applications)
158
159 def _get_number_of_application_slots(self, category):
160 category_element = self.applications_scope._get_category_element(
161 category)
162 cardgrid = category_element.select_single('CardGrid')
163 if (category_element.expanded):
164 return cardgrid.select_single('QQuickGridView').count
165 else:
166 return cardgrid.collapsedRows \
167 * cardgrid.select_single('ResponsiveGridView').columns
168119
=== modified file 'tests/mocks/Unity/fake_scopes.cpp'
--- tests/mocks/Unity/fake_scopes.cpp 2017-01-10 14:46:38 +0000
+++ tests/mocks/Unity/fake_scopes.cpp 2017-01-31 14:31:04 +0000
@@ -60,7 +60,6 @@
60 addScope(new Scope("MockScope7", "MS7", false, this));60 addScope(new Scope("MockScope7", "MS7", false, this));
61 addScope(new Scope("MockScope8", "MS8", false, this));61 addScope(new Scope("MockScope8", "MS8", false, this));
62 addScope(new Scope("MockScope9", "MS9", false, this));62 addScope(new Scope("MockScope9", "MS9", false, this));
63 addScope(new Scope("libertine-scope.ubuntu_libertine-scope", "Libertine", true, this));
6463
65 Scope *longNavigationScope = new Scope(64 Scope *longNavigationScope = new Scope(
66 "LongPrimaryNavigation", "LongPrimaryNavigation", true, this, 1, false,65 "LongPrimaryNavigation", "LongPrimaryNavigation", true, this, 1, false,
6766
=== modified file 'tests/qmltests/Dash/tst_Dash.qml'
--- tests/qmltests/Dash/tst_Dash.qml 2016-12-14 17:43:23 +0000
+++ tests/qmltests/Dash/tst_Dash.qml 2017-01-31 14:31:04 +0000
@@ -58,7 +58,7 @@
58 tryCompare(dashContentList, "count", 0);58 tryCompare(dashContentList, "count", 0);
59 scopes.load();59 scopes.load();
60 tryCompare(dashContentList, "currentIndex", 0);60 tryCompare(dashContentList, "currentIndex", 0);
61 tryCompare(dashContentList, "count", 28);61 tryCompare(dashContentList, "count", 27);
62 tryCompare(scopes, "loaded", true);62 tryCompare(scopes, "loaded", true);
63 tryCompareFunction(function() {63 tryCompareFunction(function() {
64 var mockScope1Loader = findChild(dash, "scopeLoader0");64 var mockScope1Loader = findChild(dash, "scopeLoader0");
@@ -192,24 +192,6 @@
192 tryCompare(expandingItem, "visible", false)192 tryCompare(expandingItem, "visible", false)
193 }193 }
194194
195 function test_manage_dash_clickscope_unfavoritable() {
196 open_manage_dash();
197
198 // Click scope star area is not visible (i.e. can't be unfavorited)
199 var favScopesListCategoryList = findChild(findChild(dash, "scopesListCategoryfavorites"), "scopesListCategoryInnerList");
200 var clickScope = findChild(favScopesListCategoryList, "delegateclickscope");
201 var starArea = findChild(clickScope, "starArea");
202 compare(starArea.visible, false);
203
204 // Go back
205 var bottomEdgeController = findInvisibleChild(dash, "bottomEdgeController");
206 var scopesList = findChild(dash, "scopesList");
207 var scopesListPageHeader = findChild(scopesList, "pageHeader");
208 var backButton = findChild(scopesListPageHeader, "innerPageHeader").leadingActionBar;
209 mouseClick(backButton);
210 tryCompare(bottomEdgeController, "progress", 0);
211 }
212
213 function test_manage_dash_select_same_favorite() {195 function test_manage_dash_select_same_favorite() {
214 open_manage_dash();196 open_manage_dash();
215197
@@ -627,35 +609,6 @@
627 tryCompare(dashContent, "x", 0);609 tryCompare(dashContent, "x", 0);
628 }610 }
629611
630 function test_cardIconStyle()
631 {
632 dash.setCurrentScope("clickscope");
633 var dashContent = findChild(dash, "dashContent");
634 tryCompare(dashContent.currentScope, "id", "clickscope");
635
636 scrollToCategory("dashCategorypredefined");
637 tryCompareFunction(function() {
638 var tile = getCategoryDelegate("predefined", 2);
639 var proportionalShape = findChildsByType(tile, "UCProportionalShape");
640 return proportionalShape.length === 1;
641 },
642 true
643 );
644
645 dash.setCurrentScope("libertine-scope.ubuntu_libertine-scope");
646 var dashContent = findChild(dash, "dashContent");
647 tryCompare(dashContent.currentScope, "id", "libertine-scope.ubuntu_libertine-scope");
648
649 scrollToCategory("dashCategory2");
650 tryCompareFunction(function() {
651 var tile = getCategoryDelegate("2", 2);
652 var proportionalShape = findChildsByType(tile, "UCProportionalShape");
653 return proportionalShape.length === 1;
654 },
655 true
656 );
657 }
658
659 function test_tempScopeItemXOnResize()612 function test_tempScopeItemXOnResize()
660 {613 {
661 // Go to a temp scope614 // Go to a temp scope
662615
=== modified file 'tests/qmltests/Dash/tst_DashContent.qml'
--- tests/qmltests/Dash/tst_DashContent.qml 2016-12-02 00:56:24 +0000
+++ tests/qmltests/Dash/tst_DashContent.qml 2017-01-31 14:31:04 +0000
@@ -70,7 +70,7 @@
70 function loadScopes() {70 function loadScopes() {
71 scopeLoadedSpy.clear();71 scopeLoadedSpy.clear();
72 scopesModel.load();72 scopesModel.load();
73 tryCompare(scopeLoadedSpy, "count", 28, 15000);73 tryCompare(scopeLoadedSpy, "count", 27, 15000);
74 tryCompare(scopesModel, "loaded", true);74 tryCompare(scopesModel, "loaded", true);
75 tryCompareFunction(function() {75 tryCompareFunction(function() {
76 var mockScope1Loader = findChild(shell, "scopeLoader0");76 var mockScope1Loader = findChild(shell, "scopeLoader0");
@@ -120,11 +120,11 @@
120 scopesModel.clear();120 scopesModel.clear();
121 compare(dashContentList.count, 0, "DashContent should have 0 items after clearing");121 compare(dashContentList.count, 0, "DashContent should have 0 items after clearing");
122 // pretend we're running after a model reset122 // pretend we're running after a model reset
123 dashContentList.currentIndex = 27;123 dashContentList.currentIndex = 26;
124124
125 loadScopes();125 loadScopes();
126126
127 compare(dashContentList.count, 28);127 compare(dashContentList.count, 27);
128 verify(dashContentList.currentIndex >= 0 && dashContentList.currentIndex < dashContentList.count);128 verify(dashContentList.currentIndex >= 0 && dashContentList.currentIndex < dashContentList.count);
129 }129 }
130130
131131
=== modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml'
--- tests/qmltests/Dash/tst_GenericScopeView.qml 2016-12-06 20:16:56 +0000
+++ tests/qmltests/Dash/tst_GenericScopeView.qml 2017-01-31 14:31:04 +0000
@@ -84,6 +84,7 @@
84 genericScopeView.scope = scopes.getScope(2);84 genericScopeView.scope = scopes.getScope(2);
85 genericScopeView.isCurrent = true;85 genericScopeView.isCurrent = true;
86 shell.width = units.gu(120);86 shell.width = units.gu(120);
87 shell.height = units.gu(100);
87 genericScopeView.categoryView.positionAtBeginning();88 genericScopeView.categoryView.positionAtBeginning();
88 waitForRendering(genericScopeView.categoryView);89 waitForRendering(genericScopeView.categoryView);
89 }90 }
@@ -195,6 +196,8 @@
195 }196 }
196197
197 function test_expand_collapse() {198 function test_expand_collapse() {
199 shell.height = units.gu(120);
200
198 tryCompareFunction(function() { return findChild(genericScopeView, "dashSectionHeader0", 0 /* timeout */) != null; }, true);201 tryCompareFunction(function() { return findChild(genericScopeView, "dashSectionHeader0", 0 /* timeout */) != null; }, true);
199202
200 var category = findChild(genericScopeView, "dashCategory0")203 var category = findChild(genericScopeView, "dashCategory0")
@@ -531,6 +534,8 @@
531 }534 }
532535
533 function test_seeAllTwoCategoriesScenario2() {536 function test_seeAllTwoCategoriesScenario2() {
537 shell.height = units.gu(120);
538
534 mockScope.setId("mockScope");539 mockScope.setId("mockScope");
535 mockScope.setName("Mock Scope");540 mockScope.setName("Mock Scope");
536 mockScope.isActive = true;541 mockScope.isActive = true;
@@ -587,7 +592,6 @@
587 var innerHeader = findChild(header, "innerPageHeader");592 var innerHeader = findChild(header, "innerPageHeader");
588 verify(innerHeader, "Could not find the inner header");593 verify(innerHeader, "Could not find the inner header");
589594
590 expectFail("Apps", "Click scope should not have a favorite button");
591 var favoriteAction = findChild(innerHeader, "favorite_button");595 var favoriteAction = findChild(innerHeader, "favorite_button");
592 verify(favoriteAction, "Could not find the favorite action.");596 verify(favoriteAction, "Could not find the favorite action.");
593 mouseClick(favoriteAction);597 mouseClick(favoriteAction);
@@ -697,26 +701,6 @@
697 var shape = findChildsByType(artShapeLoader, "UCUbuntuShape");701 var shape = findChildsByType(artShapeLoader, "UCUbuntuShape");
698 compare(shape.borderSource, undefined);702 compare(shape.borderSource, undefined);
699 }703 }
700
701 function test_clickScopeSizing() {
702 genericScopeView.scope = scopes.getScopeFromAll("clickscope");
703 waitForRendering(genericScopeView);
704
705 var categoryListView = findChild(genericScopeView, "categoryListView");
706 waitForRendering(categoryListView);
707
708 var categorypredefined = findChild(categoryListView, "dashCategorypredefined");
709 waitForRendering(categorypredefined);
710
711 var cardTool = findChild(categorypredefined, "cardTool");
712
713 compare(cardTool.cardWidth, units.gu(11));
714 shell.width = units.gu(46);
715 waitForRendering(genericScopeView);
716 compare(cardTool.cardWidth, units.gu(10));
717
718 shell.width = units.gu(120)
719 }
720 }704 }
721 }705 }
722}706}

Subscribers

People subscribed via source and target branches