Skip to content

Commit

Permalink
WIP 314 Remove duplicate condition check
Browse files Browse the repository at this point in the history
Perhaps it was left from some merge action?
  • Loading branch information
ascheman committed Dec 14, 2024
1 parent 6f885e7 commit 8bc470c
Showing 1 changed file with 3 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -67,10 +67,10 @@ public class HtmlSanityCheckMojo extends AbstractMojo {
public void execute() throws MojoExecutionException {
logBuildParameter();

// Setup configuration
// Setup configuration
Configuration myConfig = setupConfiguration();

// Check if configuration is valid
// Check if configuration is valid
try {
myConfig.validate();
// Create output directories
Expand Down Expand Up @@ -100,12 +100,6 @@ public void execute() throws MojoExecutionException {
);
throw new MojoExecutionException(failureMsg);
}
if (junitResultsDir != null) {
junitResultsDir.mkdirs();
if (!junitResultsDir.isDirectory() || !junitResultsDir.canWrite()) {
throw new MojoExecutionException("Cannot write to JUnit results directory.");
}
}
} catch (MisconfigurationException e) {
throw new MojoExecutionException(e);
} catch (IOException e) {
Expand Down Expand Up @@ -146,7 +140,7 @@ protected Configuration setupConfiguration() {
if (httpSuccessCodes != null && !httpSuccessCodes.isEmpty()) {
result.overrideHttpSuccessCodes(httpSuccessCodes);
}
if (httpErrorCodes != null && !httpErrorCodes.isEmpty()) {
if (httpErrorCodes != null && !httpErrorCodes.isEmpty()) {
result.overrideHttpErrorCodes(httpErrorCodes);
}
if (httpErrorCodes != null && !httpWarningCodes.isEmpty()) {
Expand Down

0 comments on commit 8bc470c

Please sign in to comment.