commit 9f1adbc3a403131ef9a63b5d7f4b2d897ab176a9
author Matt Spinler <spinler@us.ibm.com> Mon Apr 08 10:47:19 2019 -0500
committer Matt Spinler <spinler@us.ibm.com> Thu Apr 25 08:54:34 2019 -0500
tree 976cfbb62a5e9f1139701fc8036a02867c9bc8ae
parent e0b0e3a2baf0063e66f2ec7e5105117e5edcf0ce

Add testcases for addPendingAssociation

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I8725eac5da302a0b893eed7fd0a104b7ff69615d

src/test/associations.cpp

diff --git a/src/test/associations.cpp b/src/test/associations.cpp
index e443043..cebd614 100644
--- a/src/test/associations.cpp
+++ b/src/test/associations.cpp
@@ -327,3 +327,91 @@
 
     EXPECT_EQ(assocMaps.pending.size(), 0);
 }
+
+// Add 2 pending associations
+TEST_F(TestAssociations, addPendingAssocs)
+{
+    AssociationMaps assocMaps;
+
+    addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT,
+                          "error", DEFAULT_DBUS_SVC, assocMaps);
+
+    EXPECT_TRUE(assocMaps.ifaces.empty());
+    EXPECT_TRUE(assocMaps.owners.empty());
+
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+
+    addPendingAssociation("some/other/path", "inventory", DEFAULT_ENDPOINT,
+                          "error", DEFAULT_DBUS_SVC, assocMaps);
+
+    EXPECT_TRUE(assocMaps.ifaces.empty());
+    EXPECT_TRUE(assocMaps.owners.empty());
+
+    EXPECT_EQ(assocMaps.pending.size(), 2);
+}
+
+// Test adding a new endpoint to a pending association
+TEST_F(TestAssociations, addPendingAssocsNewEndpoints)
+{
+    AssociationMaps assocMaps;
+
+    addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT,
+                          "error", DEFAULT_DBUS_SVC, assocMaps);
+
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+
+    addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory",
+                          "some/other/endpoint", "error", DEFAULT_DBUS_SVC,
+                          assocMaps);
+
+    // Same pending path, so still just 1 entry
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+
+    auto assoc = assocMaps.pending.find(DEFAULT_SOURCE_PATH);
+    EXPECT_NE(assoc, assocMaps.pending.end());
+
+    auto& endpoints = assoc->second;
+    EXPECT_EQ(endpoints.size(), 2);
+}
+
+// Test adding a new owner to a pending association
+TEST_F(TestAssociations, addPendingAssocsNewOwner)
+{
+    AssociationMaps assocMaps;
+
+    addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT,
+                          "error", DEFAULT_DBUS_SVC, assocMaps);
+
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+
+    addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT,
+                          "error", "new owner", assocMaps);
+
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+
+    auto assoc = assocMaps.pending.find(DEFAULT_SOURCE_PATH);
+    EXPECT_NE(assoc, assocMaps.pending.end());
+
+    auto& endpoints = assoc->second;
+    EXPECT_EQ(endpoints.size(), 2);
+}
+
+// Add a pending association inside associationChanged
+TEST_F(TestAssociations, associationChangedPending)
+{
+    std::vector<Association> associations = {
+        {"abc", "def", "/xyz/openbmc_project/new/endpoint"}};
+
+    AssociationMaps assocMaps;
+    interface_map_type interfaceMap;
+
+    associationChanged(*server, associations, "/new/source/path",
+                       DEFAULT_DBUS_SVC, interfaceMap, assocMaps);
+
+    // No associations were actually added
+    EXPECT_EQ(assocMaps.owners.size(), 0);
+    EXPECT_EQ(assocMaps.ifaces.size(), 0);
+
+    // 1 pending association
+    EXPECT_EQ(assocMaps.pending.size(), 1);
+}