-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added a JSONPath tester tool #1017
Conversation
Hum quick thinking, should we keep the XML Validators in the same group or we could create a new Validators groups? I think it's weird to have the XML Validators with the Testers. |
Thanks for the suggestion. |
Ok we can rename the XML Validator for XML Tester. No test for this tool? |
Done !
There are tests for the helper, but I admit there aren't for the gui and command line. |
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.
Ok I will approve this pull request but we really need to add Unit Test for the GUI
Thanks @btiteux , I just pushed a few more tests for UI and command line |
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: #749
What is the new behavior?
Added a new tool dedicated to testing JSONPath.
Also, moved Regex Tester and XML Validator tools to a new
Testers
group.Other information
Quality check
Before creating this PR: