Skip to content

Commit

Permalink
update build verifiers to seperate url
Browse files Browse the repository at this point in the history
  • Loading branch information
hcoles committed Apr 2, 2024
1 parent 6f8342f commit acd0075
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,12 @@ class MissingJUnit5PluginVerifier implements BuildVerifier {
}

@Override
public List<String> verify() {
public List<BuildIssue> verifyBuild() {
if (!junit5PluginIsPresent() && junitJupiterPresent()) {
// log as well as return in case the run is aborted before messages are displayed at the end
String msg = "JUnit 5 is on the classpath but the pitest junit 5 plugin is not installed (https://github.com/pitest/pitest-junit5-plugin)";
String msg = "JUnit 5 is on the classpath but the pitest junit 5 plugin is not installed.";
Log.getLogger().warning(msg);
return asList(msg);
return asList(new BuildIssue(msg, "https://github.com/pitest/pitest-junit5-plugin", 5));
}

return Collections.emptyList();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,12 @@ class MissingTestNGPluginVerifier implements BuildVerifier {
}

@Override
public List<String> verify() {
public List<BuildIssue> verifyBuild() {
if (!testNGPluginIsPresent() && testNGisPresent()) {
// log as well as return in case the run is aborted before messages are displayed at the end
String msg = "TestNG is on the classpath but the pitest TestNG plugin is not installed (https://github.com/pitest/pitest-testng-plugin)";
String msg = "TestNG is on the classpath but the pitest TestNG plugin is not installed.";
Log.getLogger().warning(msg);
return asList(msg);
return asList(new BuildIssue(msg, "https://github.com/pitest/pitest-testng-plugin", 5));
}

return Collections.emptyList();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@ class SpringVerifier implements BuildVerifier {
}

@Override
public List<String> verify() {
public List<BuildIssue> verifyBuild() {
if (springIsOnClassPath() && !springPluginIsPresent()) {
return asList("Project uses Spring, but the Arcmutate Spring plugin is not present (https://docs.arcmutate.com/docs/spring.html)");
return asList(new BuildIssue("Project uses Spring, but the Arcmutate Spring plugin is not present.", "https://docs.arcmutate.com/docs/spring.html", 4));
}

return Collections.emptyList();
Expand Down

0 comments on commit acd0075

Please sign in to comment.