Avoid NullPointerException when customChange has no "class" attribute

liquibase · May 6, 2021 · 77ec354 · 77ec354
1 parent 8e92ac7
commit 77ec354
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/liquibase-core/src/main/java/liquibase/change/custom/CustomChangeWrapper.java b/liquibase-core/src/main/java/liquibase/change/custom/CustomChangeWrapper.java
@@ -285,7 +285,11 @@ public String getSerializedObjectNamespace() {
     @Override
     public void load(ParsedNode parsedNode, ResourceAccessor resourceAccessor) throws ParsedNodeException {
         try {
-            setClass(parsedNode.getChildValue(null, "class", String.class));
+            String classNameValue = parsedNode.getChildValue(null, "class", String.class);
+            if (classNameValue == null) {
+                throw new ParsedNodeException("Custom change node has no 'class' attribute!");
+            }
+            setClass(classNameValue);
         } catch (CustomChangeException e) {
             throw new ParsedNodeException(e);
         }

diff --git a/liquibase-core/src/test/groovy/liquibase/change/custom/CustomChangeWrapperTest.groovy b/liquibase-core/src/test/groovy/liquibase/change/custom/CustomChangeWrapperTest.groovy
@@ -257,6 +257,16 @@ class CustomChangeWrapperTest extends Specification {
 
     }
 
+    def "customChange without class fails expectedly"() {
+        when:
+        def node = new ParsedNode(null, "customChange")
+        def change = new CustomChangeWrapper()
+        change.load(node, resourceSupplier.simpleResourceAccessor)
+
+        then:
+        thrown(ParsedNodeException.class)
+    }
+
     def "load handles params in a 'params' collection"() {
         when:
         def node = new ParsedNode(null, "customChange")