ben-manes / caffeine / #5321

100%

master: 100%

push

github

fix minor expiration edge cases

1. The calculated remaining duration was incorrect in the
Policy.CacheEntry snapshot by giving a higher value. The mapping will
expire at the correct time.
2. Map.entrySet().contains(entry) now returns false if the entry is
present but has expired and is pending eviction.
3. JCache's Duration.ETERNAL returns a null TimeUnit. Previously this
caused the cache to log a failure when updating Caffeine's metadata,
though the eternal would be honored as set correctly on the adapter's
wrapped value.
4. JCache's expiry time now ensures that the sentinel flags, 0 and
Long.MAX_VALUE, are avoided in wrap around calculations due to
System.nanoTime() being randomized. In these cases the duration is
adjusted to occur up to 1ms sooner.

3838 of 3850 branches covered (99.69%)

19 of 21 new or added lines in 2 files covered. (90.48%)

7883 of 7888 relevant lines covered (99.94%)

1.0 hits per line