unit-test: Fix bug when endpoint empty
Could debate the proper handling of this situations but this commit at
least makes it consistent for the forward and reverse path
Change-Id: If0a8a16473c150f4d4361f9ca9aad9f06c391157
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
diff --git a/src/associations.cpp b/src/associations.cpp
index 6982b52..b4c0fae 100644
--- a/src/associations.cpp
+++ b/src/associations.cpp
@@ -156,18 +156,17 @@
std::string endpoint;
std::tie(forward, reverse, endpoint) = association;
+ if (endpoint.empty())
+ {
+ std::cerr << "Found invalid association on path " << path << "\n";
+ continue;
+ }
if (forward.size())
{
objects[path + "/" + forward].emplace(endpoint);
}
if (reverse.size())
{
- if (endpoint.empty())
- {
- std::cerr << "Found invalid association on path " << path
- << "\n";
- continue;
- }
objects[endpoint + "/" + reverse].emplace(path);
}
}