add test

NitorCreations · Dec 2, 2014 · 3677865 · 3677865
1 parent 84e4f5a
commit 3677865
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.../java/com/nitorcreations/nflow/engine/workflow/definition/AbstractWorkflowDefinition.java b/.../java/com/nitorcreations/nflow/engine/workflow/definition/AbstractWorkflowDefinition.java
@@ -43,7 +43,7 @@ protected AbstractWorkflowDefinition(String type, S initialState, S errorState)
 
   protected AbstractWorkflowDefinition(String type, S initialState, S errorState, WorkflowSettings settings) {
     Assert.notNull(initialState, "initialState must not be null");
-    Assert.isTrue(initialState.getType() == WorkflowStateType.start, "initial state must be a start state");
+    Assert.isTrue(initialState.getType() == WorkflowStateType.start, "initialState must be a start state");
     Assert.notNull(errorState, "errorState must not be null");
     Assert.isTrue(isStateFinal(errorState), "errorState must be a final state");
     this.type = type;

diff --git a/...test/java/com/nitorcreations/nflow/engine/workflow/definition/WorkflowDefinitionTest.java b/...test/java/com/nitorcreations/nflow/engine/workflow/definition/WorkflowDefinitionTest.java
@@ -36,6 +36,14 @@ public void initialStateIsRequired() {
     };
   }
 
+  @Test
+  public void initialStateMustBeStartState() {
+    thrown.expect(IllegalArgumentException.class);
+    thrown.expectMessage("initialState must be a start state");
+    new WorkflowDefinition<TestDefinition.TestState>("nonStartInitialState", TestState.done, TestState.error) {
+    };
+  }
+
   @Test
   public void errorStateIsRequired() {
     thrown.expect(IllegalArgumentException.class);
@@ -45,7 +53,7 @@ public void errorStateIsRequired() {
   }
 
   @Test
-  public void errorStateMustBeFinal() {
+  public void errorStateMustBeFinalState() {
     thrown.expect(IllegalArgumentException.class);
     thrown.expectMessage("errorState must be a final state");
     new WorkflowDefinition<TestDefinition.TestState>("nonFinalErrorState", TestState.start1, TestState.start1) {