Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 1 | #include "src/associations.hpp" |
| 2 | |
Andrew Geissler | b04f033 | 2019-02-08 14:07:56 -0600 | [diff] [blame] | 3 | #include "src/test/util/asio_server_class.hpp" |
Andrew Geissler | bb7b592 | 2019-02-08 14:16:48 -0600 | [diff] [blame] | 4 | #include "src/test/util/association_objects.hpp" |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 5 | #include "src/test/util/debug_output.hpp" |
Andrew Geissler | b04f033 | 2019-02-08 14:07:56 -0600 | [diff] [blame] | 6 | |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 7 | #include <sdbusplus/asio/connection.hpp> |
| 8 | #include <sdbusplus/asio/object_server.hpp> |
| 9 | |
| 10 | #include <gtest/gtest.h> |
| 11 | |
Andrew Geissler | b04f033 | 2019-02-08 14:07:56 -0600 | [diff] [blame] | 12 | class TestAssociations : public AsioServerClassTest |
Brad Bishop | 2352088 | 2022-05-26 21:39:53 -0400 | [diff] [blame] | 13 | {}; |
Andrew Geissler | b04f033 | 2019-02-08 14:07:56 -0600 | [diff] [blame] | 14 | sdbusplus::asio::object_server* TestAssociations::AsioServerClassTest::server = |
| 15 | nullptr; |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 16 | |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 17 | // Verify call when path is not in associated owners |
| 18 | TEST_F(TestAssociations, SourcePathNotInAssociations) |
| 19 | { |
| 20 | EXPECT_NE(nullptr, server); |
| 21 | std::string sourcePath = "/xyz/openbmc_project/no/association"; |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 22 | AssociationMaps assocMaps; |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 23 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 24 | removeAssociation(sourcePath, DEFAULT_DBUS_SVC, *server, assocMaps); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 25 | } |
| 26 | |
| 27 | // Verify call when owner is not in associated owners |
| 28 | TEST_F(TestAssociations, OwnerNotInAssociations) |
| 29 | { |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 30 | AssociationMaps assocMaps; |
| 31 | assocMaps.owners = createDefaultOwnerAssociation(); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 32 | |
| 33 | removeAssociation(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, *server, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 34 | assocMaps); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 35 | } |
| 36 | |
| 37 | // Verify call when path is not in associated interfaces |
| 38 | TEST_F(TestAssociations, PathNotInAssocInterfaces) |
| 39 | { |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 40 | AssociationMaps assocMaps; |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 41 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 42 | assocMaps.owners = createDefaultOwnerAssociation(); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 43 | |
| 44 | removeAssociation(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, *server, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 45 | assocMaps); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 46 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 47 | EXPECT_TRUE(assocMaps.owners.empty()); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 48 | } |
| 49 | |
| 50 | // Verify call when path is in associated interfaces |
| 51 | TEST_F(TestAssociations, PathIsInAssociatedInterfaces) |
| 52 | { |
| 53 | // Build up these objects so that an associated interface will match |
| 54 | // with the associated owner being removed |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 55 | AssociationMaps assocMaps; |
| 56 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 57 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 58 | |
| 59 | removeAssociation(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, *server, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 60 | assocMaps); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 61 | |
| 62 | // Verify owner association was deleted |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 63 | EXPECT_TRUE(assocMaps.owners.empty()); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 64 | |
| 65 | // Verify endpoint was deleted from interface association |
| 66 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 67 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 68 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 69 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 70 | EXPECT_EQ(intfEndpoints.size(), 0); |
| 71 | } |
| 72 | |
| 73 | // Verify call when path is in associated interfaces, with extra endpoints |
| 74 | TEST_F(TestAssociations, PathIsInAssociatedInterfacesExtraEndpoints) |
| 75 | { |
| 76 | // Build up these objects so that an associated interface will match |
| 77 | // with the associated owner being removed |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 78 | AssociationMaps assocMaps; |
| 79 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 80 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 81 | |
| 82 | // Add another endpoint to the assoc interfaces |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 83 | addEndpointToInterfaceAssociation(assocMaps.ifaces); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 84 | |
| 85 | removeAssociation(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, *server, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 86 | assocMaps); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 87 | |
| 88 | // Verify owner association was deleted |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 89 | EXPECT_TRUE(assocMaps.owners.empty()); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 90 | |
| 91 | // Verify all endpoints are deleted since source path was deleted |
| 92 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 93 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 94 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 95 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | 271b7dd | 2019-02-05 11:23:30 -0600 | [diff] [blame] | 96 | EXPECT_EQ(intfEndpoints.size(), 0); |
| 97 | } |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 98 | |
| 99 | // Verify no associations or endpoints removed when the change is identical |
| 100 | TEST_F(TestAssociations, checkAssociationEndpointRemovesNoEpRemove) |
| 101 | { |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 102 | AssociationPaths newAssocPaths = { |
| 103 | {DEFAULT_FWD_PATH, {DEFAULT_ENDPOINT}}, |
| 104 | {DEFAULT_REV_PATH, {DEFAULT_SOURCE_PATH}}}; |
| 105 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 106 | AssociationMaps assocMaps; |
| 107 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 108 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 109 | |
| 110 | checkAssociationEndpointRemoves(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 111 | newAssocPaths, *server, assocMaps); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 112 | |
| 113 | // Verify endpoints were not deleted because they matche with what was |
| 114 | // in the original |
| 115 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 116 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 117 | EXPECT_EQ(intfEndpoints.size(), 1); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 118 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 119 | EXPECT_EQ(intfEndpoints.size(), 1); |
| 120 | } |
| 121 | |
| 122 | // Verify endpoint is removed when assoc path is different |
| 123 | TEST_F(TestAssociations, checkAssociationEndpointRemovesEpRemoveApDiff) |
| 124 | { |
| 125 | AssociationPaths newAssocPaths = {{"/different/path", {DEFAULT_ENDPOINT}}}; |
| 126 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 127 | AssociationMaps assocMaps; |
| 128 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 129 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 130 | |
| 131 | checkAssociationEndpointRemoves(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 132 | newAssocPaths, *server, assocMaps); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 133 | |
| 134 | // Verify initial endpoints were deleted because the new path |
| 135 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 136 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 137 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 138 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 139 | EXPECT_EQ(intfEndpoints.size(), 0); |
| 140 | } |
| 141 | |
| 142 | // Verify endpoint is removed when endpoint is different |
| 143 | TEST_F(TestAssociations, checkAssociationEndpointRemovesEpRemoveEpChanged) |
| 144 | { |
| 145 | AssociationPaths newAssocPaths = { |
| 146 | {DEFAULT_FWD_PATH, {DEFAULT_ENDPOINT + "/different"}}, |
| 147 | {DEFAULT_REV_PATH, {DEFAULT_SOURCE_PATH + "/different"}}}; |
| 148 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 149 | AssociationMaps assocMaps; |
| 150 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 151 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 152 | |
| 153 | checkAssociationEndpointRemoves(DEFAULT_SOURCE_PATH, DEFAULT_DBUS_SVC, |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 154 | newAssocPaths, *server, assocMaps); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 155 | |
| 156 | // Verify initial endpoints were deleted because of different endpoints |
| 157 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 158 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 159 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 160 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | e4ab6c9 | 2019-02-21 15:07:27 -0600 | [diff] [blame] | 161 | EXPECT_EQ(intfEndpoints.size(), 0); |
| 162 | } |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 163 | |
| 164 | // Verify existing endpoint deleted when empty endpoint is provided |
| 165 | TEST_F(TestAssociations, associationChangedEmptyEndpoint) |
| 166 | { |
Andrew Geissler | eecc196 | 2020-06-10 19:58:35 -0500 | [diff] [blame] | 167 | std::vector<Association> associations = { |
| 168 | {"inventory_cee", "error_cee", ""}}; |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 169 | interface_map_type interfaceMap; |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 170 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 171 | AssociationMaps assocMaps; |
| 172 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 173 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 174 | |
| 175 | // Empty endpoint will result in deletion of corresponding assocInterface |
| 176 | associationChanged(*server, associations, DEFAULT_SOURCE_PATH, |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 177 | DEFAULT_DBUS_SVC, interfaceMap, assocMaps); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 178 | |
Andrew Geissler | 0a560a5 | 2019-03-22 10:59:07 -0500 | [diff] [blame] | 179 | // Both of these should be 0 since we have an invalid endpoint |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 180 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 181 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 0a560a5 | 2019-03-22 10:59:07 -0500 | [diff] [blame] | 182 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 183 | intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_REV_PATH]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 184 | EXPECT_EQ(intfEndpoints.size(), 0); |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 185 | |
| 186 | EXPECT_EQ(assocMaps.pending.size(), 0); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 187 | } |
| 188 | |
| 189 | // Add a new association with endpoint |
| 190 | TEST_F(TestAssociations, associationChangedAddNewAssoc) |
| 191 | { |
| 192 | std::vector<Association> associations = { |
| 193 | {"abc", "def", "/xyz/openbmc_project/new/endpoint"}}; |
| 194 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 195 | AssociationMaps assocMaps; |
| 196 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 197 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 198 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 199 | // Make it look like the assoc endpoints are on D-Bus |
| 200 | interface_map_type interfaceMap = { |
| 201 | {"/new/source/path", {{DEFAULT_DBUS_SVC, {"a"}}}}, |
| 202 | {"/xyz/openbmc_project/new/endpoint", {{DEFAULT_DBUS_SVC, {"a"}}}}}; |
| 203 | |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 204 | associationChanged(*server, associations, "/new/source/path", |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 205 | DEFAULT_DBUS_SVC, interfaceMap, assocMaps); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 206 | |
| 207 | // Two source paths |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 208 | EXPECT_EQ(assocMaps.owners.size(), 2); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 209 | |
| 210 | // Four interfaces |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 211 | EXPECT_EQ(assocMaps.ifaces.size(), 4); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 212 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 213 | // Nothing pending |
| 214 | EXPECT_EQ(assocMaps.pending.size(), 0); |
| 215 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 216 | // New endpoint so assocMaps.ifaces should be same size |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 217 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 218 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 219 | EXPECT_EQ(intfEndpoints.size(), 1); |
| 220 | } |
| 221 | |
| 222 | // Add a new association to empty objects |
| 223 | TEST_F(TestAssociations, associationChangedAddNewAssocEmptyObj) |
| 224 | { |
| 225 | std::string sourcePath = "/logging/entry/1"; |
| 226 | std::string owner = "xyz.openbmc_project.Test"; |
| 227 | std::vector<Association> associations = { |
Andrew Geissler | eecc196 | 2020-06-10 19:58:35 -0500 | [diff] [blame] | 228 | {"inventory_canaeo", "error_canaeo", |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 229 | "/xyz/openbmc_project/inventory/system/chassis"}}; |
| 230 | |
| 231 | // Empty objects because this test will ensure assocOwners adds the |
| 232 | // changed association and interface |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 233 | AssociationMaps assocMaps; |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 234 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 235 | // Make it look like the assoc endpoints are on D-Bus |
| 236 | interface_map_type interfaceMap = createDefaultInterfaceMap(); |
| 237 | |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 238 | associationChanged(*server, associations, DEFAULT_SOURCE_PATH, |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 239 | DEFAULT_DBUS_SVC, interfaceMap, assocMaps); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 240 | |
| 241 | // New associations so ensure it now contains a single entry |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 242 | EXPECT_EQ(assocMaps.owners.size(), 1); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 243 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 244 | // Nothing pending |
| 245 | EXPECT_EQ(assocMaps.pending.size(), 0); |
| 246 | |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 247 | // Verify corresponding assoc paths each have one endpoint in assoc |
| 248 | // interfaces and that those endpoints match |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 249 | auto singleOwner = assocMaps.owners[DEFAULT_SOURCE_PATH]; |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 250 | auto singleIntf = singleOwner[DEFAULT_DBUS_SVC]; |
| 251 | for (auto i : singleIntf) |
| 252 | { |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 253 | auto intfEndpoints = std::get<endpointsPos>(assocMaps.ifaces[i.first]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 254 | EXPECT_EQ(intfEndpoints.size(), 1); |
| 255 | EXPECT_EQ(intfEndpoints[0], *i.second.begin()); |
| 256 | } |
| 257 | } |
| 258 | |
| 259 | // Add a new association to same source path but with new owner |
| 260 | TEST_F(TestAssociations, associationChangedAddNewAssocNewOwner) |
| 261 | { |
| 262 | std::string newOwner = "xyz.openbmc_project.Test2"; |
| 263 | std::vector<Association> associations = { |
Andrew Geissler | eecc196 | 2020-06-10 19:58:35 -0500 | [diff] [blame] | 264 | {"inventory_canano", "error_canano", |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 265 | "/xyz/openbmc_project/inventory/system/chassis"}}; |
| 266 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 267 | // Make it look like the assoc endpoints are on D-Bus |
| 268 | interface_map_type interfaceMap = createDefaultInterfaceMap(); |
| 269 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 270 | AssociationMaps assocMaps; |
| 271 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 272 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 273 | |
| 274 | associationChanged(*server, associations, DEFAULT_SOURCE_PATH, newOwner, |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 275 | interfaceMap, assocMaps); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 276 | |
| 277 | // New endpoint so assocOwners should be same size |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 278 | EXPECT_EQ(assocMaps.owners.size(), 1); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 279 | |
| 280 | // Ensure only one endpoint under first path |
| 281 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 282 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 283 | EXPECT_EQ(intfEndpoints.size(), 1); |
| 284 | |
| 285 | // Ensure the 2 new association endpoints are under the new owner |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 286 | auto a = assocMaps.owners.find(DEFAULT_SOURCE_PATH); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 287 | auto o = a->second.find(newOwner); |
| 288 | EXPECT_EQ(o->second.size(), 2); |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 289 | |
| 290 | // Nothing pending |
| 291 | EXPECT_EQ(assocMaps.pending.size(), 0); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 292 | } |
| 293 | |
| 294 | // Add a new association to existing interface path |
| 295 | TEST_F(TestAssociations, associationChangedAddNewAssocSameInterface) |
| 296 | { |
| 297 | std::vector<Association> associations = { |
| 298 | {"abc", "error", "/xyz/openbmc_project/inventory/system/chassis"}}; |
| 299 | |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 300 | // Make it look like the assoc endpoints are on D-Bus |
| 301 | interface_map_type interfaceMap = createDefaultInterfaceMap(); |
| 302 | |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 303 | AssociationMaps assocMaps; |
| 304 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 305 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 306 | |
| 307 | associationChanged(*server, associations, DEFAULT_SOURCE_PATH, |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 308 | DEFAULT_DBUS_SVC, interfaceMap, assocMaps); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 309 | |
| 310 | // Should have 3 entries in AssociationInterfaces, one is just missing an |
| 311 | // endpoint |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 312 | EXPECT_EQ(assocMaps.ifaces.size(), 3); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 313 | |
| 314 | // Change to existing interface so it will be removed here |
| 315 | auto intfEndpoints = |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 316 | std::get<endpointsPos>(assocMaps.ifaces[DEFAULT_FWD_PATH]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 317 | EXPECT_EQ(intfEndpoints.size(), 0); |
| 318 | |
| 319 | // The new endpoint should exist though in it's place |
| 320 | intfEndpoints = std::get<endpointsPos>( |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 321 | assocMaps.ifaces[DEFAULT_SOURCE_PATH + "/" + "abc"]); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 322 | EXPECT_EQ(intfEndpoints.size(), 1); |
| 323 | |
| 324 | // Added to an existing owner path so still 1 |
Matt Spinler | e2359fb | 2019-04-05 14:11:33 -0500 | [diff] [blame] | 325 | EXPECT_EQ(assocMaps.owners.size(), 1); |
Matt Spinler | e0b0e3a | 2019-04-08 10:39:23 -0500 | [diff] [blame] | 326 | |
| 327 | EXPECT_EQ(assocMaps.pending.size(), 0); |
Andrew Geissler | 491f9ac | 2019-02-22 15:23:33 -0600 | [diff] [blame] | 328 | } |
Matt Spinler | 9f1adbc | 2019-04-08 10:47:19 -0500 | [diff] [blame] | 329 | |
| 330 | // Add 2 pending associations |
| 331 | TEST_F(TestAssociations, addPendingAssocs) |
| 332 | { |
| 333 | AssociationMaps assocMaps; |
| 334 | |
| 335 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT, |
| 336 | "error", DEFAULT_DBUS_SVC, assocMaps); |
| 337 | |
| 338 | EXPECT_TRUE(assocMaps.ifaces.empty()); |
| 339 | EXPECT_TRUE(assocMaps.owners.empty()); |
| 340 | |
| 341 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 342 | |
| 343 | addPendingAssociation("some/other/path", "inventory", DEFAULT_ENDPOINT, |
| 344 | "error", DEFAULT_DBUS_SVC, assocMaps); |
| 345 | |
| 346 | EXPECT_TRUE(assocMaps.ifaces.empty()); |
| 347 | EXPECT_TRUE(assocMaps.owners.empty()); |
| 348 | |
| 349 | EXPECT_EQ(assocMaps.pending.size(), 2); |
| 350 | } |
| 351 | |
| 352 | // Test adding a new endpoint to a pending association |
| 353 | TEST_F(TestAssociations, addPendingAssocsNewEndpoints) |
| 354 | { |
| 355 | AssociationMaps assocMaps; |
| 356 | |
| 357 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT, |
| 358 | "error", DEFAULT_DBUS_SVC, assocMaps); |
| 359 | |
| 360 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 361 | |
| 362 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", |
| 363 | "some/other/endpoint", "error", DEFAULT_DBUS_SVC, |
| 364 | assocMaps); |
| 365 | |
| 366 | // Same pending path, so still just 1 entry |
| 367 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 368 | |
| 369 | auto assoc = assocMaps.pending.find(DEFAULT_SOURCE_PATH); |
| 370 | EXPECT_NE(assoc, assocMaps.pending.end()); |
| 371 | |
| 372 | auto& endpoints = assoc->second; |
| 373 | EXPECT_EQ(endpoints.size(), 2); |
| 374 | } |
| 375 | |
| 376 | // Test adding a new owner to a pending association |
| 377 | TEST_F(TestAssociations, addPendingAssocsNewOwner) |
| 378 | { |
| 379 | AssociationMaps assocMaps; |
| 380 | |
| 381 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT, |
| 382 | "error", DEFAULT_DBUS_SVC, assocMaps); |
| 383 | |
| 384 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 385 | |
| 386 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT, |
| 387 | "error", "new owner", assocMaps); |
| 388 | |
| 389 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 390 | |
| 391 | auto assoc = assocMaps.pending.find(DEFAULT_SOURCE_PATH); |
| 392 | EXPECT_NE(assoc, assocMaps.pending.end()); |
| 393 | |
| 394 | auto& endpoints = assoc->second; |
| 395 | EXPECT_EQ(endpoints.size(), 2); |
| 396 | } |
| 397 | |
| 398 | // Add a pending association inside associationChanged |
| 399 | TEST_F(TestAssociations, associationChangedPending) |
| 400 | { |
| 401 | std::vector<Association> associations = { |
| 402 | {"abc", "def", "/xyz/openbmc_project/new/endpoint"}}; |
| 403 | |
| 404 | AssociationMaps assocMaps; |
| 405 | interface_map_type interfaceMap; |
| 406 | |
| 407 | associationChanged(*server, associations, "/new/source/path", |
| 408 | DEFAULT_DBUS_SVC, interfaceMap, assocMaps); |
| 409 | |
| 410 | // No associations were actually added |
| 411 | EXPECT_EQ(assocMaps.owners.size(), 0); |
| 412 | EXPECT_EQ(assocMaps.ifaces.size(), 0); |
| 413 | |
| 414 | // 1 pending association |
| 415 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 416 | } |
Matt Spinler | cb9bcdb | 2019-04-08 10:58:49 -0500 | [diff] [blame] | 417 | |
| 418 | // Test removing pending associations |
| 419 | TEST_F(TestAssociations, testRemoveFromPendingAssociations) |
| 420 | { |
| 421 | AssociationMaps assocMaps; |
| 422 | |
| 423 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", DEFAULT_ENDPOINT, |
| 424 | "error", DEFAULT_DBUS_SVC, assocMaps); |
| 425 | |
| 426 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory", |
| 427 | "some/other/endpoint", "error", DEFAULT_DBUS_SVC, |
| 428 | assocMaps); |
| 429 | |
| 430 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 431 | |
| 432 | removeFromPendingAssociations("some/other/endpoint", assocMaps); |
| 433 | |
| 434 | // Still 1 pending entry, but down to 1 endpoint |
| 435 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 436 | |
| 437 | auto assoc = assocMaps.pending.find(DEFAULT_SOURCE_PATH); |
| 438 | EXPECT_NE(assoc, assocMaps.pending.end()); |
| 439 | auto& endpoints = assoc->second; |
| 440 | EXPECT_EQ(endpoints.size(), 1); |
| 441 | |
| 442 | // Now nothing pending |
| 443 | removeFromPendingAssociations(DEFAULT_ENDPOINT, assocMaps); |
| 444 | EXPECT_EQ(assocMaps.pending.size(), 0); |
| 445 | } |
Matt Spinler | 11401e2 | 2019-04-08 13:13:25 -0500 | [diff] [blame] | 446 | |
| 447 | // Test moving a pending association to a real one |
| 448 | TEST_F(TestAssociations, checkIfPending) |
| 449 | { |
| 450 | AssociationMaps assocMaps; |
| 451 | interface_map_type interfaceMap = { |
| 452 | {DEFAULT_SOURCE_PATH, {{DEFAULT_DBUS_SVC, {"a"}}}}, |
| 453 | {DEFAULT_ENDPOINT, {{DEFAULT_DBUS_SVC, {"b"}}}}}; |
| 454 | |
Andrew Geissler | eecc196 | 2020-06-10 19:58:35 -0500 | [diff] [blame] | 455 | addPendingAssociation(DEFAULT_SOURCE_PATH, "inventory_cip", |
| 456 | DEFAULT_ENDPOINT, "error_cip", DEFAULT_DBUS_SVC, |
| 457 | assocMaps); |
Matt Spinler | 11401e2 | 2019-04-08 13:13:25 -0500 | [diff] [blame] | 458 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 459 | |
| 460 | // Move the pending association to a real association |
| 461 | checkIfPendingAssociation(DEFAULT_SOURCE_PATH, interfaceMap, assocMaps, |
| 462 | *server); |
| 463 | |
| 464 | EXPECT_TRUE(assocMaps.pending.empty()); |
| 465 | EXPECT_EQ(assocMaps.owners.size(), 1); |
| 466 | EXPECT_EQ(assocMaps.ifaces.size(), 2); |
| 467 | |
| 468 | // This shouldn't do anything, since /new/path isn't pending |
| 469 | checkIfPendingAssociation("/new/path", interfaceMap, assocMaps, *server); |
| 470 | EXPECT_TRUE(assocMaps.pending.empty()); |
| 471 | EXPECT_EQ(assocMaps.owners.size(), 1); |
| 472 | EXPECT_EQ(assocMaps.ifaces.size(), 2); |
| 473 | } |
Matt Spinler | 7f8fd1f | 2019-04-08 15:21:59 -0500 | [diff] [blame] | 474 | |
| 475 | TEST_F(TestAssociations, findAssociations) |
| 476 | { |
| 477 | std::vector<std::tuple<std::string, Association>> associationData; |
| 478 | AssociationMaps assocMaps; |
| 479 | |
| 480 | assocMaps.owners = { |
| 481 | {"pathA", |
| 482 | {{"ownerA", |
| 483 | {{"pathA/typeA", {"endpointA", "endpointB"}}, |
| 484 | {"endpointA/type0", {"pathA"}}}}}}, |
| 485 | |
| 486 | {"pathJ", |
| 487 | {{"ownerC", |
| 488 | {{"pathJ/typeA", {"endpointF"}}, {"endpointF/type0", {"pathJ"}}}}}}, |
| 489 | |
| 490 | {"pathX", |
| 491 | {{"ownerB", |
| 492 | {{"pathX/typeB", {"endpointA"}}, {"endpointA/type1", {"pathX"}}}}}}}; |
| 493 | |
| 494 | findAssociations("endpointA", assocMaps, associationData); |
| 495 | ASSERT_EQ(associationData.size(), 2); |
| 496 | |
| 497 | { |
| 498 | auto ad = std::find_if( |
| 499 | associationData.begin(), associationData.end(), |
| 500 | [](const auto& ad) { return std::get<0>(ad) == "ownerA"; }); |
| 501 | ASSERT_NE(ad, associationData.end()); |
| 502 | |
| 503 | auto& a = std::get<1>(*ad); |
| 504 | ASSERT_EQ(std::get<0>(a), "type0"); |
| 505 | ASSERT_EQ(std::get<1>(a), "typeA"); |
| 506 | ASSERT_EQ(std::get<2>(a), "pathA"); |
| 507 | } |
| 508 | { |
| 509 | auto ad = std::find_if( |
| 510 | associationData.begin(), associationData.end(), |
| 511 | [](const auto& ad) { return std::get<0>(ad) == "ownerB"; }); |
| 512 | ASSERT_NE(ad, associationData.end()); |
| 513 | |
| 514 | auto& a = std::get<1>(*ad); |
| 515 | ASSERT_EQ(std::get<0>(a), "type1"); |
| 516 | ASSERT_EQ(std::get<1>(a), "typeB"); |
| 517 | ASSERT_EQ(std::get<2>(a), "pathX"); |
| 518 | } |
| 519 | } |
Matt Spinler | 9c3d285 | 2019-04-08 15:57:19 -0500 | [diff] [blame] | 520 | |
| 521 | TEST_F(TestAssociations, moveAssocToPendingNoOp) |
| 522 | { |
| 523 | AssociationMaps assocMaps; |
| 524 | |
| 525 | // Not an association, so it shouldn't do anything |
| 526 | moveAssociationToPending(DEFAULT_ENDPOINT, assocMaps, *server); |
| 527 | |
| 528 | EXPECT_TRUE(assocMaps.pending.empty()); |
| 529 | EXPECT_TRUE(assocMaps.owners.empty()); |
| 530 | EXPECT_TRUE(assocMaps.ifaces.empty()); |
| 531 | } |
| 532 | |
| 533 | TEST_F(TestAssociations, moveAssocToPending) |
| 534 | { |
| 535 | AssociationMaps assocMaps; |
| 536 | assocMaps.owners = createDefaultOwnerAssociation(); |
| 537 | assocMaps.ifaces = createDefaultInterfaceAssociation(server); |
| 538 | |
| 539 | moveAssociationToPending(DEFAULT_ENDPOINT, assocMaps, *server); |
| 540 | |
| 541 | // Check it's now pending |
| 542 | EXPECT_EQ(assocMaps.pending.size(), 1); |
| 543 | EXPECT_EQ(assocMaps.pending.begin()->first, DEFAULT_ENDPOINT); |
| 544 | |
| 545 | // No more assoc owners |
| 546 | EXPECT_TRUE(assocMaps.owners.empty()); |
| 547 | |
| 548 | // Check the association interfaces were removed |
| 549 | { |
| 550 | auto assocs = assocMaps.ifaces.find(DEFAULT_FWD_PATH); |
| 551 | auto& iface = std::get<ifacePos>(assocs->second); |
| 552 | auto& endpoints = std::get<endpointsPos>(assocs->second); |
| 553 | |
| 554 | EXPECT_EQ(iface.get(), nullptr); |
| 555 | EXPECT_TRUE(endpoints.empty()); |
| 556 | } |
| 557 | { |
| 558 | auto assocs = assocMaps.ifaces.find(DEFAULT_REV_PATH); |
| 559 | auto& iface = std::get<ifacePos>(assocs->second); |
| 560 | auto& endpoints = std::get<endpointsPos>(assocs->second); |
| 561 | |
| 562 | EXPECT_EQ(iface.get(), nullptr); |
| 563 | EXPECT_TRUE(endpoints.empty()); |
| 564 | } |
| 565 | } |