JENKINS-43538 API change: remove getTestUrl

jenkinsci · Apr 13, 2017 · 30b0b0d · 30b0b0d
1 parent 0e12d95
commit 30b0b0d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/pom.xml b/pom.xml
@@ -68,7 +68,7 @@
     <dependency>
       <groupId>org.jenkins-ci.plugins</groupId>
       <artifactId>display-url-api</artifactId>
-      <version>1.1.1</version>
+      <version>1.2-SNAPSHOT</version>
     </dependency>
     <dependency>
       <groupId>org.jenkins-ci.plugins.workflow</groupId>

diff --git a/src/main/java/org/jenkinsci/plugins/blueoceandisplayurl/BlueOceanDisplayURLImpl.java b/src/main/java/org/jenkinsci/plugins/blueoceandisplayurl/BlueOceanDisplayURLImpl.java
@@ -7,7 +7,6 @@
 import hudson.model.FreeStyleProject;
 import hudson.model.Job;
 import hudson.model.Run;
-import hudson.tasks.test.TestResult;
 import jenkins.branch.MultiBranchProject;
 import jenkins.model.Jenkins;
 import org.jenkinsci.plugins.displayurlapi.DisplayURLProvider;
@@ -94,20 +93,15 @@ public String getJobURL(Job<?, ?> job) {
         }
     }
 
-    @Override
-    public String getTestUrl(TestResult result) {
-        return getRunURL(result.getRun()) + "/tests";
-    }
-
-    static boolean isSupported(Run<?, ?> run) {
+    private static boolean isSupported(Run<?, ?> run) {
         return isInstance(run, SUPPORTED_RUNS);
     }
 
-    static boolean isSupported(Job<?, ?> job) {
+    private static boolean isSupported(Job<?, ?> job) {
         return isInstance(job, SUPPORTED_JOBS);
     }
 
-    static boolean isInstance(Object o, Set<String> clazzes) {
+    private static boolean isInstance(Object o, Set<String> clazzes) {
         for (String clazz : clazzes) {
             if (o != null && o.getClass().getName().equals(clazz)) {
                 return true;