jmxtrans/jmxtrans — pull request #98 by LanyonM (2013-10-05 02:33:48+00:00)
Title: Crontrigger names

This commit fixes #97 by updating the cron trigger naming to use `server.getUrl()` instead of `server.getHost() + "-" + server.getPort()`. It goes further by adding functionality to `server.getHost()` and `server.getPort()` to infer the host and port from the url if it exists. If neither the host|port or url exist, the accessor will throw a runtime error.

I amended tests to allow this functionality since the existing tests did not specify a port when testing `Server` objects. This lead to some more general test writing.

To provide easy utilization of Maven's reporting capabilities, I added cobertura and findbugs to the pom. You can now run `mvn site` and get those reports in addition to other handy reports.