Visitar URL original
[ggj][ast][engx] fix: validate non-null elements for ConcreteReferenc… · googleapis/sdk-platform-java@ca05b80 · GitHub
Skip to content
This repository was archived by the owner on Oct 29, 2026. It is now read-only.

Commit ca05b80

Browse files
authored
[ggj][ast][engx] fix: validate non-null elements for ConcreteReference (#460)
* fix: swap assertEquals args in JavaWriterVisitorTest to match (expected, actusl) order * fix: swap assertEquals args in ImportWriterVisitorTest to match (expected, actusl) order * fix: add node validator to refactor/centralize null element checks * fix: validate non-null elements for IfStatement * fix: validate non-null elements for AnonymousClassExpr * fix: validate non-null elements for BlockStatement * fix: validate non-null elements for ClassDefinition * fix: validate non-null elements for ConcreteReference
1 parent 5f02fe6 commit ca05b80

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

‎src/main/java/com/google/api/generator/engine/ast/ConcreteReference.java‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -231,9 +231,14 @@ public Builder setGenerics(Reference... references) {
231231
// Private.
232232
abstract Class clazz();
233233

234+
abstract ImmutableList<Reference> generics();
235+
234236
abstract boolean isStaticImport();
235237

236238
public ConcreteReference build() {
239+
NodeValidator.checkNoNullElements(
240+
generics(), "generics", String.format("concrete reference %s", clazz().getSimpleName()));
241+
237242
setIsStaticImport(clazz().getEnclosingClass() != null && isStaticImport());
238243
return autoBuild();
239244
}

0 commit comments

Comments
 (0)