summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian Mollekopf <chrigi_1@fastmail.fm>2017-04-27 15:52:56 +0200
committerChristian Mollekopf <chrigi_1@fastmail.fm>2017-04-27 15:52:56 +0200
commitb28d4cf578f4ceabb0d40f65e55ac533027e8733 (patch)
tree70d46ce0bce71d29b2e30e021cba18c5ffbb3118
parent3017e10e7f06f41f1d64996a8191407c7a0a8f69 (diff)
downloadsink-b28d4cf578f4ceabb0d40f65e55ac533027e8733.tar.gz
sink-b28d4cf578f4ceabb0d40f65e55ac533027e8733.zip
We get an offline status notification initially
-rw-r--r--common/notification.cpp4
-rw-r--r--tests/notificationtest.cpp22
2 files changed, 18 insertions, 8 deletions
diff --git a/common/notification.cpp b/common/notification.cpp
index e688b6d..da31e20 100644
--- a/common/notification.cpp
+++ b/common/notification.cpp
@@ -48,8 +48,8 @@ static QByteArray name(int type)
48 48
49QDebug operator<<(QDebug dbg, const Sink::Notification &n) 49QDebug operator<<(QDebug dbg, const Sink::Notification &n)
50{ 50{
51 dbg << "Notification(Type: " << name(n.type) << "Id, : " << n.id << ", Code: "; 51 dbg << "Notification(Type:" << name(n.type) << "Id, :" << n.id << ", Code:";
52 dbg << n.code; 52 dbg << n.code;
53 dbg << ", Message: " << n.message << ", Entities: " << n.entities << ")"; 53 dbg << ", Message:" << n.message << ", Entities:" << n.entities << ")";
54 return dbg.space(); 54 return dbg.space();
55} 55}
diff --git a/tests/notificationtest.cpp b/tests/notificationtest.cpp
index a34d325..e268a17 100644
--- a/tests/notificationtest.cpp
+++ b/tests/notificationtest.cpp
@@ -67,12 +67,22 @@ private slots:
67 VERIFYEXEC(Sink::Store::synchronize(query)); 67 VERIFYEXEC(Sink::Store::synchronize(query));
68 VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); 68 VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1"));
69 69
70 QVERIFY(statusNotifications.size() <= 3); 70 using namespace Sink::ApplicationDomain;
71 QTRY_COMPARE(statusNotifications.size(), 3); 71 {
72 //Sync 72 QList<Status> expected = {
73 QCOMPARE(statusNotifications.at(0).code, static_cast<int>(ApplicationDomain::Status::ConnectedStatus)); 73 Status::OfflineStatus,
74 QCOMPARE(statusNotifications.at(1).code, static_cast<int>(ApplicationDomain::Status::BusyStatus)); 74 Status::ConnectedStatus,
75 QCOMPARE(statusNotifications.at(2).code, static_cast<int>(ApplicationDomain::Status::ConnectedStatus)); 75 Status::BusyStatus,
76 Status::ConnectedStatus
77 };
78 qInfo() << "Received notifications " << statusNotifications;
79 QVERIFY2(statusNotifications.size() <= expected.size(), "More notifications than expected.");
80 QTRY_COMPARE(statusNotifications.size(), expected.size());
81 qInfo() << "All received notifications " << statusNotifications;
82 for (auto i = 0; i < statusNotifications.size(); i++) {
83 QCOMPARE(statusNotifications.at(i).code, static_cast<int>(expected.at(i)));
84 }
85 }
76 //Changereplay 86 //Changereplay
77 // It can happen that we get a changereplay notification pair first and then a second one at the end, 87 // It can happen that we get a changereplay notification pair first and then a second one at the end,
78 // we therefore currently filter all changereplay notifications (see above). 88 // we therefore currently filter all changereplay notifications (see above).