haisi / error-prone-support / 32053671580

100%

push

github

fix: guard against null symbol in ForbiddenApiChecker

A static import of an overloaded method (e.g. AssertJ's assertThat) has no
single resolved symbol at the import site - ASTHelpers.getSymbol(tree)
returns null there, unlike every other identifier/member-select this checker
sees - which crashed matchClassOrFieldSymbol with a NullPointerException on
symbol.getKind(). Discovered wiring this checker into java-lib-archetype,
whose own generated test scaffolding statically imports assertThat.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

97 of 97 branches covered (100.0%)

Branch coverage included in aggregate %.

256 of 256 relevant lines covered (100.0%)

4.93 hits per line