-
Notifications
You must be signed in to change notification settings - Fork 44
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
test: more enhancements #1190
test: more enhancements #1190
Conversation
@@ -51,7 +51,7 @@ polymorphism: | |||
`; | |||
|
|||
exports[`rdme openapi inspect > feature reports > should generate a report for '@readme/oas-examples/3.0/json/readme-…' (w/ [ 'readme' ]) 1`] = ` | |||
" | |||
[SoftError: |
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.
SoftError?
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.
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.
he got me. that f***ing kanad gupta boomed me.
🧰 Changes
continuing my work in #1188 with even more work to migrate over our tests to the more comprehensive
runCommand
helper:openapi convert
openapi inspect
openapi reduce
login
logout
whoami
the only outstanding tests using the now deprecated
runCommandAndReturnResult
helper are the changelog tests, but i'll swap those out if/when we get around to migrating that command over to APIv2.🧬 QA & Testing
no functional changes. do tests still pass?