15319a9 · unity8 · git.iko.soy

git.iko.soy/unity8

on xenial help sign in

add a mock and test for the fullscreen notifications

15319a9Lukáš Tinkl10 years ago

qml/Notifications/Notification.qml+2−1

compare with xenial

unifiedsplit

@@ -37,7 +37,8 @@ StyledItem {

37 property var hints

38 property var notification

39 property color color: theme.palette.normal.background

40− property bool fullscreen: false

40+ property bool fullscreen: notification.notification && typeof notification.notification.fullscreen != "undefined" ?

41+ notification.notification.fullscreen : false // fullscreen prop only exists in the mock

41 property int maxHeight

42 property int maxHeight

42 property int margins: units.gu(1)

43 property int margins: units.gu(1)

tests/mocks/Unity/Notifications/MockNotification.cpp+15

@@ -32,6 +32,7 @@ struct MockNotificationPrivate {

32 QStringList actions;

33 ActionModel* actionsModel;

34 QVariantMap hints;

35+ bool fullscreen = false;

35};

36};

37MockNotification::MockNotification(QObject *parent) : QObject(parent), p(new MockNotificationPrivate()) {

38MockNotification::MockNotification(QObject *parent) : QObject(parent), p(new MockNotificationPrivate()) {

@@ -181,3 +182,17 @@ void MockNotification::invokeAction(const QString &action) {

181void MockNotification::close() {

182void MockNotification::close() {

182 Q_EMIT completed(p->id);

183 Q_EMIT completed(p->id);

183}

184}

185+

186+bool MockNotification::fullscreen() const

187+{

188+ return p->fullscreen;

189+}

190+

191+void MockNotification::setFullscreen(bool fullscreen)

192+{

193+ if (p->fullscreen == fullscreen)

194+ return;

195+

196+ p->fullscreen = fullscreen;

197+ Q_EMIT fullscreenChanged(fullscreen);

198+}

tests/mocks/Unity/Notifications/MockNotification.h+6

@@ -41,6 +41,7 @@ class MockNotification : public QObject {

41 Q_PROPERTY(QStringList rawActions READ rawActions WRITE setActions)

42 Q_PROPERTY(ActionModel* actions READ getActions NOTIFY actionsChanged)

43 Q_PROPERTY(QVariantMap hints READ getHints WRITE setHints NOTIFY hintsChanged)

44+ Q_PROPERTY(bool fullscreen READ fullscreen WRITE setFullscreen NOTIFY fullscreenChanged) // only in mock

45private:

46private:

46 QScopedPointer<MockNotificationPrivate> p;

47 QScopedPointer<MockNotificationPrivate> p;

@@ -64,6 +65,8 @@ Q_SIGNALS:

64 void completed(int nid);

65 void completed(int nid);

65 void actionInvoked(const QString &action);

66 void actionInvoked(const QString &action);

68+ void fullscreenChanged(bool fullscreen);

69+

67public:

70public:

68 MockNotification(QObject *parent=nullptr);

71 MockNotification(QObject *parent=nullptr);

69 virtual ~MockNotification();

72 virtual ~MockNotification();

@@ -87,9 +90,12 @@ public:

87 void setActions(const QStringList &actions);

90 void setActions(const QStringList &actions);

88 QVariantMap getHints() const;

91 QVariantMap getHints() const;

89 void setHints(const QVariantMap& hints);

92 void setHints(const QVariantMap& hints);

93+ bool fullscreen() const;

94+ void setFullscreen(bool fullscreen);

91 Q_INVOKABLE void invokeAction(const QString &action);

96 Q_INVOKABLE void invokeAction(const QString &action);

92 Q_INVOKABLE void close();

97 Q_INVOKABLE void close();

98+

93};

99};

100

95#endif // MOCK_NOTIFICATION_H

101#endif // MOCK_NOTIFICATION_H

tests/qmltests/Notifications/Notification.qml+1

@@ -28,4 +28,5 @@ Notification {

28 secondaryIcon: ""

29 value: 0

30 rawActions: []

31+ fullscreen: false

31}

32}

tests/qmltests/Notifications/tst_Notifications.qml+70

@@ -141,6 +141,18 @@ Item {

141 mockModel.append(n)

142 }

143

144+ function addFullscreenNotification() {

145+ var component = Qt.createComponent("Notification.qml")

146+ var n = component.createObject("notification", {"nid": index++,

147+ "type": Notification.SnapDecision,

148+ "hints": {"x-canonical-private-affirmative-tint": "true"},

149+ "summary": "SIM PIN screen",

150+ "body": "Enter your PIN to unlock the SIM",

151+ "fullscreen": true})

152+ n.completed.connect(mockModel.onCompleted)

153+ mockModel.append(n)

154+ }

155+

144 function clearNotifications() {

156 function clearNotifications() {

145 while(mockModel.count > 0) {

157 while(mockModel.count > 0) {

146 remove1stNotification();

158 remove1stNotification();

@@ -228,6 +240,12 @@ Item {

228 onClicked: rootRow.add2ndConfirmationNotification()

240 onClicked: rootRow.add2ndConfirmationNotification()

229 }

241 }

230

242

243+ Button {

244+ width: parent.width

245+ text: "add a fullscreen notification"

246+ onClicked: rootRow.addFullscreenNotification()

247+ }

248+

231 Button {

249 Button {

232 width: parent.width

250 width: parent.width

233 text: "remove 1st notification"

251 text: "remove 1st notification"

@@ -371,6 +389,14 @@ Item {

371 rawActions: ["ok_id", "Ok",

389 rawActions: ["ok_id", "Ok",

372 "snooze_id", "Snooze",

390 "snooze_id", "Snooze",

373 "view_id", "View"]

391 "view_id", "View"]

392+ },

393+ Notification {

394+ nid: 10

395+ type: Notification.SnapDecision

396+ hints: {"x-canonical-private-affirmative-tint": "true"}

397+ summary: "SIM PIN screen"

398+ body: "Enter your PIN to unlock the SIM"

399+ fullscreen: true

374 }

400 }

375 ]

401 ]

376

402

@@ -797,6 +823,50 @@ Item {

797 tryCompareFunction(function() { return notification2.expanded; }, undefined);

823 tryCompareFunction(function() { return notification2.expanded; }, undefined);

798 }

824 }

799

825

826+ function topmostIsFullscreen_data() {

827+ return [{

828+ tag: "Confirmation notification with value",

829+ n: nlist[6]

830+ },

831+ {

832+ tag: "SIM PIN fullscreen notification",

833+ n: nlist[9]

834+ },

835+ {

836+ tag: "Snap Decision without secondary icon and no button-tint",

837+ n: nlist[3]

838+ }]

839+ }

840+

841+ function test_topmostIsFullscreen() {

842+ var data = topmostIsFullscreen_data();

843+

844+ // fill the model

845+ data.forEach(function(notification) {

846+ mockModel.append(notification.n);

847+ notification.n.completed.connect(mockModel.onCompleted);

848+ })

849+

850+ // make sure the view is properly updated before going on

851+ notifications.forceLayout();

852+ waitForRendering(notifications);

853+

854+ // initially, topmost is not fullscreen

855+ verify(!notifications.topmostIsFullscreen)

856+

857+ // close the 1st one, 2nd one should be fullscreen

858+ var notification0 = findChild(notifications, "notification0")

859+ notification0.closeNotification();

860+ waitForRendering(notifications);

861+ verify(notifications.topmostIsFullscreen)

862+

863+ // close the 1st one (SIM PIN) again, topmost should no longer be fullscreen

864+ notification0 = findChild(notifications, "notification0")

865+ notification0.closeNotification();

866+ waitForRendering(notifications);

867+ verify(!notifications.topmostIsFullscreen)

868+ }

869+

800 function cleanupTestCase() {

870 function cleanupTestCase() {

801 notifications.hasMouse = false;

871 notifications.hasMouse = false;

802 }

872 }