cleaned up the test a bit, added changelog

Adobe-Consulting-Services · Nov 9, 2018 · 944741e · 944741e
1 parent 28b2f8f
commit 944741e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com)
 
 ## [Unreleased]
 
+### Fixed
+- #1551 - ThrottledTaskRunner avoid overflow errors when comparing priority with large absolute (negative or positive) values
+
 ## [3.19.0] - 2018-11-03
 
 ### Added

diff --git a/bundle/src/test/java/com/adobe/acs/commons/fam/impl/ThrottledTaskRunnerTest.java b/bundle/src/test/java/com/adobe/acs/commons/fam/impl/ThrottledTaskRunnerTest.java
@@ -57,7 +57,7 @@ public void testExecutionOrderOverflow() throws NotCompliantMBeanException, Inte
                 } catch (InterruptedException e) {
                     log.error("", e);
                 }
-                log.info("normal priority: {}",  finalI);
+                log.info("very low priority: {}", finalI);
                 executions.add(1L);
             }, Integer.MIN_VALUE);
         }
@@ -66,9 +66,9 @@ public void testExecutionOrderOverflow() throws NotCompliantMBeanException, Inte
         for(int i=0;i<10;i++) {
             int finalI = i;
             ttr.scheduleWork(() -> {
-                log.info("high priority: {}", finalI);
+                log.info("very high priority: {}", finalI);
                 executions.add(5L);
-            }, 10);
+            }, Integer.MAX_VALUE);
         }
 
         while(ttr.getActiveCount() > 0) {