Merge lp:~saviq/unity/phablet.notification-interface-tests into lp:unity/phablet

[Michał Sawicz]

add Unity.Notifications interface tests

This is the first installment of our "let's drive the shell-facing interfaces with QML tests" approach.

This test is failing (hence skipped), but can be run by `make testNotificationInterface`, and is an acceptance factor for the Notifications backend.

Please look through to see if everything is clear and comment away.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:521
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/199/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/200
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/200/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/199
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/75
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/75/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/75
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/472

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/199/rebuild

review: Approve (continuous-integration)

[Mirco Müller (macslow)]

While the hint "synchronous" is implicitly know to be true, if a notification of Type.Confirmation is passed, we should still test for this particular hint.

[Michał Sawicz (saviq)]

I disagree, there's no value in passing that hint to the UI, it won't care about it, and if we move away from libnotify at some point, we'd have to artificially set that hint just for the sake of it being there.

[kevin gunn (kgunn72)]

maybe i'm early & this would be expected, but will there be additional failing tests added to determine what's valid? for instance, mutually exclusive typing enums, e.g. can a snap decision ever not have a button? or not contain text?...i would think not...e.g. its a requirement.

[Michał Sawicz (saviq)]

All the additional tests would happen on the C++ side. This set is only supposed to verify that the QML-exposed parts are working as expected.

[Mirco Müller (macslow)]

test_5_actions() should either check for Type.Interactive or have at least two actions defined. Going back to our discussion about the way how to define an interactive notification (and pass in the callback), we said we'd use actions too, but only a single one. Snap-decisions are meant to have at least two actions defined.

[Michał Sawicz (saviq)]

We have to balance out here, I'm not against adding such tests here, but then they have to be written as C++ tests anyway.

The idea is to have the smallest possible test suite in QML to ensure API is what we expect it to be. What, BTW, should happen when an app sends a SnapDecision with just one action? I know that's against the guidelines, but we might silently convert it to an Interactive notification.

BTW, what's the official naming? Ephemeral or Asynchronous?

[lp:~saviq/unity/phablet.notification-interface-tests]

522. By Michał Sawicz on 2013-03-28

rework most of the tests to fail, not only warn on mismatched interfaces

[Michał Sawicz (saviq)]

I reworked the tests to be a little more structured and a lot more robust.

[lp:~saviq/unity/phablet.notification-interface-tests]

523. By Michał Sawicz on 2013-03-28

fix comment

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:523
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/204/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/205
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/205/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/204
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/80
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/80/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/80
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/493

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/204/rebuild

