-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add repeat tag #13
Add repeat tag #13
Conversation
basicAuth = apiPort.resolveObject(basicAuth) | ||
delete basicAuth.$file | ||
} | ||
requests.forEach((_req) => { |
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.
because of this new loop, most of the change fro here are indentation only, I would advice to review with Hide whitespace changes
enabled. Here is a shortcut to it :) https://github.com/testbetter/openapitest/pull/13/files?w=1
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 looks great. There is only one small change needed I think, but it would be good to provide feedback on some of my questions/comments for my understanding.
Co-Authored-By: Dave Reich <dreich@nucleushealth.io>
The intention here is to add the capability to repeat a test
N
times if the user wants to. For now, we are implementing a straightforward approach.I have added some tests to demo how the feature works for now as well as update the documentation.
Bonus: I added tests to skip and only tags.
Ps: This review will be easier if you do it with
Hide whitespace changes
enabled, try it here https://github.com/testbetter/openapitest/pull/13/files?w=1