commit ea21d995389412b4cea411f647176641b5cf0a9a
author Patrick Williams <patrick@stwcx.xyz> Fri Nov 22 17:06:35 2024 -0500
committer Patrick Williams <patrick@stwcx.xyz> Thu Dec 05 02:33:34 2024 -0500
tree ca25ff3574733ebf9bb95ddeff5f926f107a947f
parent 5402fa6866b281349385360673bee2e2f0dd09b1

entry: use map for metadata in ctor

Transition the entry constructor to use the `map<string,string>`
rather than the `vector<string>`.

Tested: UTs pass and daemon still creates logs.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icf3ead163cc99a1498734d33335fb6da52e6c98c

test/elog_quiesce_test.cpp

diff --git a/test/elog_quiesce_test.cpp b/test/elog_quiesce_test.cpp
index 7894f95..9c822df 100644
--- a/test/elog_quiesce_test.cpp
+++ b/test/elog_quiesce_test.cpp
@@ -41,7 +41,8 @@
     std::string message{"test error"};
     std::string fwLevel{"level42"};
     std::string path{"/tmp/99"};
-    std::vector<std::string> testData{"no", "callout"};
+    std::map<std::string, std::string> testData{{"no", "no"},
+                                                {"callout", "callout"}};
     phosphor::logging::AssociationList associations{};
 
     Entry elog{mockedBus,
@@ -67,9 +68,9 @@
     std::string message{"test error"};
     std::string fwLevel{"level42"};
     std::string path{"/tmp/99"};
-    std::vector<std::string> testData{
-        "CALLOUT_INVENTORY_PATH=/xyz/openbmc_project/inventory/system/chassis/"
-        "motherboard/powersupply0/"};
+    std::map<std::string, std::string> testData{
+        {"CALLOUT_INVENTORY_PATH",
+         "/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply0/"}};
     phosphor::logging::AssociationList associations{};
 
     Entry elog{mockedBus,
@@ -95,9 +96,9 @@
     std::string message{"test error"};
     std::string fwLevel{"level42"};
     std::string path{"/tmp/99"};
-    std::vector<std::string> testData{
-        "CALLOUT_INVENTORY_PATH=/xyz/openbmc_project/inventory/system/chassis/"
-        "motherboard/powersupply0/"};
+    std::map<std::string, std::string> testData{
+        {"CALLOUT_INVENTORY_PATH",
+         "/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply0/"}};
     phosphor::logging::AssociationList associations{};
 
     // Ensure D-Bus object created for this blocking error
@@ -152,9 +153,9 @@
     std::string message{"test error"};
     std::string fwLevel{"level42"};
     std::string path{"/tmp/99"};
-    std::vector<std::string> testData{
-        "CALLOUT_INVENTORY_PATH=/xyz/openbmc_project/inventory/system/chassis/"
-        "motherboard/powersupply0/"};
+    std::map<std::string, std::string> testData{
+        {"CALLOUT_INVENTORY_PATH",
+         "/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply0/"}};
     phosphor::logging::AssociationList associations{};
 
     // Ensure D-Bus object created for this blocking error
@@ -211,9 +212,9 @@
     std::string message{"test error"};
     std::string fwLevel{"level42"};
     std::string path{"/tmp/99"};
-    std::vector<std::string> testData{
-        "CALLOUT_INVENTORY_PATH=/xyz/openbmc_project/inventory/system/chassis/"
-        "motherboard/powersupply0/"};
+    std::map<std::string, std::string> testData{
+        {"CALLOUT_INVENTORY_PATH",
+         "/xyz/openbmc_project/inventory/system/chassis/motherboard/powersupply0/"}};
     phosphor::logging::AssociationList associations{};
 
     // Ensure D-Bus object created for this blocking error