uber/NullAway

Replace calls to deprecated `com.google.errorprone.CompilationTestHelper#addSourceFile` in tests

Open

#1425 opened on Jan 1, 2026

View on GitHub
 (2 comments) (0 reactions) (0 assignees)Java (3,462 stars) (288 forks)batch import
cleanupgood first issue

Description

E.g., here:

https://github.com/uber/NullAway/blob/f1c302eea74a6535bc167af2ba79cb941858c4f2/nullaway/src/test/java/com/uber/nullaway/UnannotatedTests.java#L12-L12

We should call com.google.errorprone.CompilationTestHelper#addSourceLines instead and copy the relevant source into a text block, and then delete the separate source files. We can also remove any related @SuppressWarnings("deprecation") annotations.

Contributor guide

Tech stack
java
Domain
testingtooling
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JavaError ProneTesting with CompilationTestHelper
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
First, locate all test files in the repository that contain calls to `addSourceFile`. Use the provided example as a reference: `UnannotatedTests.java`. Replace each call with `addSourceLines`, copying the file content into a text block within the test method. Then, delete the separate source file if one exists. Remove any `@SuppressWarnings("deprecation")` annotations that were added due to the deprecated method. Search the codebase for similar patterns using grep or IDE find features.