Skip to content
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

adds method to match regex for a given json path #228

Merged
merged 6 commits into from
Aug 7, 2024

Conversation

rishiranjjan
Copy link
Member

@rishiranjjan rishiranjjan commented Aug 5, 2024

Similar to bodyContainsJsonPath method a new method bodyContainsJsonPathRegex is added to match a given regex string to json path.

This could be useful when matching something like dates, numeric ids, etc. on a response body. Currently we do have support to create a regex based assertion on the DD UI (screenshot)

Screenshot 2024-08-05 at 21 00 55

Similar to `bodyContainsJsonPath` method a new method `bodyContainsJsonPathRegex` is added to match a given regex string to json path.

This could be useful when matching something like dates, etc on a response body
@rishiranjjan rishiranjjan self-assigned this Aug 5, 2024
@azzmiks
Copy link
Collaborator

azzmiks commented Aug 6, 2024

Please also add your change to the Changelog file

@rishiranjjan rishiranjjan marked this pull request as ready for review August 6, 2024 12:14
@rishiranjjan rishiranjjan requested a review from a team as a code owner August 6, 2024 12:14
azzmiks
azzmiks previously approved these changes Aug 6, 2024
matt-long-92
matt-long-92 previously approved these changes Aug 7, 2024
Co-authored-by: Matt Long <1992.mdl@gmail.com>
@rishiranjjan rishiranjjan dismissed stale reviews from matt-long-92 and azzmiks via 3442359 August 7, 2024 09:56
@azzmiks azzmiks merged commit a8ad89f into master Aug 7, 2024
3 checks passed
@azzmiks azzmiks deleted the jsonpath-regex-match branch August 7, 2024 11:45
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.

3 participants