Merge pull request pentaho#4393 from kurtwalker/BACKLOG-18774

[BACKLOG-18774] - trans with only one step should consider that step …
mutaohah · Sep 15, 2017 · 71cf24d · 71cf24d
2 parents 33e6f61 + 7dcd817
commit 71cf24d
Show file tree

Hide file tree

Showing 4 changed files with 485 additions and 5 deletions.
diff --git a/engine/src/main/java/org/pentaho/di/trans/TransMeta.java b/engine/src/main/java/org/pentaho/di/trans/TransMeta.java
@@ -1008,6 +1008,9 @@ public List<StepMeta> getUsedSteps() {
         list.add( stepMeta );
       }
     }
+    if ( list.isEmpty() && getSteps().size() == 1 ) {
+      list = getSteps();
+    }
 
     return list;
   }

diff --git a/engine/src/test/java/org/pentaho/di/trans/TransMetaTest.java b/engine/src/test/java/org/pentaho/di/trans/TransMetaTest.java
@@ -457,4 +457,9 @@ public void testLoadXml() throws KettleException {
     Assert.assertEquals( repDirectory.getPath(), meta.getVariable( Const.INTERNAL_VARIABLE_TRANSFORMATION_REPOSITORY_DIRECTORY ) );
   }
 
+  @Test
+  public void testTransWithOneStepIsConsideredUsed() throws Exception {
+    TransMeta transMeta = new TransMeta( getClass().getResource( "one-step-trans.ktr" ).getPath() );
+    assertEquals( 1, transMeta.getUsedSteps().size() );
+  }
 }