-
Notifications
You must be signed in to change notification settings - Fork 460
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
Add support for ScheduledQueryRun #637
Conversation
) | ||
|
||
func TestSigmaScheduledQueryRunGet(t *testing.T) { | ||
t.Skip("skipping test as stripe-mock does not support this resource") |
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.
It would be really cool if we could add some way to stub requests like we do in other libraries!
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.
Yes, that would be cool! Mocking isn't really possible in Go, but I think we can accomplish something similar with a helper to boot up a little JSON test server that responds with a blob you can hand it.
In this case though, I think we should make sure that stripe-mock supports scheduled query runs. I'm just winding that through the backend now.
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.
Okay, can you try bumping stripe-mock to 0.24.0 and seeing if you can activate these tests? Thanks!
ptal @remi-stripe
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.
oh neat, thanks for shipping this one!
6def002
to
d8ac125
Compare
@brandur-stripe fixed, PTAL! |
LGTM. Thanks! |
Released as 37.0.0. |
r? @brandur-stripe
cc @stripe/api-libraries