Merge pull request #284 from fjtirado/id_really_mandatory

Force id to be mandatory

Author: ricardozanini

api/src/main/resources/schema/workflow.json

@@ -161,6 +161,7 @@
     }
   },
  "required": [
+        "id",
         "name",
         "version",
         "states"

validation/src/test/java/io/serverlessworkflow/validation/test/WorkflowValidationTest.java

@@ -58,7 +58,7 @@ public void testIncompleteYamlWithSchemaValidation() {
     List<ValidationError> validationErrors =
         workflowValidator.setSource("---\n" + "key: abc\n").validate();
     Assertions.assertNotNull(validationErrors);
-    Assertions.assertEquals(3, validationErrors.size());
+    Assertions.assertEquals(4, validationErrors.size());
   }
 
   @Test
@@ -119,11 +119,10 @@ public void testWorkflowMissingStatesIdAndKey() {
                     + "}")
             .validate();
     Assertions.assertNotNull(validationErrors);
-    Assertions.assertEquals(2, validationErrors.size());
+    Assertions.assertEquals(1, validationErrors.size());
 
     Assertions.assertEquals(
-        "Workflow id or key should not be empty", validationErrors.get(0).getMessage());
-    Assertions.assertEquals("No states found", validationErrors.get(1).getMessage());
+        "$.id: is missing but it is required", validationErrors.get(0).getMessage());
   }
 
   @Test