Throw exception if modifiedObjects is null.

Lucie Votypkova · Lucie Votypkova · commit 2b07ff042d8c · 2019-07-24T14:35:11.000+02:00
diff --git a/job-dsl-plugin/src/main/groovy/javaposse/jobdsl/plugin/actions/GeneratedObjectsRunAction.java b/job-dsl-plugin/src/main/groovy/javaposse/jobdsl/plugin/actions/GeneratedObjectsRunAction.java
@@ -49,7 +49,7 @@ public Collection<T> getModifiedObjects() {
 
     public void addModifiedObjects(Collection<T> modifiedObjects) {
         if (this.modifiedObjects == null) {
-            this.modifiedObjects = new LinkedHashSet<>(modifiedObjects);
+            throw new IllegalStateException("Modified object should not be null. Probably wrong state after serialization/deserialization");
         } else {
             this.modifiedObjects.addAll(modifiedObjects);
         }

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public Collection<T> getModifiedObjects() {`
`49`	`49`
`50`	`50`	`public void addModifiedObjects(Collection<T> modifiedObjects) {`
`51`	`51`	`if (this.modifiedObjects == null) {`
`52`		`- this.modifiedObjects = new LinkedHashSet<>(modifiedObjects);`
	`52`	`+ throw new IllegalStateException("Modified object should not be null. Probably wrong state after serialization/deserialization");`
`53`	`53`	`} else {`
`54`	`54`	`this.modifiedObjects.addAll(modifiedObjects);`
`55`	`55`	`}`