[ggj][ast][engx] fix: validate non-null elements for TryCatchStatement (

#466) * 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 * fix: validate non-null elements for ForStatement * fix: validate non-null elements for GeneralForStatement * fix: validate non-null elements for MethodDefinition * fix: validate non-null elements for PackageInfoDefinition * fix: validate non-null elements for SynchronizedStatement * fix: validate non-null elements for TryCatchStatement
googleapis · Nov 7, 2020 · a1ee8f8 · a1ee8f8
1 parent df14ba8
commit a1ee8f8
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/main/java/com/google/api/generator/engine/ast/TryCatchStatement.java b/src/main/java/com/google/api/generator/engine/ast/TryCatchStatement.java
@@ -64,6 +64,9 @@ public abstract static class Builder {
 
     public TryCatchStatement build() {
       TryCatchStatement tryCatchStatement = autoBuild();
+      NodeValidator.checkNoNullElements(tryCatchStatement.tryBody(), "try body", "try-catch");
+      NodeValidator.checkNoNullElements(tryCatchStatement.catchBody(), "catch body", "try-catch");
+
       if (!tryCatchStatement.isSampleCode()) {
         Preconditions.checkNotNull(
             tryCatchStatement.catchVariableExpr(),