-
Notifications
You must be signed in to change notification settings - Fork 181
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
[tests-only][full-ci] add tests for CLI command uploads sessions #10229
Conversation
3e86c52
to
71a52a4
Compare
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.
LGTM
Note: we could write "Then" steps in the form "Then the CLI response should contain only these entries". And that step would check for the required entries, and also check that there are no other entries. And then we would not need the "should not contain" steps.
But it reads nicely like it is - the scenarios are clear about what they are checking for.
71a52a4
to
ce2b695
Compare
ce2b695
to
4a01ed6
Compare
f44f155
to
ed72331
Compare
ed72331
to
c836695
Compare
Quality Gate passedIssues Measures |
[tests-only][full-ci] add tests for CLI command uploads sessions
Description
This PR adds test for CLI command related to uploads sessions.
Test scenario covered:
Related Issue
storage-users uploads sessions
command #10167Types of changes
Checklist: