Merge pull request #326 from benja-M-1/benja-M-1-patch-1

theburningmonk · web-flow · commit a07672daa643 · 2020-04-15T14:58:55.000+02:00
Trigger error when 'definition' is missing
diff --git a/lib/deploy/stepFunctions/compileIamRole.js b/lib/deploy/stepFunctions/compileIamRole.js
@@ -381,6 +381,10 @@ module.exports = {
         return;
       }
 
+      if (!stateMachineObj.definition) {
+        throw new Error(`Missing "definition" for state machine ${stateMachineName}`);
+      }
+
       const taskStates = getTaskStates(stateMachineObj.definition.States);
       let iamPermissions = getIamPermissions.bind(this)(taskStates);
 

Original file line number	Diff line number	Diff line change
`@@ -381,6 +381,10 @@ module.exports = {`
`381`	`381`	`return;`
`382`	`382`	`}`
`383`	`383`
	`384`	`+ if (!stateMachineObj.definition) {`
	`385`	+ throw new Error(`Missing "definition" for state machine ${stateMachineName}`);
	`386`	`+ }`
	`387`	`+`
`384`	`388`	`const taskStates = getTaskStates(stateMachineObj.definition.States);`
`385`	`389`	`let iamPermissions = getIamPermissions.bind(this)(taskStates);`
`386`	`390`