GENIUS-109: make checkDmacSameDPN more robust

This sorts the actions before comparing the flows, avoiding issues
when upstream changes cause the action order to change in the data
structure.

Change-Id: Ia4b62e2af5bc432f076432422236adacb097885b
Signed-off-by: Stephen Kitt <skitt@redhat.com>

diff --git a/vpnservice/elanmanager/elanmanager-impl/src/test/java/org/opendaylight/netvirt/elanmanager/tests/ElanServiceTest.java b/vpnservice/elanmanager/elanmanager-impl/src/test/java/org/opendaylight/netvirt/elanmanager/tests/ElanServiceTest.java
index bb9e3365a6e609a6b5ee6db3a01c2286b31eb060..6d1c007d1db2eb4d8c724b2ac20590c827f015cf 100644 (file)
--- a/vpnservice/elanmanager/elanmanager-impl/src/test/java/org/opendaylight/netvirt/elanmanager/tests/ElanServiceTest.java
+++ b/vpnservice/elanmanager/elanmanager-impl/src/test/java/org/opendaylight/netvirt/elanmanager/tests/ElanServiceTest.java
@@ -157,7 +157,7 @@ public class ElanServiceTest extends  ElanServiceTestBase {
         flowDst = getFlowWithoutCookie(flowDst);
 
         Flow expected = ExpectedObjects.checkDmacOfSameDpn(flowId, interfaceInfo, actualElanInstances);
-        AssertDataObjects.assertEqualBeans(expected, flowDst);
+        AssertDataObjects.assertEqualBeans(getSortedActions(expected), getSortedActions(flowDst));
     }
 
     @Test public void checkDmacOfOtherDPN() throws Exception {