common-workflow-language / common-workflow-language/cwltool

`format` checking should be part of `--validate`

Abierto
#1,290 0 comentarios 0 reacciones 0 asignados Ver en GitHub
bug gsoc help wanted
Lenguaje dominante
Python
Estrellas
376
Forks
255
Merge medio
2 d 7 h
PR fusionados (30 d)
12

Descripción

In the example below, `cwltool --validate` should fail, as step `first`'s `that` output has a static format that does not match step `second`'s `other` input.

If `steps.first.run.outputs.that.format` was changed from `iana:text/plain` to `$(inputs.this.format)` then `--validate` should also fail as neither `iana:text/plain` nor `iana:text/rtf` match the required format `iana:text/xml`

- [ ] format checking of static formats
- [ ] format checking of pass through CWL expression formats

``` cwl
#!/usr/bin/env cwl-runner
cwlVersion: v1.0
class: Workflow

inputs: []

steps:
first:
run:
class: CommandLineTool
inputs:
this:
type: File
format: [iana:text/plain, iana:text/rtf]
baseCommand: whoami
stdout: stdout.txt
outputs:
that:
type: File
format: iana:text/plain
outputBinding:
glob: stdout.txt
in:
this:
default:
class: File
contents: Foo
format: iana:text/plain

out: [ that ]
second:
run:
class: CommandLineTool
inputs:
other:
type: File
format: iana:text/xml
baseCommand: echo
arguments: [ $(inputs.other.format) ]
outputs:
result: stdout
in:
other: first/that
out: [ result ]

outputs:
result:
type: File
outputSource: second/result

$namespaces: { iana: https://www.iana.org/assignments/media-types/ }

```

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.