Improved resource access caching

* Smarter caching. ResourceAccess instances close after a timeout, if not reused.
* Introduced a start command to avoid race condition when sending
commands to a resource that is currently shutting down.
* We resend pending commands after we lost access to the resource
* unexpectedly.

1 files changed, 29 insertions, 0 deletions

diff --git a/tests/resourcecommunicationtest.cpp b/tests/resourcecommunicationtest.cpp
index fa2b5a1..18e9223 100644
--- a/tests/resourcecommunicationtest.cpp
+++ b/tests/resourcecommunicationtest.cpp
@@ -63,6 +63,35 @@ private Q_SLOTS:
         QTRY_COMPARE(complete, count);
         QVERIFY(!errors);
     }
+
+    void testResourceAccessReuse()
+    {
+        qDebug();
+        const QByteArray resourceIdentifier("test");
+        Listener listener(resourceIdentifier);
+        Akonadi2::ResourceAccess resourceAccess(resourceIdentifier);
+        resourceAccess.open();
+
+        const int count = 10;
+        int complete = 0;
+        int errors = 0;
+        for (int i = 0; i < count; i++) {
+            resourceAccess.sendCommand(Akonadi2::Commands::PingCommand)
+                .then<void>([&complete]() {
+                    complete++;
+                },
+                [&errors, &complete](int error, const QString &msg) {
+                    qWarning() << msg;
+                    errors++;
+                    complete++;
+                }).then<void>([&resourceAccess]() {
+                    resourceAccess.close();
+                    resourceAccess.open();
+                }).exec().waitForFinished();
+        }
+        QTRY_COMPARE(complete, count);
+        QVERIFY(!errors);
+    }
 };
 
 QTEST_MAIN(ResourceCommunicationTest)