-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(test): Add unit tests for aicore-clients #12
Conversation
… into ai-core-client-tests merge
… into ai-core-client-tests merge latest
|
||
nock(destination.url).get('/lm/artifacts').reply(200, expectedResponse, { | ||
'Content-Type': 'application/json', | ||
'AI-Resource-Group': 'default' |
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.
[q] Aren't these part of the request header?
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.
You're right. The resource group is only a part of the request header. Only content type should be a part of response. Thanks for catching it.
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 would also not bother so much with response headers for now.
Adds unit tests for the AI core clients generated by using Cloud SDK. The following 5 AI Core endpoints covered:
Closes AI/gen-ai-hub-sdk-js#47