-
Notifications
You must be signed in to change notification settings - Fork 173
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
test: we should add tests for the stateManagerCtx #446
Comments
/assign |
adding links here from the chat :) Line 136 in e16e035
Thank you for helping us improve the go sdk @pieramarchesini during the Grace Hopper Conference open source day event! 🙌 |
/assign |
|
@pieramarchesini @sadath-12 Heyo, are you both still working on adding some tests? |
Hi @mikeee, Is it okay if I take this one? |
@KentHsu - Absolutely, thank you for picking this up! |
/assign |
|
You're already assigned, it's all good 👍 |
Didn't notice I was assigned when I typed it 😅 |
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 thestateManagerCtx
.Additional context
You can refer to the other test files in the
state
directory for inspiration and guidance to setup the tests for thestate_manager_test.go
.The text was updated successfully, but these errors were encountered: