ben-manes / caffeine / #5703 / 1

100%

master: 100%

push

github

Cover the synchronous listener failure paths

Preferring an operation's own failure over a throwing synchronous
listener's added branches that no test reached, so exercise each: an
entry processor that succeeds and one that throws over a lazily expired
prior, a putAll whose copier aborts the loop midway, a read-through get
and getAll, and a dispatch that completes exceptionally rather than
returning its failure.

Fold awaitAndSuppressFailure into suppress(), which already guards
against self-suppression and whose guard is exercised. Duplicating it
there left a branch that the operation paths cannot reach, as a
listener's exception is captured as the dispatch future's value and
never becomes the operation's own error.

4225 of 4235 branches covered (99.76%)

8527 of 8528 relevant lines covered (99.99%)

1.0 hits per line