Add auxiliary API for Proxy

src/ipc/ipc-common/CMakeLists.txt

@@ -23,6 +23,7 @@ facelift_add_library(FaceliftIPCCommonLib
         IPCServiceAdapterBase.h
         NewIPCServiceAdapterBase.h
         IPCAttachedPropertyFactory.h
+        observer.h
     HEADERS_NO_MOC
         AppendDBUSSignatureFunction.h
         ipc-serialization.h

src/ipc/ipc-common/IPCProxyBase.h

@@ -35,12 +35,6 @@
 #include "IPCProxyBinderBase.h"
 
 
-#if defined(FaceliftIPCCommonLib_LIBRARY)
-#  define FaceliftIPCCommonLib_EXPORT Q_DECL_EXPORT
-#else
-#  define FaceliftIPCCommonLib_EXPORT Q_DECL_IMPORT
-#endif
-
 namespace facelift {
 
 template<typename AdapterType>

src/ipc/ipc-common/observer.h

@@ -0,0 +1,129 @@
+/**********************************************************************
+**
+** Copyright (C) 2020 Luxoft Sweden AB
+**
+** This file is part of the FaceLift project
+**
+** Permission is hereby granted, freIPCServiceAdapterBasee of charge, to any person
+** obtaining a copy of this software and associated documentation files
+** (the "Software"), to deal in the Software without restriction,
+** including without limitation the rights to use, copy, modify, merge,
+** publish, distribute, sublicense, and/or sell copies of the Software,
+** and to permit persons to whom the Software is furnished to do so,
+** subject to the following conditions:
+**
+** The above copyright notice and this permission notice shall be
+** included in all copies or substantial portions of the Software.
+**
+** THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+** EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+** MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+** NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+** BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+** ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+** CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+** SOFTWARE.
+**
+** SPDX-License-Identifier: MIT
+**
+**********************************************************************/
+
+#pragma once
+
+#include <memory>
+#include <QVector>
+#include <QObject>
+
+namespace facelift {
+
+class IObserver : public QObject
+{
+    Q_OBJECT
+public:
+    virtual void IsReadyObserver(const std::shared_ptr<QMetaObject::Connection> &connection) = 0;
+};
+
+class IsReadyObserver: public QObject
+{
+    Q_OBJECT
+    QVector< QPointer<IObserver> > m_observers{};
+    bool isReady{};
+
+public:
+    IsReadyObserver() {}
+
+    template<typename T, typename S, typename F>
+    void watch(T const *object, S signal, F function)
+    {
+        QObject::connect(object, std::move(signal), object, [this, function, object](){
+             std::function<bool()> functor { std::bind(function, object) };
+             if(functor != nullptr) {
+                 isReady = functor();
+             }
+         });
+         QObject::connect(object, std::move(signal), this, &IsReadyObserver::onReadyChanged );
+    }
+
+    // Set observers
+    void setObservers(const QVector< QPointer<IObserver> > &observers) {
+        m_observers.clear();
+        for (auto observer: observers) {
+            Q_ASSERT (observer != nullptr);
+            m_observers.push_back(observer);
+            auto connection = std::make_shared<QMetaObject::Connection>();
+            *connection = QObject::connect(this, &IsReadyObserver::readyChanged, observer, [observer, connection](){
+                observer->IsReadyObserver( connection );
+            });
+        }
+    }
+
+    // Get observers
+    const QVector< QPointer<IObserver> > &getObservers() const {
+        return m_observers;
+    }
+
+    Q_SIGNAL void readyChanged();
+
+    void onReadyChanged() {
+        if(isReady) {
+            emit readyChanged();
+        }
+    }
+};
+
+// Single-time observer which will unregister itself when done
+template<typename T>
+class SingleTimeObserver : public IObserver
+{
+    T m_function;
+
+public:
+    explicit SingleTimeObserver(T function) : m_function{function} {}
+    ~SingleTimeObserver() = default;
+
+    void IsReadyObserver(const std::shared_ptr<QMetaObject::Connection> &connection) override {
+        if(m_function != nullptr) {
+            m_function();
+        }
+        QObject::disconnect(*connection);
+    }
+};
+
+// Standard observer which will work for each signal
+template<typename T>
+class StandartObserver : public IObserver
+{
+    T m_function;
+
+public:
+    explicit StandartObserver(T function) : m_function{function} {}
+    ~StandartObserver() = default;
+
+    void IsReadyObserver(const std::shared_ptr<QMetaObject::Connection> &) override {
+        if(m_function != nullptr) {
+            m_function();
+        }
+    }
+};
+
+}

tests/unittest/CMakeLists.txt

@@ -6,18 +6,34 @@ if(${GTEST_FOUND})
         add_library(GTest::GMock UNKNOWN IMPORTED)
         set_target_properties(GTest::GMock PROPERTIES IMPORTED_LOCATION ${GMOCK_LIBRARY})
     else()
-        message(WARNING "Google test/mock not found.")
+        message(ERROR "Google test/mock not found.")
     endif()
 
+    find_package(Qt5 COMPONENTS Test REQUIRED)
+    if(${QT5TEST_NOTFOUND})
+        message(ERROR "Required package Qt5Test not found.")
+    endif()
     find_package(Threads REQUIRED)
-    set(FACELIFT_GTEST_LIBRARIES ${GTEST_BOTH_LIBRARIES} GTest::GMock Threads::Threads)
+    if(${THREADS_NOTFOUND})
+        message(ERROR "Required package Threads not found.")
+    endif()
+    set(FACELIFT_GTEST_LIBRARIES ${GTEST_BOTH_LIBRARIES} GTest::GMock Qt5::Test Threads::Threads)
     include_directories(${GTEST_INCLUDE_DIRS})
 
     facelift_add_test(UnitTests
         SOURCES FaceliftUtilsTest.cpp
-        LINK_LIBRARIES ${FACELIFT_GTEST_LIBRARIES} FaceliftCommonLib)
+        LINK_LIBRARIES
+            ${FACELIFT_GTEST_LIBRARIES}
+            FaceliftCommonLib
+    )
 
+    facelift_add_test(UnitTestsObserver
+        SOURCES FaceliftObserverTest.cpp
+        LINK_LIBRARIES
+            ${FACELIFT_GTEST_LIBRARIES}
+            FaceliftIPCCommonLib
+    )
 else()
-    message(WARNING "Required package google test not found!")
+    message(ERROR "Required package google test not found!")
 endif()