Skip to content

Commit 7b80473

Browse files
committed
Disallow launch-scheduled with wait which doesn't make sense and isn't tested
Former-commit-id: 84e6f8c
1 parent ab75703 commit 7b80473

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

seqware-pipeline/src/main/java/net/sourceforge/seqware/pipeline/plugin/WorkflowPlugin.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,12 +143,17 @@ public ReturnValue init() {
143143
}
144144

145145
if (options.has("workflow-engine")){
146-
if (!ENGINES.contains(options.valueOf("workflow-engine"))){
146+
if (!ENGINES.contains((String)options.valueOf("workflow-engine"))){
147147
Log.error("Invalid workflow-engine value. Must be one of: "+ENGINES_LIST);
148148
ret.setExitStatus(ReturnValue.INVALIDARGUMENT);
149149
return ret;
150150
}
151151
}
152+
if (options.has(WAIT) && options.has(LAUNCH_SCHEDULED)){
153+
Log.error("Cannot launch scheduled workflows with wait");
154+
ret.setExitStatus(ReturnValue.INVALIDARGUMENT);
155+
return ret;
156+
}
152157

153158
// wrong assumption here I think, --host is possible even when not
154159
// scheduling or waiting

0 commit comments

Comments
 (0)