ben-manes / caffeine / #5596

100%

master: 100%

push

github

Match ConcurrentHashMap for containsAll on the map views

keySet()/values().containsAll inherited AbstractCollection's loop,
which threw NullPointerException on a null element and iterated even
for a self-check. CHM and Guava both skip a null element (false) and
short-circuit containsAll(self) to true; Caffeine's removeAll and
entrySet were already lenient, leaving these two views the holdout.
Override containsAll across the bounded, unbounded, and async views.
The CaffeinatedGuava facade's override (a workaround for the old NPE)
is now redundant, so drop it and delegate to the view.

4117 of 4132 branches covered (99.64%)

42 of 42 new or added lines in 3 files covered. (100.0%)

8366 of 8374 relevant lines covered (99.9%)

1.0 hits per line