Closed fmigneault closed 1 month ago
Attention: Patch coverage is 79.89418%
with 76 lines
in your changes are missing coverage. Please review.
Project coverage is 85.76%. Comparing base (
aae0904
) to head (ee56c8d
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Changes:
CWL
MultipleInputFeatureRequirement
support.CWL
SubworkflowFeatureRequirement
support.CWL
Workflow
explicit schema validation of itssteps
.CWL
requirements
. Only fully defined and resolved definitions will be allowed. If an unsupportedCWL
requirement byWeaver
must be provided (but is a valid definition supported bycwltool
), it must now be provided throughhints
to succeed schema validation.CWL
output definition usingloadContents
to anoutputBinding.glob
reference to load theFile
contents into astring
output.CWL
JavaScript expressions within intermediate steps of aWorkflow
to collect output results from relevant sources with better data manipulation flexibility.weaver.processes.wps_process_base.WpsProcessInterface
to allow better reuse of the common operations shared by derivedCWL
Workflow
steps implemented byESGFProcess
,Wps1Process
,Wps3Process
andOGCAPIRemoteProcessBase
.ESGFProcess
to use the common operations ofCWL
Workflow
steps defined byWpsProcessInterface
.Fixes:
CWL
schema definition forScatterFeatureRequirement
that directly contained the corresponding fieldsscatter
andscatterMethod
, instead of the expected definition within aWorkflow Step
.CWL
requirements
schema definition usingOneOf
and thediscriminator
property that could sometime drop a definition when it only contained an empty mapping{}
, and that the corresponding requirement allows it.weaver.wps_restapi.colander_extras.AnyOfKeywordSchema
not allowing distinctJSON
structuretype
to be combined simultaneously.CWL
Workflow
not retrieving output results when returned directly as literal data from a remoteProcess
.CWL
Workflow
potentially failing tool resolution for a local stepProcess
ifhints
where omitted.CWL
Workflow
resolution of steprequirements
from one of theWeaver
application types (i.e.:builtin
,docker
,ESGF-CWT
,OGCAPI
,WPS1
) due tocwltool
namespace adding a prefixed URI.