uber/NullAway

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

Open

#1 425 ouverte le 1 janv. 2026

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)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.

Guide contributeur

Stack technique
java
Domaine
testingtooling
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaError ProneTesting with CompilationTestHelper
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
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.