Add TCK model test for Actions

cloudstateio / cloudstate

Distributed State Management for Serverless

https://cloudstate.io

Apache License 2.0

763 stars 97 forks source link

Add TCK model test for Actions #468

Closed pvlugter closed 3 years ago

pvlugter commented 3 years ago

More on evolving the TCK: #316. Add simple TCK model tests for Actions.

Currently only implemented for java-support, which is set up for TCK model tests. This did uncover a couple of things in the java-support implementation: side effects were not being returned to the proxy, and there was no support for failure replies.

marcellanz commented 3 years ago

This did uncover a couple of things

nice! Best applied usage of a TCK :) Looking forward to use it for Go support.

pvlugter commented 3 years ago

As it's been a couple of weeks, I'll merge this now, and make it simpler for the next TCK extensions. Post-merge reviews still welcome of course.