From 54fc031c2a1632a69f4d6effa56dc9ba75448937 Mon Sep 17 00:00:00 2001 From: Christian Mollekopf Date: Fri, 2 Mar 2018 11:56:10 +0100 Subject: A more stable flagChangeTest --- tests/mailsynctest.cpp | 30 ++++++++++-------------------- 1 file changed, 10 insertions(+), 20 deletions(-) (limited to 'tests') diff --git a/tests/mailsynctest.cpp b/tests/mailsynctest.cpp index 764b006..e9e5bb2 100644 --- a/tests/mailsynctest.cpp +++ b/tests/mailsynctest.cpp @@ -363,39 +363,29 @@ void MailSyncTest::testFetchNewRemovedMessages() void MailSyncTest::testFlagChange() { + Sink::Query syncScope; + syncScope.resourceFilter(mResourceInstanceIdentifier); + Sink::Query query; query.resourceFilter(mResourceInstanceIdentifier); query.filter(true); + query.filter(Sink::Query{}.filter("test")); query.request().request(); auto messageIdentifier = createMessage(QStringList() << "test", newMessage("Foobar")); - VERIFYEXEC(Store::synchronize(query)); - VERIFYEXEC(ResourceControl::flushMessageQueue(QByteArrayList() << mResourceInstanceIdentifier)); + VERIFYEXEC(Store::synchronize(syncScope)); + VERIFYEXEC(ResourceControl::flushMessageQueue(mResourceInstanceIdentifier)); - { - auto job = Store::fetchAll(query).then([](const QList &mails) { - ASYNCCOMPARE(mails.size(), 0); - return KAsync::null(); - }); - VERIFYEXEC(job); - } + QCOMPARE(Store::read(query).size(), 0); markAsImportant(QStringList() << "test", messageIdentifier); // Ensure all local data is processed - VERIFYEXEC(Store::synchronize(query)); - VERIFYEXEC(ResourceControl::flushMessageQueue(QByteArrayList() << mResourceInstanceIdentifier)); - - { - auto job = Store::fetchAll(query).then([](const QList &mails) { - ASYNCCOMPARE(mails.size(), 1); - ASYNCVERIFY(mails.first()->getImportant()); - return KAsync::null(); - }); - VERIFYEXEC(job); - } + VERIFYEXEC(Store::synchronize(syncScope)); + VERIFYEXEC(ResourceControl::flushMessageQueue(mResourceInstanceIdentifier)); + QCOMPARE(Store::read(query).size(), 1); } void MailSyncTest::testSyncSingleFolder() -- cgit v1.2.3