uber-go / cadence-client

Framework for authoring workflows and activities running on top of the Cadence orchestration engine.
https://cadenceworkflow.io
MIT License
339 stars 128 forks source link

Update idl and add wrapper implementaton for async start/signalwithstart APIs #1321

Closed taylanisikdemir closed 3 months ago

taylanisikdemir commented 3 months ago

What changed? Updating client to expose newly added async APIs:

Steps followed to update idls

How did you test it? Added unit tests for new methods

codecov[bot] commented 3 months ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@00ff688). Click here to learn what that means. The diff coverage is 100.00%.

Additional details and impacted files | [Files](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go) | Coverage Δ | | |---|---|---| | [internal/common/auth/service\_wrapper.go](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go#diff-aW50ZXJuYWwvY29tbW9uL2F1dGgvc2VydmljZV93cmFwcGVyLmdv) | `93.54% <100.00%> (ø)` | | | [internal/common/isolationgroup/service\_wrapper.go](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go#diff-aW50ZXJuYWwvY29tbW9uL2lzb2xhdGlvbmdyb3VwL3NlcnZpY2Vfd3JhcHBlci5nbw==) | `91.47% <100.00%> (ø)` | | | [internal/common/metrics/service\_wrapper.go](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go#diff-aW50ZXJuYWwvY29tbW9uL21ldHJpY3Mvc2VydmljZV93cmFwcGVyLmdv) | `66.10% <100.00%> (ø)` | | | [internal/internal\_utils.go](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go#diff-aW50ZXJuYWwvaW50ZXJuYWxfdXRpbHMuZ28=) | `72.83% <ø> (ø)` | | | [internal/oauth\_authorization.go](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go#diff-aW50ZXJuYWwvb2F1dGhfYXV0aG9yaXphdGlvbi5nbw==) | `100.00% <ø> (ø)` | | ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go). Last update [00ff688...08cde0b](https://app.codecov.io/gh/uber-go/cadence-client/pull/1321?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber-go).