ben-manes / caffeine / #5447

100%

master: 100%

push

github

Apply creation expiry, not access, to loader-created entries on get()

LoadingCacheProxy.getOrLoad() called setAccessExpireTime on every result,
overwriting the creation expiry that JCacheLoaderAdapter had just set on
the freshly loaded entry. Move the call into the hit-only branch so a
loader-created entry retains its creation expiry, matching the JSR-107
RI's mutually-exclusive if/else and Hazelcast's miss path. Existing
fixtures used CreatedExpiryPolicy (access=null) or AccessedExpiryPolicy
(creation=access) and could not distinguish the two; new regressions in
JCacheCreationExpiryTest pin the contract with distinct durations.

issue found by detail.dev

3988 of 3996 branches covered (99.8%)

1 of 1 new or added line in 1 file covered. (100.0%)

8191 of 8199 relevant lines covered (99.9%)

1.0 hits per line