Fixed stc to properly account for skipped steps when completing enclosing group.

Change-Id: I9b297f8a48a9f4423e34fbbb954bf039e98e29b8

@@ -217,8 +217,8 @@ public class Coordinator {
             if (step instanceof Group) {
             if (step instanceof Group) {
                 Group group = (Group) step;
                 Group group = (Group) step;
                 group.children().forEach(child -> delegate.onCompletion(child, SKIPPED));
                 group.children().forEach(child -> delegate.onCompletion(child, SKIPPED));
-                delegate.onCompletion(step, SKIPPED);
             }
             }
+            delegate.onCompletion(step, SKIPPED);
         }
         }
     }
     }
 
 
@@ -268,7 +268,7 @@ public class Coordinator {
             boolean failed = false;
             boolean failed = false;
             for (Step child : group.children()) {
             for (Step child : group.children()) {
                 Status status = store.getStatus(child);
                 Status status = store.getStatus(child);
-                done = done && (status == SUCCEEDED || status == FAILED);
+                done = done && (status == SUCCEEDED || status == FAILED || status == SKIPPED);
                 failed = failed || status == FAILED;
                 failed = failed || status == FAILED;
             }
             }
             if (done) {
             if (done) {