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
Update Group Info. #536
Update Group Info. #536
Conversation
time.sleep(1) | ||
self.fail(f"group {self.namespace}/{name} still exists") | ||
description = "this is a test description" | ||
logo = "testLogo" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logo is expected to be a base64 encoded image. If you are looking for a test to validate it end to end perhaps we should hardcode an image value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be better, although a string value would pass just the same
self.assertCountEqual(group_info.tags, tags) | ||
|
||
# Cleanup | ||
groups.deregister(group_uri) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should delete not just deregister so we cleanup storage too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have just copied the "delete"/deregister from the existing test.
It has a TODO about that same issue.
Do we have a real delete method @Shelnutt2?
SDK to allow Group Info updating, using the v2 of the API.