review: Approve (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

524. By Michał Sawicz on 2013-03-28

reorder the calls to wait for the dataChanged signal first, then verify the number of notifications

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:524
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/205/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/206
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/206/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/205
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/81
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/81/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/81
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/496

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/205/rebuild

review: Approve (continuous-integration)

[Mirco Müller (macslow)]

> We have to balance out here, I'm not against adding such tests here, but then
> they have to be written as C++ tests anyway.

I agree.

> The idea is to have the smallest possible test suite in QML to ensure API is
> what we expect it to be. What, BTW, should happen when an app sends a
> SnapDecision with just one action? I know that's against the guidelines, but
> we might silently convert it to an Interactive notification.

No, this could lead to odd side-effects in terms of behaviour. The request of a notification of a specific type should always be explicit. An interactive notification should always use the dedicated hint "x-canonical-switch-to-application". I agree that we can then allow the triggering application to pass the single action via the actions parameter. If an application issues a wrongly formed notification, the backend should simply reject it. We have 3rd-party developer documentation to educate people about such requirements. Of course these need to be updated once we're done with the new implementation to reflect all the additions and changes we made.

> BTW, what's the official naming? Ephemeral or Asynchronous?

"Ephemeral" is official. "Asynchronous" is the internal/legacy term.

[lp:~saviq/unity/phablet.notification-interface-tests]

525. By Michał Sawicz on 2013-04-03

make Notifications.Model a registered singleton instead of a context property

526. By Michał Sawicz on 2013-04-03

remove numbers from tests

527. By Michał Sawicz on 2013-04-03

add documentation for the tests

528. By Michał Sawicz on 2013-04-03

fix references to enumerations

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:528
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/248/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/249
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/249/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/248
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/124
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/124/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/124
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/612

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/248/rebuild

review: Approve (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

529. By Michał Sawicz on 2013-04-04

fix two remaining test comments

530. By Michał Sawicz on 2013-04-04

two more comment fixes

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:530
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/253/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/254
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/254/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/253
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/129
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/129/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/129
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/625

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/253/rebuild

review: Approve (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

531. By Michał Sawicz on 2013-04-05

fix enum references, verify there's no notifications on init

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:531
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/270/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/271
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/271/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/270
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/146
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/146/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/146
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/641

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/270/rebuild

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

Ending test functions with _data() has a special meaning in QML. While this might still work, its very confusing to someone who knows the meaning of _data() functions for tests. I'd vote for either removing the _data() or change it in a way that it actually makes use of that mechanism.

[lp:~saviq/unity/phablet.notification-interface-tests]

532. By Michał Sawicz on 2013-04-11

rename _data tests to avoid confusion with data-driven tests

[Michał Sawicz (saviq)]

Indeed, fixed.

[lp:~saviq/unity/phablet.notification-interface-tests]

533. By Michał Sawicz on 2013-04-11

merge trunk

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:532
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/445/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/209/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/446/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/445/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/321/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/325/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/445/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:533
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/447/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/211/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/448/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/447/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/323/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/327/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/447/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:533
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/479/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/252/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/480/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/479/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/355/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/359/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/479/rebuild

review: Needs Fixing (continuous-integration)

[Daniel d'Andrada (dandrader)]

coding style: some lines are too long. I think we should strive for a maximum of around 90 chars so that having windows with code side-by-side is possible.

[Daniel d'Andrada (dandrader)]

147 + compare(repeater.itemAt(1).data.type, Notifications.Ephemeral, "the second notification should be of Ephemeral type");

Was that a typo?
s/Notifications.Ephemeral/Notifications.Type.Ephemeral

[lp:~saviq/unity/phablet.notification-interface-tests]

534. By Michał Sawicz on 2013-04-15

missed a .Type

535. By Michał Sawicz on 2013-04-15

wrap the code to 90 lines

[Michał Sawicz (saviq)]

Fixed on both accounts.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:535
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/532/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/326/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/533/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/532/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/408/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/412/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/532/rebuild

review: Needs Fixing (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

536. By Michał Sawicz on 2013-04-15

merge trunk

537. By Michał Sawicz on 2013-04-15

add copyright header to tst_NotificationInterface.qml

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:536
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/534/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/328/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/535/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/534/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/410/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/414/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/534/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:537
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/535/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/329/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/536/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/535/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/411/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/415/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/535/rebuild

review: Needs Fixing (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

538. By Michał Sawicz on 2013-04-15

add missing CMakeLists.txt

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:538
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/548/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/341
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/549
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/549/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/548
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/424
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/424/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/428
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1039

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/548/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:538
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/553/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/348
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/554
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/554/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/553
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/429
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/429/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/433
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1049

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/553/rebuild

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

coding style: Wrapping lines at 90 chars makes half of my screen empty while making it clumsy to read being squeezed on the left side.

Jokes aside... I think all editors nowadays support automatically wrapping lines. But there is no editor that supports automatic removal of newlines even though there is plenty of space on the screen. Also we have screens with 2880x1800 pixels and 30 inches of space - as a developer, usually multiple of those side by side.

IMO we should not follow 1980's style guides and manually wrap lines.

[Michi Henning (michihenning)]

I don't think this is an issue of technology. It's more about ergonomics. Long lines are harder to read than short ones. But I wouldn't get too obsessive about it. Somewhere around 120-130 columns is probably where things start to go out of hand, and two lines of 65 characters each would be easier to read.

Somewhere around 180 columns, we are definitely crossing over into insanity…

[lp:~saviq/unity/phablet.notification-interface-tests]

539. By Michał Sawicz on 2013-04-19

merge trunk

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:539
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/655/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/462
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/656
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/656/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/654
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/531
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/531/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/535
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1177

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/655/rebuild

review: Approve (continuous-integration)

[lp:~saviq/unity/phablet.notification-interface-tests]

540. By Michał Sawicz on 2013-04-22

merge trunk

541. By Michał Sawicz on 2013-04-22

relax column limit, don't expect Notification::completed to be available to QML, s/int/number/

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:541
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/702/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/519
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/704
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/704/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/702
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/578
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/578/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/582
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1233

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/702/rebuild

review: Approve (continuous-integration)

[Michał Sawicz (saviq)]

This has now been "converted" to lp:~saviq/unity-api/add-shell-notifications-api for merging into the separate lp:unity-api project, and mock plugins implemented to make sure that these pass.

review: Disapprove

Unmerged revisions

541. By Michał Sawicz on 2013-04-22

relax column limit, don't expect Notification::completed to be available to QML, s/int/number/

540. By Michał Sawicz on 2013-04-22

merge trunk

539. By Michał Sawicz on 2013-04-19

merge trunk

538. By Michał Sawicz on 2013-04-15

add missing CMakeLists.txt

537. By Michał Sawicz on 2013-04-15

add copyright header to tst_NotificationInterface.qml

536. By Michał Sawicz on 2013-04-15

merge trunk

535. By Michał Sawicz on 2013-04-15

wrap the code to 90 lines

534. By Michał Sawicz on 2013-04-15

missed a .Type

533. By Michał Sawicz on 2013-04-11

merge trunk

532. By Michał Sawicz on 2013-04-11

rename _data tests to avoid confusion with data-driven tests

=== modified file 'tests/qmltests/CMakeLists.txt'
--- tests/qmltests/CMakeLists.txt	2013-04-22 09:57:23 +0000
+++ tests/qmltests/CMakeLists.txt	2013-04-22 12:17:29 +0000
@@ -27,6 +27,7 @@
 add_qml_test(Greeter Clock)
 add_qml_test(Panel IndicatorItem)
 add_qml_test(utils/Unity/Test UnityTest)
+add_qml_test(interfaces NotificationInterface NO_ADD_TEST NO_TARGETS)
 
 set(qmltest_DEFAULT_TARGETS qmluitests qmltests)
 set(qmltest_DEFAULT_NO_ADD_TEST TRUE)

=== added directory 'tests/qmltests/interfaces'
=== added file 'tests/qmltests/interfaces/tst_NotificationInterface.qml'
--- tests/qmltests/interfaces/tst_NotificationInterface.qml	1970-01-01 00:00:00 +0000
+++ tests/qmltests/interfaces/tst_NotificationInterface.qml	2013-04-22 12:17:29 +0000
@@ -0,0 +1,198 @@
+/*
+ * Copyright 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/>.
+ */
+
+import QtQuick 2.0
+import QtTest 1.0
+import Unity.Notifications 0.1 as Notifications
+import Unity.Notifications.Mocks 0.1 as NotificationMocks
+
+TestCase {
+
+    NotificationMocks.Source {
+        id: source
+    }
+
+    Repeater {
+        id: repeater
+        model: Notifications.Model
+        delegate: Item {
+            property var data: model
+            property Repeater actionRepeater
+
+            actionRepeater: Repeater {
+                model: parent.data.actions
+                delegate: Item {
+                    property var data: model
+                }
+            }
+        }
+    }
+
+    SignalSpy {
+        id: dataSpy
+        target: Notifications.Model
+        signalName: "dataChanged"
+    }
+
+    function initTestCase() {
+        Notifications.Model.source = source;
+    }
+
+    function init() {
+        Notifications.Model.confirmationPlaceholder = false;
+        tryCompare(repeater, "count", "0", "There should be no notifications")
+    }
+
+    function cleanup() {
+        // dismiss all Notifications
+        for (var i = 0; i < repeater.count; i++) {
+            repeater.itemAt(i).data.notification.dismissed()
+        }
+        dataSpy.clear()
+    }
+
+    /* make sure all the required types are registered */
+    function test_types() {
+        compare(typeof Notifications.Urgency.Low, "number",
+                "there should be an Urgency::Low enum");
+        compare(typeof Notifications.Urgency.Normal, "number",
+                "there should be an Urgency::Normal enum");
+        compare(typeof Notifications.Urgency.Critical, "number",
+                "there should be an Urgency::Critical enum");
+        compare(typeof Notifications.Type.Confirmation, "number",
+                "there should be a Type::Confirmation enum");
+        compare(typeof Notifications.Type.Ephemeral, "number",
+                "there should be a Type::Ephemeral enum");
+        compare(typeof Notifications.Type.Interactive, "number",
+                "there should be a Type::Interactive enum");
+        compare(typeof Notifications.Type.SnapDecision, "number",
+                "there should be a Type::SnapDecision enum");
+        compare(typeof Notifications.Type.Placeholder, "number",
+                "there should be a Type::Placeholder enum");
+        compare(typeof Notifications.Hint.ButtonTint, "number",
+                "there should be a Hint::ButtonTint enum");
+        compare(typeof Notifications.Hint.IconOnly, "number",
+                "there should be a Hint::IconOnly enum");
+        compare(typeof Notifications.Notification, "object",
+                "Notification should be a registered type");
+        compare(typeof Notifications.Action, "object",
+                "Action should be a registered type");
+        compare(typeof Notifications.Model, "object",
+                "Notifications.Model should be a registered singleton");
+        compare(typeof Notifications.Model.source, "object",
+                "Notifications.Model should have a source property");
+        compare(typeof Notifications.Model.confirmationPlaceholder, "boolean",
+                "Notifications.Model should have a confirmationPlaceholder property");
+    }
+
+    /* make sure all the required roles and methods of Notification are exposed */
+    function test_notification_members() {
+        source.send({
+            "type": Notifications.Type.Interactive,
+        });
+        tryCompare(repeater, "count", 1, "there should be one notification");
+
+        var data = repeater.itemAt(0).data;
+        compare(typeof data.type, "number",
+                "NotificationModel should expose a \"type\" role of type Type");
+        compare(typeof data.urgency, "number",
+                "NotificationModel should expose a \"urgency\" role of type Urgency");
+        compare(typeof data.id, "number",
+                "NotificationModel should expose a \"id\" role of type int");
+        compare(typeof data.summary, "string",
+                "NotificationModel should expose a \"summary\" role of type QString");
+        compare(typeof data.body, "string",
+                "NotificationModel should expose a \"body\" role of type QString");
+        compare(typeof data.value, "number",
+                "NotificationModel should expose a \"value\" role of type int");
+        compare(typeof data.icon, "object",
+                "NotificationModel should expose an \"icon\" role of type QUrl");
+        compare(typeof data.secondaryIcon, "object",
+                "NotificationModel should expose a \"secondaryIcon\" role of type QUrl");
+        compare(typeof data.actions, "object",
+                "NotificationModel should expose an \"actions\" role of type QQmlListProperty");
+        compare(typeof data.hints, "number",
+                "NotificationModel should expose a \"hints\" role of type Hints");
+        compare(typeof data.notification, "object",
+                "NotificationModel should expose a \"notification\" role of type Notification");
+
+        var notification = data.notification;
+        compare(typeof notification.dismissed, "function",
+                "Notification::dismissed should be a signal");
+        compare(typeof notification.onHovered, "function",
+                "Notification::onHovered should be a slot");
+        compare(typeof notification.onDisplayed, "function",
+                "Notification::onDisplayed should be a slot");
+    }
+
+    /* make sure all the required roles and methods of Action are exposed */
+    function test_action_members() {
+        source.send({
+            "type": Notifications.Type.Interactive,
+            "actions": [
+                {"label": "test"}
+            ]
+        });
+        tryCompare(repeater, "count", 1, "there should be one notification");
+
+        var data = repeater.itemAt(0).data;
+        data = repeater.itemAt(0).actionRepeater.itemAt(0).data;
+        compare(typeof data.label, "string",
+                "Notification::actions should expose a \"label\" role of type QString");
+        compare(typeof data.action, "object",
+                "Notification::actions should expose an \"action\" role of type Action");
+
+        var action = data.action;
+        compare(typeof action.invoke, "function", "Action::invoke should be a function");
+    }
+
+    /* make sure the model is empty by default */
+    function test_empty() {
+        tryCompare(repeater, "count", 0, "there should be no notifications");
+    }
+
+    /* make sure there is a placeholder item used as the first item when
+       confirmationPlaceholder is true and that any additional notification is added
+       after it */
+    function test_placeholder() {
+        Notifications.Model.confirmationPlaceholder = true;
+        tryCompare(repeater, "count", 1, "there should be one notification");
+        compare(repeater.itemAt(0).data.type, Notifications.Type.Placeholder,
+                "the notification should be of Placeholder type");
+        source.send({
+            "type": Notifications.Type.Ephemeral
+        })
+        tryCompare(repeater, "count", 2, "second notification was added");
+        compare(repeater.itemAt(0).data.type, Notifications.Type.Placeholder,
+                "the first notification should be of Placeholder type");
+        compare(repeater.itemAt(1).data.type, Notifications.Type.Ephemeral,
+                "the second notification should be of Ephemeral type");
+    }
+
+    /* make sure the placeholder item is updated with data incoming in a Confirmation
+       notification */
+    function test_confirmation() {
+        Notifications.Model.confirmationPlaceholder = true;
+        tryCompare(repeater, "count", 1, "there should be only one notification");
+        source.send({
+            "type": Notifications.Type.Confirmation
+        });
+        tryCompare(dataSpy, "count", 1, "notification data should have been updated");
+        compare(repeater.count, 1, "there should be only one notification");
+        compare(repeater.itemAt(0).type, Notifications.Type.Confirmation,
+                "the first notification should be of Confirmation type");
+    }
+}