commit 29b0e439c1536dff4348202069b568795367f108
author Ratan Gupta <ratagupt@in.ibm.com> Thu May 25 12:51:40 2017 +0530
committer Patrick Williams <patrick@stwcx.xyz> Thu Jun 08 13:41:37 2017 +0000
tree a8e3aa4f386017abfd73fbf7eb83a0e18f6e94ff
parent 594123ddf6b14d4696342223394f8e0f8c553432

Restructure the code so that it is unit testable

Change-Id: I2af7a26d054522beae82f58769f8c2420a22eca2
Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>

network_manager.cpp

diff --git a/network_manager.cpp b/network_manager.cpp
index a4a9b54..40fce1d 100644
--- a/network_manager.cpp
+++ b/network_manager.cpp
@@ -26,25 +26,31 @@
 namespace fs = std::experimental::filesystem;
 
 Manager::Manager(sdbusplus::bus::bus& bus, const char* objPath):
-    details::VLANCreateIface(bus, objPath, true)
+    details::VLANCreateIface(bus, objPath, true),
+    bus(bus),
+    objectPath(objPath)
 {
+}
+
+void Manager::createInterfaces()
+{
+
     auto interfaceInfoList = getInterfaceAddrs();
 
     for (const auto& intfInfo : interfaceInfoList)
-
     {
-
-        fs::path objectPath = std::string(OBJ_NETWORK);
-        objectPath /= intfInfo.first;
+        fs::path objPath = objectPath;
+        objPath /= intfInfo.first;
 
         this->interfaces.emplace(std::make_pair(
                                      intfInfo.first,
                                      std::make_unique<
                                      phosphor::network::EthernetInterface>
                                      (bus,
-                                      objectPath.string(),
-                                      false,
-                                      intfInfo.second)));
+                                      objPath.string(),
+                                      false)));
+
+        interfaces[intfInfo.first]->setAddressList(intfInfo.second);
     }
 }