test: we should add tests for the stateManagerCtx

[sicoyle]

Is your feature request related to a problem? Please describe. https://github.com/dapr/go-sdk/blob/main/actor/state/state_manager.go is the only file in the state directory without tests. We should add tests to improve test coverage here.

Describe the solution you'd like We need a state_manager_test.go file added testing the functionality of the stateManagerCtx.

Additional context You can refer to the other test files in the state directory for inspiration and guidance to setup the tests for the state_manager_test.go.

[pieramarchesini]

/assign

[sicoyle]

adding links here from the chat :)

https://github.com/dapr/go-sdk/blob/e16e0350a52349b5a05138edc0b58e3be78ee753/service/http/health_check_test.go

https://github.com/dapr/go-sdk/blob/e16e0350a52349b5a05138edc0b58e3be78ee753/client/client_test.go#L136

Thank you for helping us improve the go sdk @pieramarchesini during the Grace Hopper Conference open source day event! 🙌

[sadath-12]

/assign

[github-actions[bot]]

⚠️ Unable to assign issue

[mikeee]

@pieramarchesini @sadath-12 Heyo, are you both still working on adding some tests?

[KentHsu]

Hi @mikeee, Is it okay if I take this one?

[mikeee]

@KentHsu - Absolutely, thank you for picking this up!

[KentHsu]

/assign

[github-actions[bot]]

⚠️ Unable to assign issue

[mikeee]

/assign

You're already assigned, it's all good 👍

[KentHsu]

/assign

You're already assigned, it's all good 👍

Didn't notice I was assigned when I typed it 😅 I created a PR for this issue. Please help to review it and let me know if I'm doing it correctly. Thank you!