-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[usage] ListUsage filters by time range #11507
Conversation
0fca675
to
3759b20
Compare
1bc7b37
to
0fa7126
Compare
/werft run 👍 started the job as gitpod-build-mp-usage-query-range.7 |
Thanks for this! Is there a reason for choosing 31 instead of 30? Not that I'm leaning towards either. |
Hmm, the tests in both |
Just to account for months that have 31 days. It's not ideal but "works". |
42f98b2
to
0d27d25
Compare
@@ -16,87 +16,6 @@ import ( | |||
"github.com/stretchr/testify/require" | |||
) | |||
|
|||
func TestCanCreateUsageRecords(t *testing.T) { |
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.
This test was also covered later when it tests the records get updated, there was no reason to keep it.
} | ||
} | ||
|
||
func TestNoErrorOnCreatingDuplicateRecords(t *testing.T) { |
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.
Likewise, this is also tested in the test below which ensures we can update fields
Description
Implements filtering of Usage records by time
Related Issue(s)
Fixes #
How to test
Unit tests
Release Notes
Documentation
Werft options: