#481 reproduced and fixed

yegor256 · Dec 1, 2017 · 2e1701d · 2e1701d
1 parent ed0b73f
commit 2e1701d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/src/main/java/org/cactoos/func/StickyBiFunc.java b/src/main/java/org/cactoos/func/StickyBiFunc.java
@@ -23,7 +23,7 @@
  */
 package org.cactoos.func;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import org.cactoos.BiFunc;
 import org.cactoos.map.MapEntry;
@@ -81,7 +81,7 @@ public StickyBiFunc(final BiFunc<X, Y, Z> fnc) {
      */
     public StickyBiFunc(final BiFunc<X, Y, Z> fnc, final int max) {
         this.func = fnc;
-        this.cache = new HashMap<>(0);
+        this.cache = new LinkedHashMap<>(0);
         this.size = max;
     }
 

diff --git a/src/test/java/org/cactoos/func/StickyFuncTest.java b/src/test/java/org/cactoos/func/StickyFuncTest.java
@@ -52,12 +52,19 @@ public void cachesFuncResults() throws Exception {
 
     @Test
     public void cachesWithLimitedBuffer() throws Exception {
-        final Func<Boolean, Integer> func = new StickyFunc<>(
-            input -> new SecureRandom().nextInt(), 1
+        final Func<Integer, Integer> func = new StickyFunc<>(
+            input -> new SecureRandom().nextInt(), 2
         );
+        final int first = func.apply(0);
+        final int second = func.apply(1);
         MatcherAssert.assertThat(
-            func.apply(true) + func.apply(true),
-            Matchers.equalTo(func.apply(true) + func.apply(true))
+            first + second,
+            Matchers.equalTo(func.apply(0) + func.apply(1))
+        );
+        final int third = func.apply(-1);
+        MatcherAssert.assertThat(
+            second + third,
+            Matchers.equalTo(func.apply(1) + func.apply(-1))
         );
     }