Skip to content

[BUG][7.0.0] Error when parsing listen task (one,all,any) #412

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

Closed
uasouz opened this issue Aug 22, 2024 · 2 comments · Fixed by #414
Closed

[BUG][7.0.0] Error when parsing listen task (one,all,any) #412

uasouz opened this issue Aug 22, 2024 · 2 comments · Fixed by #414
Assignees
Labels
bug Something isn't working spec:1.0.0

Comments

@uasouz
Copy link
Contributor

uasouz commented Aug 22, 2024

What happened:

During a debug session on a runtime implementation in Java one of our new tests failed due to error when reading and parsing the workflow YAML file.
After some time we found suppresed errors showing that the expected fields inside the to block of listen was expecting diferent keys than the ones specified on the spec.

Captura de tela de 2024-08-21 11-09-17

I also replicated the failure with tests on the SDK project by adding a new test.

Captura de tela de 2024-08-21 11-22-02

What you expected to happen:

For the YAML to be parsed correctly and expect the format complying with the SW specification.

How to reproduce it:
Add a listen task to any workflow using with.to.any or with.to.one or with.to.all

Anything else we need to know?:

Environment:

  • Specification version used: 1.0.0-alpha2
@fjtirado
Copy link
Collaborator

@uasouz For further issues, it would be far easier to debug, rather than attaching images, a string with the content of the workflow being tested and the stack trace.

fjtirado added a commit to fjtirado/sdk-java that referenced this issue Aug 26, 2024
@uasouz
Copy link
Contributor Author

uasouz commented Aug 26, 2024

@uasouz For further issues, it would be far easier to debug, rather than attaching images, a string with the content of the workflow being tested and the stack trace.

Thanks for the feedback, next time I will add more text and the stack trace.

ricardozanini added a commit that referenced this issue Aug 26, 2024
Fix #412 Adding OneOf support to EventConsumptionStrategy
fjtirado added a commit to fjtirado/sdk-java that referenced this issue Oct 28, 2024
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:1.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants