Fix MessageTestCase to compare elements inside collection.

Defined assertSameElements method which converts collections to Sets and
only then tries to compare them. This method has to be moved somewhere else
but there is no apparent place for at the moment.

Signed-off-by: Grzegorz Kossakowski <grek@google.com>

1 files changed, 8 insertions, 5 deletions

diff --git a/src/test/scala/com/google/gimd/MessageTestCase.scala b/src/test/scala/com/google/gimd/MessageTestCase.scala
index a4aa3f9..ac8f527 100644
--- a/src/test/scala/com/google/gimd/MessageTestCase.scala
+++ b/src/test/scala/com/google/gimd/MessageTestCase.scala
@@ -21,12 +21,12 @@ class MessageTestCase {
 
   @Test
   def getAll {
-    val list = List(Field("name1", 2), Field("name1", "v1"))
+    val set = Set(Field("name1", 2), Field("name1", "v1"))
     val name0 = Field("name0", 0)
-    val message = Message(List(name0) ++ list ++ List(Field("name2", 3)))
-    assertEquals(List(name0), message.all("name0"))
-    assertEquals(list, message.all("name1"))
-    assertEquals(Nil, message.all("nonExistingName"))
+    val message = Message(List(name0) ++ set ++ List(Field("name2", 3)))
+    assertSameElements(Set(name0), message.all("name0"))
+    assertSameElements(set, message.all("name1"))
+    assertSameElements(Set(), message.all("nonExistingName"))
   }
 
   @Test{val expected = classOf[NoSuchElementException]}
@@ -54,4 +54,7 @@ class MessageTestCase {
     assertEquals(Field("name", "value"), message.one("name"))
   }
 
+  private def assertSameElements[T](a: Iterable[T], b: Iterable[T]) =
+    org.junit.Assert.assertEquals(Set(a.toSeq: _*), Set(b.toSeq: _*))
+
 }