uber/NullAway

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

Open

#1.425 geöffnet am 1. Jan. 2026

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (3.462 Stars) (288 Forks)batch import
cleanupgood first issue

Beschreibung

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
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaError ProneTesting with CompilationTestHelper
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
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.