Add unittests for PresenceSubscriber

Change-Id: I7102ad07b25f91600f6b4f397482575622d77ca7
Signed-off-by: coderhyme <jhyo.kim@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/1320
Reviewed-by: Uze Choi <uzchoi@samsung.com>
Tested-by: Uze Choi <uzchoi@samsung.com>

diff --git a/service/basis/common/primitiveResource/SConscript b/service/basis/common/primitiveResource/SConscript
index 5dc7497..c371ce4 100644 (file)
--- a/service/basis/common/primitiveResource/SConscript
+++ b/service/basis/common/primitiveResource/SConscript
@@ -72,7 +72,10 @@ service_common_env.InstallTarget([service_common_static,service_common_shared],
 service_common_test_env = service_common_env.Clone();
 
 service_common_test_env.AppendUnique(LIBPATH = [env.get('BUILD_DIR')])
-service_common_test_env.PrependUnique(CPPPATH = [gtest_dir + '/include'])
+service_common_test_env.PrependUnique(CPPPATH = [
+    env.get('SRC_DIR')+'/extlibs/hippomocks-master',
+    gtest_dir + '/include'
+    ])
 
 gtest = File(gtest_dir + '/lib/.libs/libgtest.a')
 gtest_main = File(gtest_dir + '/lib/.libs/libgtest_main.a')

diff --git a/service/basis/common/primitiveResource/unittests/PresenceSubscriberTest.cpp b/service/basis/common/primitiveResource/unittests/PresenceSubscriberTest.cpp
index 137ec12..2e81c7f 100644 (file)
--- a/service/basis/common/primitiveResource/unittests/PresenceSubscriberTest.cpp
+++ b/service/basis/common/primitiveResource/unittests/PresenceSubscriberTest.cpp
@@ -22,12 +22,151 @@
 #include <HippoMocks/hippomocks.h>
 
 #include <PresenceSubscriber.h>
+#include <PrimitiveException.h>
+
+#include <OCPlatform.h>
 
 using namespace testing;
 using namespace OIC::Service;
 
+typedef OCStackResult (*subscribePresenceSig1)(OC::OCPlatform::OCPresenceHandle&,
+        const std::string&, OCConnectivityType, SubscribeCallback);
+typedef OCStackResult (*subscribePresenceSig2)(OC::OCPlatform::OCPresenceHandle&,
+        const std::string&, const std::string&, OCConnectivityType, SubscribeCallback);
+
+const std::string HOST{ "host" };
+const OCConnectivityType CONTYPE{ OCConnectivityType::OC_ALL };
+
 class PresenceSubscriberNonMemberTest: public Test
 {
 public:
+    OCDoHandle handle;
+    MockRepository mocks;
+
+};
+
+TEST_F(PresenceSubscriberNonMemberTest, OCPlatformSubscribePresenceWillBeCalled)
+{
+    mocks.ExpectCallFuncOverload(
+            static_cast< subscribePresenceSig1 >(OC::OCPlatform::subscribePresence))
+                        .With(_, HOST,CONTYPE, _).Return(OC_STACK_OK);
+
+    subscribePresence(handle, HOST, CONTYPE, SubscribeCallback());
+}
+
+TEST_F(PresenceSubscriberNonMemberTest, SubscribePresenceThrowsIfResultIsNotOK)
+{
+    mocks.ExpectCallFuncOverload(
+            static_cast< subscribePresenceSig1>(OC::OCPlatform::subscribePresence))
+                    .Return(OC_STACK_ERROR);
+
+    ASSERT_THROW(subscribePresence(handle, "", CONTYPE, SubscribeCallback()), PlatformException);
+}
+
+TEST_F(PresenceSubscriberNonMemberTest, OCPlatformUnsubscribePresenceWillBeCalled)
+{
+    mocks.ExpectCallFuncOverload(OC::OCPlatform::unsubscribePresence).Return(OC_STACK_OK);
+
+    unsubscribePresence(handle);
+}
+
+TEST_F(PresenceSubscriberNonMemberTest, UnsubscribePresenceThrowIfResultIsNotOK)
+{
+    mocks.ExpectCallFuncOverload(OC::OCPlatform::unsubscribePresence).Return(OC_STACK_ERROR);
+
+    ASSERT_THROW(unsubscribePresence(handle), PlatformException);
+}
+
+
+
+class PresenceSubscriberTest: public Test
+{
+public:
     MockRepository mocks;
+
+protected:
+    void SetUp() override {
+        mocks.OnCallFuncOverload(
+                static_cast< subscribePresenceSig1 >(OC::OCPlatform::subscribePresence)).Do(
+
+            [](OC::OCPlatform::OCPresenceHandle& handle, const std::string&,
+                    OCConnectivityType, SubscribeCallback) -> OCStackResult
+            {
+                handle = reinterpret_cast<OC::OCPlatform::OCPresenceHandle>(1);
+                return OC_STACK_OK;
+            }
+        );
+
+        mocks.OnCallFunc(OC::OCPlatform::unsubscribePresence).Return(OC_STACK_OK);
+    }
 };
+
+TEST_F(PresenceSubscriberTest, IsNotSubscribingWhenCreatedWithDefaultConstructor)
+{
+    PresenceSubscriber subscriber;
+    ASSERT_FALSE(subscriber.isSubscribing());
+}
+
+TEST_F(PresenceSubscriberTest, ConstructorCallOCPlatformSubscribe)
+{
+    mocks.ExpectCallFuncOverload(
+            static_cast< subscribePresenceSig1 >(OC::OCPlatform::subscribePresence))
+                     .With(_, HOST, CONTYPE, _).Return(OC_STACK_OK);
+
+    PresenceSubscriber subscriber{ HOST, CONTYPE, SubscribeCallback() };
+}
+
+TEST_F(PresenceSubscriberTest, ConstructorWithResourceTypeCallOCPlatformSubscribe)
+{
+    const std::string resType { "resType" };
+
+    mocks.ExpectCallFuncOverload(
+            static_cast< subscribePresenceSig2 >(OC::OCPlatform::subscribePresence))
+                     .With(_, HOST, resType, CONTYPE, _).Return(OC_STACK_OK);
+
+    PresenceSubscriber subscriber{ HOST, resType, CONTYPE, SubscribeCallback() };
+}
+
+TEST_F(PresenceSubscriberTest, ConstructorThrowsIfResultIsNotOK)
+{
+    mocks.ExpectCallFuncOverload(
+            static_cast< subscribePresenceSig1 >(OC::OCPlatform::subscribePresence))
+                    .Return(OC_STACK_ERROR);
+
+    ASSERT_THROW(PresenceSubscriber(HOST, CONTYPE, SubscribeCallback()), PlatformException);
+}
+
+TEST_F(PresenceSubscriberTest, IsSubscribingIfConstructedWithoutException)
+{
+    PresenceSubscriber subscriber{ HOST, CONTYPE, SubscribeCallback() };
+
+    ASSERT_TRUE(subscriber.isSubscribing());
+}
+
+TEST_F(PresenceSubscriberTest, IsSubscribingOfMovedSubscriberReturnsFalse)
+{
+    PresenceSubscriber subscriber{ HOST, CONTYPE, SubscribeCallback() };
+
+    PresenceSubscriber newSubscriber{ std::move(subscriber) };
+
+    ASSERT_FALSE(subscriber.isSubscribing());
+}
+
+TEST_F(PresenceSubscriberTest, IsSubscribingOfMovedSubscriberWithAssignmentReturnsFalse)
+{
+    PresenceSubscriber subscriber{ HOST, CONTYPE, SubscribeCallback() };
+
+    PresenceSubscriber newSubscriber;
+
+    newSubscriber = std::move(subscriber);
+
+    ASSERT_FALSE(subscriber.isSubscribing());
+}
+
+TEST_F(PresenceSubscriberTest, UnsubscribeWillBeCalledWhenSubscriberIsDestoryed)
+{
+    mocks.ExpectCallFunc(OC::OCPlatform::unsubscribePresence).Return(OC_STACK_OK);
+    {
+        PresenceSubscriber subscriber{ HOST, CONTYPE, SubscribeCallback() };
+    }
+}