fix and add unittests - draft

Author: aviramd

lib/ClientSai.cpp

@@ -1557,7 +1557,8 @@ sai_status_t ClientSai::dbgGenerateDump(
     SWSS_LOG_ENTER();
     REDIS_CHECK_API_INITIALIZED();
 
-    const std::vector<swss::FieldValueTuple> entry =
+    return SAI_STATUS_NOT_IMPLEMENTED;
+    /*const std::vector<swss::FieldValueTuple> entry =
     {
         swss::FieldValueTuple("DBG_GENERATE_DUMP", dump_file_name),
     };
@@ -1567,5 +1568,5 @@ sai_status_t ClientSai::dbgGenerateDump(
     m_communicationChannel->set(key, entry, REDIS_ASIC_STATE_COMMAND_DBG_GEN_DUMP);
 
     swss::KeyOpFieldsValuesTuple kco;
-    return m_communicationChannel->wait(REDIS_ASIC_STATE_COMMAND_DBG_GEN_DUMPRESPONSE, kco);
+    return m_communicationChannel->wait(REDIS_ASIC_STATE_COMMAND_DBG_GEN_DUMPRESPONSE, kco);*/
 }

proxylib/Sai.cpp

@@ -1131,7 +1131,7 @@ sai_status_t Sai::dbgGenerateDump(
 
     std::string key = "DBG_GEN_DUMP:01";
 
-    m_communicationChannel->set(key, entry, "dbg_gen_dump");
+    m_communicationChannel->set("api", entry, "dbg_gen_dump");
 
     /*swss::KeyOpFieldsValuesTuple kco;
 

unittest/lib/TestClientSai.cpp

@@ -56,7 +56,7 @@ TEST(ClientSai, dbgGenerateDump)
 
     sai.apiInitialize(0,&test_services);
 
-    const std::string filePath = "/var/log/dbgGenerateDump.log";
+    const std::string filePath = "/var/log/testDump.log";
 
-    EXPECT_EQ(SAI_STATUS_NOT_IMPLEMENTED, sai.dbgGenerateDump(filePath.c_str()));
-}
+    EXPECT_EQ(sai.dbgGenerateDump(filePath.c_str()), SAI_STATUS_NOT_IMPLEMENTED);
+}

unittest/lib/TestServerSai.cpp

@@ -49,3 +49,4 @@ TEST(ServerSai, bulkGet)
                 SAI_BULK_OP_ERROR_MODE_STOP_ON_ERROR,
                 statuses));
 }
+

unittest/syncd/TestVendorSai.cpp

@@ -1558,3 +1558,10 @@ TEST_F(VendorSaiTest, bulk_prefix_compression_entry)
     EXPECT_EQ(SAI_STATUS_NOT_SUPPORTED,
             m_vsai->bulkSet(0, e, nullptr, SAI_BULK_OP_ERROR_MODE_STOP_ON_ERROR, nullptr));
 }
+
+TEST_F(VendorSaiTest, dbgGenerateDump)
+{
+    const std::string filePath = "/var/log/testDump.log";
+
+    EXPECT_EQ(SAI_STATUS_SUCCESS, m_vsai->dbgGenerateDump(filePath.c_str()));
+}

unittest/vslib/TestSai.cpp

@@ -24,4 +24,13 @@ TEST(Sai, bulkGet)
                 attrs,
                 SAI_BULK_OP_ERROR_MODE_STOP_ON_ERROR,
                 statuses));
-}
+}
+
+TEST(Sai, dbgGenerateDump)
+{
+    Sai sai;
+
+    const std::string filePath = "/var/log/testDump.log";
+
+    EXPECT_EQ(SAI_STATUS_NOT_IMPLEMENTED, sai.dbgGenerateDump(filePath.c_str()));
+}

vslib/Sai.cpp

@@ -856,11 +856,11 @@ sai_status_t Sai::queryApiVersion(
 sai_status_t Sai::dbgGenerateDump(
         _In_ const char *dump_file_name)
 {
-    MUTEX();
     SWSS_LOG_ENTER();
-    VS_CHECK_API_INITIALIZED();
 
-    return  m_meta->dbgGenerateDump(dump_file_name);
+    SWSS_LOG_ERROR("not implemented, FIXME");
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
 }