Merge pull request #151 from tsurdilo/actioncondition

adding condition to actions

Author: tsurdilo

api/src/main/resources/schema/actions/action.json

@@ -48,6 +48,11 @@
     },
     "actionDataFilter": {
       "$ref": "../filters/actiondatafilter.json"
+    },
+    "condition": {
+      "description": "Expression, if defined, must evaluate to true for this action to be performed. If false, action is disregarded",
+      "type": "string",
+      "minLength": 1
     }
   },
   "oneOf": [

api/src/test/java/io/serverlessworkflow/api/test/MarkupToWorkflowTest.java

@@ -747,6 +747,8 @@ public void testErrorsParams(String workflowLocation) {
     assertEquals("testRetry", actions.get(0).getRetryRef());
     assertNotNull(actions.get(0).getNonRetryableErrors());
     assertEquals(2, actions.get(0).getNonRetryableErrors().size());
+    assertNotNull(actions.get(0).getCondition());
+    assertEquals("${ .data }", actions.get(0).getCondition());
 
     assertNotNull(operationState.getOnErrors());
     assertEquals(1, operationState.getOnErrors().size());

api/src/test/resources/features/errors.json

@@ -23,7 +23,8 @@
         {
           "functionRef": "addPet",
           "retryRef": "testRetry",
-          "nonRetryableErrors": ["A", "B"]
+          "nonRetryableErrors": ["A", "B"],
+          "condition": "${ .data }"
         }
       ],
       "onErrors": [

api/src/test/resources/features/errors.yml

@@ -18,6 +18,7 @@ states:
         nonRetryableErrors:
           - A
           - B
+        condition: "${ .data }"
     onErrors:
       - errorRefs:
           - A