ben-manes / caffeine / #5569

100%

master: 100%

push

github

Add regression tests for unpinned cache behaviors

Cover behaviors that had no test, so a future regression is caught.
All are test-only changes plus one design note; no production code
changes.

- Same-instance compute resets the fixed expireAfterWrite clock
  (ExpireAfterWriteTest) and re-weighs an in-place-mutated value
  (EvictionTest, COLLECTION weigher).
- putIfAbsent's lock-free expireAfterRead throw path
  (ExpireAfterVarTest).
- Restore the hasSize pin on the compute/merge _differentValue
  tests, the REJECTING removal-listener guard on compute_error and
  compute_throwsCheckedException, and @CheckNoStats on
  compute_nullKey (AsMapTest).
- Issue30Test's method-local cache now asserts isValid(); the
  validation interceptor only reaches parameterized/argument caches.
- jcache expired-remove counts an eviction not a removal, and an
  entry-processor load-then-remove writes through as a delete
  (JCacheCombinedExpiryTest, CacheWriterTest).
- Document and pin the async put re-registration double-completion
  (AsyncCacheTest + design-decisions).

4047 of 4056 branches covered (99.78%)

8260 of 8261 relevant lines covered (99.99%)

1.0 hits per line