Fix bug with workflow shadower: ALL is documented as an allowed Status; test and fix.

uber-go / cadence-client

Framework for authoring workflows and activities running on top of the Cadence orchestration engine.

https://cadenceworkflow.io

MIT License

344 stars 130 forks source link

Fix bug with workflow shadower: ALL is documented as an allowed Status; test and fix. #1187

Closed ZackLK closed 1 year ago

ZackLK commented 1 year ago

Found this while trying to use the workflow shadower. ALL is documented as an allowed status and looks like it should work with the rest of the code, its just not being allowed through the initial filter.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 01831ef1-edd6-4d31-af1f-8a91b08d8b15

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.164%

Totals
Change from base Build 0182ff72-046c-458e-b5f7-ff5dc431e729:	0.0%
Covered Lines:	12598
Relevant Lines:	19634

uber-go / cadence-client

Fix bug with workflow shadower: ALL is documented as an allowed Status; test and fix. #1187

Pull Request Test Coverage Report for Build 01831ef1-edd6-4d31-af1f-8a91b08d8b15

💛 - Coveralls