Request for hosting a new plugin: APICoverage

21 views
Skip to first unread message

Priti Changlani

unread,
Aug 13, 2015, 4:24:24 AM8/13/15
to Jenkins Developers
Hello there!

This is a request to host a plugin called API Coverage. The details are:

Repo name: api-coverage-plugin
Personal GitHub: pritic
Jenkins Account: pritichanglani
Description: The API Coverage plugin will serve as a tool to measure testing of the API contract

Thank you,
Priti

Oleg Nenashev

unread,
Aug 16, 2015, 7:41:02 AM8/16/15
to Jenkins Developers
I don't get the purpose of the plugin even after reading the code a bit. 
  • Does it just iterate through steps file and checks patterns from another file?
  • How is it related to "API coverage"? It seems to be a generic checker of external JSON file
  • Where are the format descriptions/requirements?
  • If there's a support of specific tools, please mention them
FYI, sometimes its better to create pull requests and ask the community to review the code before the first release.
  • there are bad practices like storing non-transient Build classes in Reports. It may cause many issues n the cause of the improper usage.
  • FileNotFound exceptions should be converted to AbortException, because you know the failure reason
  • Strange JSON handling magic in the publisher code, could be significantly refactored and optimized
  • Inconsistent null/empty string checks in parameters from config.xml
  • etc.




четверг, 13 августа 2015 г., 11:24:24 UTC+3 пользователь Priti Changlani написал:
Reply all
Reply to author
Forward
0 new messages