Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix up missing/duplicate labels and missing tags (1/n) #56

Merged
merged 2 commits into from
Nov 10, 2020
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 19 additions & 12 deletions conformance_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,7 @@
output:
count_output: 16
tool: tests/count-lines19-wf.cwl
label: wf_wc_nomultiple
label: wf_wc_nomultiple_merge_nested
id: count-lines19-wf
doc: >-
Test when step source is a single-item list and linkMerge is
Expand Down Expand Up @@ -879,7 +879,7 @@
label: metadata
id: 63
doc: Test metadata
tags: [ required ]
tags: [ required, command_line_tool ]

- job: tests/formattest-job.json
output:
Expand Down Expand Up @@ -1894,7 +1894,7 @@
checksum: "sha1$7d5ca8c0c03e883c56c4eb1ef6f6bb9bccad4d86"
size: 8
tool: tests/envvar3.cwl
label: env_home_tmpdir_docker
label: env_home_tmpdir_docker_no_return_code
id: 134
doc: Test $HOME and $TMPDIR are set correctly in Docker without using return code
tags: [ shell_command, command_line_tool ]
Expand Down Expand Up @@ -2894,7 +2894,7 @@
tool: tests/networkaccess2.cwl
label: networkaccess_disabled
doc: Test networkaccess is disabled by default
tags: [ networkaccess ]
tags: [ networkaccess, command_line_tool ]
id: 224

- job: tests/stage-array-job.json
Expand Down Expand Up @@ -2986,10 +2986,13 @@
"class": "File"
"size": 12010
tool: tests/initialworkdir-glob-fullpath.cwl
doc: Test output of InitialWorkDir
tags: [ initial_work_dir ]
label: initial_workdir_output_glob
doc: Test full path glob output of InitialWorkDir
tags: [ initial_work_dir, command_line_tool ]
id: 230

# same test as above but keeping it for preserve ids maybe? I don't know -
# there is a reason I added labels before these ids were here :)
- job: tests/initialworkdirrequirement-docker-out-job.json
output:
OUTPUT:
Expand All @@ -3004,15 +3007,17 @@
"class": "File"
"size": 12010
tool: tests/initialworkdir-glob-fullpath.cwl
doc: Test if full paths are allowed in glob
tags: [ initial_work_dir ]
label: initial_workdir_output_glob_duplicate
doc: Test full path glob output of InitialWorkDir (duplicate)
tags: [ initial_work_dir, command_line_tool ]
id: 231

- job: tests/empty.json
should_fail: true
tool: tests/glob-path-error.cwl
label: glob_outside_outputs_fails
doc: Test fail trying to glob outside output directory
tags: [ required ]
tags: [ required, command_line_tool, docker ]
id: 232

- job: tests/empty.json
Expand All @@ -3025,6 +3030,7 @@
"baesname": "symlink.txt"
"checksum": "sha1$cd28ec34f3f9425aca544b6332453708e8aaa82a"
should_fail: true
tags: [ command_line_tool ]
id: 233

- job: tests/empty.json
Expand All @@ -3037,11 +3043,12 @@
"basename": "symlink.txt"
"checksum": "sha1$cd28ec34f3f9425aca544b6332453708e8aaa82a"
id: 234
tags: [ command_line_tool ]

- job: tests/empty.json
tool: tests/inp_update_wf.cwl
doc: inplace update has side effect on file content
tags: [ inplace_update ]
tags: [ inplace_update, workflow ]
output:
a: 4
b: 4
Expand All @@ -3050,7 +3057,7 @@
- job: tests/empty.json
tool: tests/inpdir_update_wf.cwl
doc: inplace update has side effect on directory content
tags: [ inplace_update ]
tags: [ inplace_update, workflow ]
output: {
a: [
{
Expand Down Expand Up @@ -3158,7 +3165,7 @@
id: 239

- doc: Test that if array of input files are staged to directory with basename and entryname, entryname overrides
label: stage_file_array
label: stage_file_array_entryname_overrides
tool: tests/stage_file_array_basename_and_entryname.cwl
job: tests/stage_file_array.job.json
output:
Expand Down