Merge pull request #657 from NitorCreations/651-bug-when-requesting-workflow-instance-from-archive-with-includes-contains-actions

651 bug when requesting workflow instance from archive with includes contains actions

Author: eputtone

nflow-engine/src/main/java/io/nflow/engine/internal/dao/WorkflowInstanceDao.java

@@ -809,7 +809,7 @@ private void fillActions(WorkflowInstance instance, boolean includeStateVariable
     long maxActions = getMaxActions(requestedMaxActions);
     String tableName = ACTION.tableFor(instance);
     String sql = sqlVariants
-        .limit("select nflow_workflow_action.* from " + tableName + " where workflow_id = ? order by id desc", maxActions);
+        .limit("select * from " + tableName + " where workflow_id = ? order by id desc", maxActions);
     List<WorkflowInstanceAction.Builder> actionBuilders = jdbc.query(sql, workflowInstanceActionRowMapper,
         instance.id);
     if (includeStateVariables) {

nflow-engine/src/test/java/io/nflow/engine/internal/dao/MaintenanceDaoTest.java

@@ -29,6 +29,7 @@
 
 import io.nflow.engine.model.ModelObject;
 import io.nflow.engine.service.WorkflowInstanceInclude;
+import io.nflow.engine.workflow.instance.QueryWorkflowInstances;
 import io.nflow.engine.workflow.instance.WorkflowInstance;
 import io.nflow.engine.workflow.instance.WorkflowInstanceAction;
 
@@ -177,6 +178,9 @@ public void archiveWorkflowsWithActionsWorks() {
 
     assertEquals(actionIds.size(), getArchiveActionCount());
     assertEquals(actionCountAfter, actionCountBefore - actionIds.size());
+
+    assertWorkflowActionsThroughQuery(archivable1, 1);
+    assertWorkflowActionsThroughQuery(archivable2, 3);
   }
 
   @Test
@@ -476,6 +480,15 @@ private void assertArchivedWorkflowsDoNotExist(List<Long> workflowIds) {
     }
   }
 
+  private void assertWorkflowActionsThroughQuery(Long workflowId, int expectedActions) {
+    List<WorkflowInstance> archivedWorkflow = workflowInstanceDao.queryWorkflowInstances(new QueryWorkflowInstances.Builder()
+        .setQueryArchive(true)
+        .setIncludeActions(true)
+        .addIds(workflowId)
+        .build());
+    assertEquals(archivedWorkflow.get(0).actions.size(), expectedActions);
+  }
+
   private void assertActiveActionsRemoved(List<Long> actionIds) {
     for (long actionId : actionIds) {
       int found = rowCount("select 1 from nflow_workflow_action where id = ?", actionId);

pom.xml

@@ -211,7 +211,6 @@
           <artifactId>maven-surefire-plugin</artifactId>
           <version>${maven-surefire.version}</version>
           <configuration>
-            <argLine>@{jacocoArgLine}</argLine>
             <includes>
               <include>**/*Test.java</include>
             </includes>
@@ -222,7 +221,7 @@
             <threadCount>1</threadCount>
             <forkCount>${surefire.forkcount}</forkCount>
             <!-- workaround for buggy cxf implementation on top of the "new" java async http client -->
-            <argLine>-Dorg.apache.cxf.transport.http.forceURLConnection=true</argLine>
+            <argLine>@{jacocoArgLine} -Dorg.apache.cxf.transport.http.forceURLConnection=true</argLine>
           </configuration>
         </plugin>
         <plugin>