Skip to content

Comments

Handle single-element list for CWL Workflow outputs#884

Draft
LanderOtto wants to merge 3 commits intomasterfrom
fix/single-processors
Draft

Handle single-element list for CWL Workflow outputs#884
LanderOtto wants to merge 3 commits intomasterfrom
fix/single-processors

Conversation

@LanderOtto
Copy link
Collaborator

@LanderOtto LanderOtto commented Dec 6, 2025

This commit fixes a special case of the pickValue directive. As defined in https://www.commonwl.org/v1.2/Workflow.html#WorkflowStepInput, if the pickValue has a single source and the resulting list contains only one element (i.e., the source is not null), the element itself is returned, rather than a single-element list.

@codecov
Copy link

codecov bot commented Dec 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.75%. Comparing base (e4ecf3a) to head (149011e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #884      +/-   ##
==========================================
+ Coverage   73.72%   73.75%   +0.02%     
==========================================
  Files          91       91              
  Lines       12281    12287       +6     
  Branches     2156     2157       +1     
==========================================
+ Hits         9054     9062       +8     
+ Misses       2714     2713       -1     
+ Partials      513      512       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LanderOtto LanderOtto force-pushed the fix/single-processors branch 3 times, most recently from ac48c95 to 6e24f0f Compare December 8, 2025 20:18
@GlassOfWhiskey GlassOfWhiskey force-pushed the master branch 7 times, most recently from b21f362 to e7820f1 Compare January 12, 2026 08:25
@GlassOfWhiskey GlassOfWhiskey force-pushed the master branch 4 times, most recently from 8c05dac to 4a6edc9 Compare January 24, 2026 13:16
…e `TokenProcessor`. This fix aligns the StreamFlow behavior with the cwltool reference implementation. Similar fix to #672
@LanderOtto LanderOtto force-pushed the fix/single-processors branch from 6e24f0f to 149011e Compare February 7, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant