garydgregory commented on pull request #189:
URL:
https://github.com/apache/commons-collections/pull/189#issuecomment-739312957
@tongxin97
I do not think so.
While it is not documented, the iteration order for a Flat3Map is in fact
predictable when the size <= 3, and this is in fact checked, intentionally or
not by the test.
So this PR fixes the sanity check portion of test and looses the ordering
test for the map we actually want to test.
Therefore, the simple fix should be for the sanity check to use a
`LinkedHashMap` instead of a `HashMap`, like this:
```
public void testEntrySet() {
// Sanity check
putAndRemove(new LinkedHashMap<>());
// Actual test
putAndRemove(new Flat3Map<>());
}
```
This is presumably the intent of the test since the `putAndRemove` test
method uses exactly three entries which matches exactly the intended size of
useful Flat3Map instances.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
