Skip to content
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

MG-888 - Update channels SDK tests #2269

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

felixgateru
Copy link
Contributor

@felixgateru felixgateru commented Jun 4, 2024

What type of PR is this?

This is a feature as it updates the channels SDK tests.

What does this do?

This pr updates teh current tests in the sdk to conform to existing changes as well as increasing the coverage of the channels sdk tests.

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

Yes, the tests are the changes.

Did you document any new/modified feature?

No,

Notes

arvindh123
arvindh123 previously approved these changes Jun 10, 2024
pkg/sdk/go/channels.go Outdated Show resolved Hide resolved
pkg/sdk/go/channels_test.go Show resolved Hide resolved
rodneyosodo
rodneyosodo previously approved these changes Jun 12, 2024
arvindh123
arvindh123 previously approved these changes Jun 26, 2024
rodneyosodo
rodneyosodo previously approved these changes Jun 26, 2024
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚀 Ready for merge
Development

Successfully merging this pull request may close these issues.

None yet

3 participants