Skip to content

Fix example code; add support for jsonpath vs json check if "$" is present in the JSON response example #19

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

markusthegeek
Copy link

No description provided.

…onpath" vs "json" and can be used when not all keys can be compared via "json" mode e.g. eTags, last modified date, dynamic URLs
…body example(s) code that would always use the fake Latin values
@markusthegeek
Copy link
Author

markusthegeek commented Feb 14, 2025

Hey @mishushakov, this fixes some logical errors in the previous example fix or param and request body examples. The current code always produces Latin fake values...

Also an idea to switch to jsonpath va json checker if author adds "$" in the JSON response example. Otherwise I do not know how people deal with changing etags, dates, and URLs when comparing responses. Using captures did not work within the same call and would be even more complicated to derive from the Open API schema.

Will this automatically be picked up by npm install -g stepci once merged and published?

Gruesse von LA nach Berlin, war dort 9 Jahre vor meiner Zeit in LA 😉

@markusthegeek
Copy link
Author

markusthegeek commented Feb 14, 2025

looks like since 0.3.9 nothing has been published which explains why the main stepci tool was still generating fine.

image

@markusthegeek markusthegeek changed the title Fix example code; add support for jsonpath vs json check if "$." is present in the JSON response example Fix example code; add support for jsonpath vs json check if "$" is present in the JSON response example Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant