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

Validation missing out on refname in foreach>actions #212

Closed
brlala opened this issue Sep 7, 2022 · 0 comments · Fixed by #360
Closed

Validation missing out on refname in foreach>actions #212

brlala opened this issue Sep 7, 2022 · 0 comments · Fixed by #360
Assignees
Labels
bug Something isn't working spec:0.8

Comments

@brlala
Copy link

brlala commented Sep 7, 2022

What happened:
Validation passed
What you expected to happen:
Validation Fail
How to reproduce it:
use this yaml for validation

id: Test
version: "1.0"
specVersion: "0.8"
name: Test
description: Test
functions:
  - name: Test
states:
  - name: TestingForEach
    type: foreach
    inputCollection: "${ .archives }"
    iterationParam: archive
    outputCollection: "${ .output}"
    actions:
      - functionRef:
          refName: DoesNotExist
    end:
      terminate: true

Anything else we need to know?:
for type operation, the functionRef will be checked against the list of functions provided at the top of the file however the validation is missing if the type is foreach

Environment:

  • Specification version used: v0.8
@ricardozanini ricardozanini added bug Something isn't working spec:0.8 labels May 27, 2024
@ricardozanini ricardozanini self-assigned this May 27, 2024
ricardozanini added a commit to ricardozanini/sdk-java that referenced this issue May 27, 2024
ricardozanini added a commit that referenced this issue May 28, 2024
Fix #212 - Add validation for actions in ForEachState
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working spec:0.8
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants