broadinstitute / wdltool

BSD 3-Clause "New" or "Revised" License
18 stars 4 forks source link

Validation misses case where nonexistent output is assigned to Array #32

Closed MatthewMah closed 6 years ago

MatthewMah commented 7 years ago

The following example passes validation, and I think it should not. I think validation should be able to identify that a nonexistent output field is trying to be read.

workflow ShouldNotValidate{
    call A{}
    Array[File] simple = [A.nonexistent]
    call B{ input:
        in = simple
    }
}

task A{
    command{
        echo "A" > out
    }
    output{
        File out = "out"
    }
}

task B{
    Array[File] in

    command{
        cat ${sep=' ' in}
    }
    output{
    }
}
mcovarr commented 6 years ago

Issue moved to broadinstitute/cromwell #2881 via ZenHub