Commit 342ba888 authored by contextshuffling's avatar contextshuffling Committed by GitHub

Use LinkedHashSet for deterministic iterations (#2918)

parent a1d45a29
...@@ -94,13 +94,15 @@ public class ReleaseMessageServiceWithCacheTest { ...@@ -94,13 +94,15 @@ public class ReleaseMessageServiceWithCacheTest {
List<ReleaseMessage> latestReleaseMsgGroupByMsgContent = List<ReleaseMessage> latestReleaseMsgGroupByMsgContent =
releaseMessageServiceWithCache releaseMessageServiceWithCache
.findLatestReleaseMessagesGroupByMessages(Sets.newHashSet(someMsgContent, anotherMsgContent)); .findLatestReleaseMessagesGroupByMessages(Sets.newLinkedHashSet(
Arrays.asList(someMsgContent, anotherMsgContent))
);
assertEquals(2, latestReleaseMsgGroupByMsgContent.size()); assertEquals(2, latestReleaseMsgGroupByMsgContent.size());
assertEquals(1, latestReleaseMsgGroupByMsgContent.get(1).getId()); assertEquals(3, latestReleaseMsgGroupByMsgContent.get(1).getId());
assertEquals(someMsgContent, latestReleaseMsgGroupByMsgContent.get(1).getMessage()); assertEquals(anotherMsgContent, latestReleaseMsgGroupByMsgContent.get(1).getMessage());
assertEquals(3, latestReleaseMsgGroupByMsgContent.get(0).getId()); assertEquals(1, latestReleaseMsgGroupByMsgContent.get(0).getId());
assertEquals(anotherMsgContent, latestReleaseMsgGroupByMsgContent.get(0).getMessage()); assertEquals(someMsgContent, latestReleaseMsgGroupByMsgContent.get(0).getMessage());
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment