diff --git a/common/util/src/main/java/org/thingsboard/common/util/LinkedHashMapRemoveEldest.java b/common/util/src/main/java/org/thingsboard/common/util/LinkedHashMapRemoveEldest.java index c1e28ca4c9..42a4d26d87 100644 --- a/common/util/src/main/java/org/thingsboard/common/util/LinkedHashMapRemoveEldest.java +++ b/common/util/src/main/java/org/thingsboard/common/util/LinkedHashMapRemoveEldest.java @@ -50,11 +50,11 @@ import java.util.function.BiConsumer; @EqualsAndHashCode(callSuper = true) public class LinkedHashMapRemoveEldest extends LinkedHashMap { final long maxEntries; - final BiConsumer removeConsumer; + final BiConsumer removalConsumer; - public LinkedHashMapRemoveEldest(long maxEntries, BiConsumer removeConsumer) { + public LinkedHashMapRemoveEldest(long maxEntries, BiConsumer removalConsumer) { this.maxEntries = maxEntries; - this.removeConsumer = removeConsumer; + this.removalConsumer = removalConsumer; } @Override @@ -62,7 +62,7 @@ public class LinkedHashMapRemoveEldest extends LinkedHashMap { if (size() <= maxEntries) { return false; } - removeConsumer.accept(eldest.getKey(), eldest.getValue()); + removalConsumer.accept(eldest.getKey(), eldest.getValue()); return true; } } diff --git a/common/util/src/test/java/org/thingsboard/common/util/LinkedHashMapRemoveEldestTest.java b/common/util/src/test/java/org/thingsboard/common/util/LinkedHashMapRemoveEldestTest.java index 62d7f788af..3107aa8b07 100644 --- a/common/util/src/test/java/org/thingsboard/common/util/LinkedHashMapRemoveEldestTest.java +++ b/common/util/src/test/java/org/thingsboard/common/util/LinkedHashMapRemoveEldestTest.java @@ -37,6 +37,7 @@ import java.util.LinkedHashMap; import static org.hamcrest.CoreMatchers.instanceOf; import static org.hamcrest.CoreMatchers.is; +import static org.hamcrest.CoreMatchers.notNullValue; import static org.hamcrest.MatcherAssert.assertThat; public class LinkedHashMapRemoveEldestTest { @@ -44,7 +45,7 @@ public class LinkedHashMapRemoveEldestTest { public static final long MAX_ENTRIES = 10L; long removeCount = 0; - void removeConsumer(Long id, String name) { + void removalConsumer(Long id, String name) { removeCount++; assertThat(id, is(Matchers.lessThan(MAX_ENTRIES))); assertThat(name, is(id.toString())); @@ -54,9 +55,10 @@ public class LinkedHashMapRemoveEldestTest { public void givenMap_whenOverSized_thenVerifyRemovedEldest() { //given LinkedHashMapRemoveEldest map = - new LinkedHashMapRemoveEldest<>(MAX_ENTRIES, this::removeConsumer); + new LinkedHashMapRemoveEldest<>(MAX_ENTRIES, this::removalConsumer); assertThat(map.getMaxEntries(), is(MAX_ENTRIES)); + assertThat(map.getRemovalConsumer(), notNullValue()); assertThat(map, instanceOf(LinkedHashMap.class)); assertThat(map, instanceOf(LinkedHashMapRemoveEldest.class)); assertThat(map.size(), is(0));