Remove dependency on OC from serverBuilder tests using mock

To keep unit tests simple and quick

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

diff --git a/service/resource-manipulation/modules/serverBuilder/unittests/PrimitiveServerResourceTest.cpp b/service/resource-manipulation/modules/serverBuilder/unittests/PrimitiveServerResourceTest.cpp
index 58f81a9..925a712 100644 (file)
--- a/service/resource-manipulation/modules/serverBuilder/unittests/PrimitiveServerResourceTest.cpp
+++ b/service/resource-manipulation/modules/serverBuilder/unittests/PrimitiveServerResourceTest.cpp
@@ -44,22 +44,32 @@ static constexpr char RESOURCE_URI[]{ "a/test" };
 static constexpr char RESOURCE_TYPE[]{ "resourceType" };
 static constexpr char KEY[]{ "key" };
 
-TEST(ServerResourceBuilderTest, ThrowIfUriIsInvalid)
+TEST(ServerResourceBuilderCreateTest, ThrowIfUriIsInvalid)
 {
     ASSERT_THROW(PrimitiveServerResource::Builder("", "", "").create(), PlatformException);
 }
 
-TEST(ServerResourceBuilderTest, RegisterResourceWhenCallCreate)
+class ServerResourceBuilderTest: public Test
 {
+public:
     MockRepository mocks;
 
-    mocks.ExpectCallFuncOverload(
-            static_cast<registerResourceSig>(OCPlatform::registerResource)).Return(OC_STACK_OK);
+protected:
+    void SetUp() override
+    {
+        mocks.OnCallFuncOverload(static_cast<registerResourceSig>(OCPlatform::registerResource))
+                .Return(OC_STACK_OK);
+    }
+};
 
+TEST_F(ServerResourceBuilderTest, RegisterResourceWhenCallCreate)
+{
+    mocks.ExpectCallFuncOverload(static_cast<registerResourceSig>(OCPlatform::registerResource))
+            .Return(OC_STACK_OK);
     PrimitiveServerResource::Builder(RESOURCE_URI, RESOURCE_TYPE, "").create();
 }
 
-TEST(ServerResourceBuilderTest, ResourceServerHasPropertiesSetByBuilder)
+TEST_F(ServerResourceBuilderTest, ResourceServerHasPropertiesSetByBuilder)
 {
     auto serverResource = PrimitiveServerResource::Builder(RESOURCE_URI, RESOURCE_TYPE, "").
             setDiscoverable(false).setObservable(true).create();
@@ -68,7 +78,7 @@ TEST(ServerResourceBuilderTest, ResourceServerHasPropertiesSetByBuilder)
     EXPECT_TRUE(serverResource->isObservable());
 }
 
-TEST(ServerResourceBuilderTest, ResourceServerHasAttrsSetByBuilder)
+TEST_F(ServerResourceBuilderTest, ResourceServerHasAttrsSetByBuilder)
 {
     ResourceAttributes attrs;
     attrs[KEY] = 100;

diff --git a/service/resource-manipulation/modules/serverBuilder/unittests/RequestHandlerTest.cpp b/service/resource-manipulation/modules/serverBuilder/unittests/RequestHandlerTest.cpp
index b5bb49d..d5e258b 100644 (file)
--- a/service/resource-manipulation/modules/serverBuilder/unittests/RequestHandlerTest.cpp
+++ b/service/resource-manipulation/modules/serverBuilder/unittests/RequestHandlerTest.cpp
@@ -33,6 +33,9 @@ using namespace OIC::Service;
 constexpr char EXISTING[]{ "ext" };
 constexpr int ORIGIN_VALUE{ 100 };
 
+using RegisterResource = OCStackResult (*)(OCResourceHandle&, std::string&,
+        const std::string&, const std::string&, OC::EntityHandler, uint8_t);
+
 class SimpleRequestHandlerTest: public Test
 {
 public:
@@ -40,9 +43,14 @@ public:
 
     ResourceAttributes requestAttrs;
 
+    MockRepository mocks;
+
 protected:
     void SetUp() override
     {
+        mocks.OnCallFuncOverload(static_cast<RegisterResource>(OC::OCPlatform::registerResource))
+                .Return(OC_STACK_OK);
+
         server = PrimitiveServerResource::Builder("a/test", "resourceType", "").create();
 
         server->setAttribute(EXISTING, ORIGIN_VALUE);
@@ -91,9 +99,14 @@ public:
     ResourceAttributes requestAttrs;
     RequestHandler::Ptr setRequestProxyHandler;
 
+    MockRepository mocks;
+
 protected:
     void SetUp() override
     {
+        mocks.OnCallFuncOverload(static_cast<RegisterResource>(OC::OCPlatform::registerResource))
+                .Return(OC_STACK_OK);
+
         setRequestProxyHandler = make_shared<SetRequestProxyHandler>(
                 make_shared<SimpleRequestHandler>());