Stopped using the @deprecated constructor and use Run#addAction instead

petr-klemsinsky · Aug 30, 2015 · 54970b9 · 54970b9
1 parent 19e9df4
commit 54970b9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/src/main/java/org/zeroturnaround/jenkins/flowbuildtestaggregator/FlowTestAggregator.java b/src/main/java/org/zeroturnaround/jenkins/flowbuildtestaggregator/FlowTestAggregator.java
@@ -41,7 +41,8 @@ public BuildStepMonitor getRequiredMonitorService() {
 
   @Override
   public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListener listener) throws InterruptedException, IOException {
-    FlowTestResults testResults = new FlowTestResults(build);
+    FlowTestResults testResults = new FlowTestResults();
+    build.addAction(testResults);
     FlowRun flowRun = (FlowRun) build;
     flowRun.addAction(testResults);
     listener.getLogger().println("Starting to gather test results!");

diff --git a/src/main/java/org/zeroturnaround/jenkins/flowbuildtestaggregator/FlowTestResults.java b/src/main/java/org/zeroturnaround/jenkins/flowbuildtestaggregator/FlowTestResults.java
@@ -8,10 +8,6 @@
 
 public class FlowTestResults extends AggregatedTestResultAction {
 
-  public FlowTestResults(AbstractBuild owner) {
-    super(owner);
-  }
-
   @Override
   protected String getChildName(AbstractTestResultAction tr) {
     return tr.owner.getProject().getFullName();