Skip to content
This repository has been archived by the owner on Apr 6, 2022. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-14989] Fixed NPE in WarningsResult when re-loading
warnings. Introduced new base class HealthAwareRecorder that has no
template method for perform.
  • Loading branch information
uhafner committed Sep 4, 2012
1 parent df6526b commit abf23c9
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 12 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Expand Up @@ -36,7 +36,7 @@
<dependency>
<groupId>org.jvnet.hudson.plugins</groupId>
<artifactId>analysis-core</artifactId>
<version>1.45</version>
<version>1.46-SNAPSHOT</version>
</dependency>
<dependency>
<groupId>org.jvnet.hudson.plugins</groupId>
Expand Down
44 changes: 33 additions & 11 deletions src/main/java/hudson/plugins/warnings/WarningsPublisher.java
Expand Up @@ -9,9 +9,8 @@
import hudson.model.AbstractProject;
import hudson.plugins.analysis.core.AnnotationsClassifier;
import hudson.plugins.analysis.core.FilesParser;
import hudson.plugins.analysis.core.HealthAwarePublisher;
import hudson.plugins.analysis.core.HealthAwareRecorder;
import hudson.plugins.analysis.core.ParserResult;
import hudson.plugins.analysis.core.BuildResult;
import hudson.plugins.analysis.util.ModuleDetector;
import hudson.plugins.analysis.util.NullModuleDetector;
import hudson.plugins.analysis.util.PluginLogger;
Expand All @@ -38,7 +37,7 @@
* @author Ulli Hafner
*/
// CHECKSTYLE:COUPLING-OFF
public class WarningsPublisher extends HealthAwarePublisher {
public class WarningsPublisher extends HealthAwareRecorder {
private static final String PLUGIN_NAME = "WARNINGS";

private static final long serialVersionUID = -5936973521277401764L;
Expand Down Expand Up @@ -264,8 +263,9 @@ private List<String> getParsers() {
return parsers;
}

/** {@inheritDoc} */
@Override
public BuildResult perform(final AbstractBuild<?, ?> build, final PluginLogger logger)
protected boolean perform(final AbstractBuild<?, ?> build, final Launcher launcher, final PluginLogger logger)
throws InterruptedException, IOException {
try {
if (!hasConsoleParsers() && !hasFileParsers()) {
Expand All @@ -279,18 +279,40 @@ public BuildResult perform(final AbstractBuild<?, ?> build, final PluginLogger l
add(totals, consoleResults);
add(totals, fileResults);

return new WarningsResult(build, new WarningsBuildHistory(build, null), totals, getDefaultEncoding(), null);
if (isThresholdEnabled()) {
evaluateBuildHealth(build, logger);
}

copyFilesWithAnnotationsToBuildFolder(build.getRootDir(), launcher.getChannel(), totals.getAnnotations());

return true;
}
catch (ParsingCanceledException exception) {
throw createInterruptedException();
return stopParsing(logger, exception);
}
catch (InterruptedException exception) {
return stopParsing(logger, exception);
}
}

@Override
protected void updateBuildResult(final BuildResult result, final PluginLogger logger) {
for (WarningsResultAction action : result.getOwner().getActions(WarningsResultAction.class)) {
action.getResult().evaluateStatus(getThresholds(), getUseDeltaValues(), canComputeNew(), logger,
action.getUrlName());
private boolean stopParsing(final PluginLogger logger, final Exception exception) {
logger.log(exception.getMessage());

return false;
}

private void evaluateBuildHealth(final AbstractBuild<?, ?> build, final PluginLogger logger) {
for (WarningsResultAction action : build.getActions(WarningsResultAction.class)) {
WarningsBuildHistory history = new WarningsBuildHistory(build, action.getParser());
AbstractBuild<?, ?> referenceBuild = history.getReferenceBuild();
if (referenceBuild == null) {
logger.log("Skipping warning delta computation since no reference build is found");
}
else {
logger.log("Computing warning deltas based on reference build " + referenceBuild.getDisplayName());
action.getResult().evaluateStatus(getThresholds(), getUseDeltaValues(), canComputeNew(),
logger, action.getUrlName());
}
}
}

Expand Down

0 comments on commit abf23c9

Please sign in to comment.