[veler]

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• New feature or enhancement
• UI change (please include screenshot!)
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Internationalization and localization
• Other (please describe):

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:

• Did you follow the code style guideline as described in CONTRIBUTING.md
• Did you build the app and test your changes?
• Did you check for accessibility? On Windows, you can use Accessibility Insights for this.
• Did you verify that the change work in Release build configuration
• Did you verify that all unit tests pass
• If necessary and if possible, did you verify your changes on:
  • Windows
  • macOS (DevToys 2.0)
  • Linux (DevToys 2.0)

[btiteux]

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.

[veler]

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. XML Validator, in the end, "tests" an XML against its XSD. How about renaming XML Validator to XML / XSD Tester or XML Tester ? What do you think @btiteux ?

[btiteux]

Ok we can rename the XML Validator for XML Tester.

No test for this tool?

[veler]

Ok we can rename the XML Validator for XML Tester.

Done !

No test for this tool?

There are tests for the helper, but I admit there aren't for the gui and command line.

[btiteux]

Ok I will approve this pull request but we really need to add Unit Test for the GUI

[veler]

Thanks @btiteux , I just pushed a few more tests for UI and command line