diff --git a/core/src/main/java/hudson/tasks/junit/TestResult.java b/core/src/main/java/hudson/tasks/junit/TestResult.java index 143fa42b0b1fd2911e80536268520454a3e91508..c462cbae6109aacdcf2a901e20c50d05e3c30e53 100644 --- a/core/src/main/java/hudson/tasks/junit/TestResult.java +++ b/core/src/main/java/hudson/tasks/junit/TestResult.java @@ -296,7 +296,7 @@ public final class TestResult extends MetaTabulatedResult { @Override public Run getRun() { - return (parentAction == null? null: parentAction.owner); + return (parentAction == null? null: parentAction.run); } @Override diff --git a/core/src/main/java/hudson/tasks/junit/TestResultAction.java b/core/src/main/java/hudson/tasks/junit/TestResultAction.java index f0f83dc71351ccfe30ab53993e602edab5a07b39..1d87288a8a4ee3331b91b5b282fd80ae0df30222 100644 --- a/core/src/main/java/hudson/tasks/junit/TestResultAction.java +++ b/core/src/main/java/hudson/tasks/junit/TestResultAction.java @@ -117,7 +117,7 @@ public class TestResultAction extends AbstractTestResultAction } private XmlFile getDataFile() { - return new XmlFile(XSTREAM,new File(owner.getRootDir(), "junitResult.xml")); + return new XmlFile(XSTREAM, new File(run.getRootDir(), "junitResult.xml")); } public synchronized TestResult getResult() { diff --git a/core/src/main/java/hudson/tasks/test/AbstractTestResultAction.java b/core/src/main/java/hudson/tasks/test/AbstractTestResultAction.java index e63a8101e480259ea29a5b735fe781e4d2eb5085..a113457d806f357de9d1674e976c0025bad639c4 100644 --- a/core/src/main/java/hudson/tasks/test/AbstractTestResultAction.java +++ b/core/src/main/java/hudson/tasks/test/AbstractTestResultAction.java @@ -251,7 +251,7 @@ public abstract class AbstractTestResultAction dsb = new DataSetBuilder(); for( AbstractTestResultAction a=this; a!=null; a=a.getPreviousResult(AbstractTestResultAction.class) ) { - dsb.add( a.getFailCount(), "failed", new NumberOnlyBuildLabel(a.owner)); + dsb.add( a.getFailCount(), "failed", new NumberOnlyBuildLabel(a.run)); if(!failureOnly) { - dsb.add( a.getSkipCount(), "skipped", new NumberOnlyBuildLabel(a.owner)); - dsb.add( a.getTotalCount()-a.getFailCount()-a.getSkipCount(),"total", new NumberOnlyBuildLabel(a.owner)); + dsb.add( a.getSkipCount(), "skipped", new NumberOnlyBuildLabel(a.run)); + dsb.add( a.getTotalCount()-a.getFailCount()-a.getSkipCount(),"total", new NumberOnlyBuildLabel(a.run)); } } return dsb.build(); @@ -351,20 +351,20 @@ public abstract class AbstractTestResultAction { + + private final Run run; + + @Deprecated public final AbstractBuild build; + /** @since TODO */ + public NumberOnlyBuildLabel(Run run) { + this.run = run; + this.build = run instanceof AbstractBuild ? (AbstractBuild) run : null; + } + + @Deprecated public NumberOnlyBuildLabel(AbstractBuild build) { + this.run = build; this.build = build; } + /* @since TODO */ + public Run getRun() { + return run; + } + public int compareTo(NumberOnlyBuildLabel that) { - return this.build.number-that.build.number; + return this.run.number-that.run.number; } @Override public boolean equals(Object o) { if(!(o instanceof NumberOnlyBuildLabel)) return false; NumberOnlyBuildLabel that = (NumberOnlyBuildLabel) o; - return build==that.build; + return run==that.run; } @Override public int hashCode() { - return build.hashCode(); + return run.hashCode(); } @Override public String toString() { - return build.getDisplayName(); + return run.getDisplayName(); } }