Fix assert

Arrays.equals still looks at order, which we don't want to

refs #464

src/test/ProphetRouterTest.java

@@ -277,8 +277,10 @@ public class ProphetRouterTest extends AbstractRouterTest {
             sentIds.add(this.mc.getLastMsg().getId());
             this.h1.update(true);
         }
-        String[] expectedIds = { messageToH4.getId(), newMulticast.getId() };
-        assertTrue("Expected different message set to be sent.", Arrays.equals(sentIds.toArray(), expectedIds));
+        HashSet<String> expectedIds = new HashSet<>(2);
+        expectedIds.add(messageToH4.getId());
+        expectedIds.add(newMulticast.getId());
+        assertEquals("Expected different message set to be sent.", sentIds, expectedIds);
     }
 
     /**

[Ghost User]

SonarQube analysis reported 1 issue

• ⚠ 1 major

1. ⚠ Remove this unused private "getMaxPredFor" method. 📘