Merge pull request #302 from dellgreen/dpg/refactor/301/httpclientint…

…erface

Fixed #301 Decoupled JenkinsServer and JenkinsHttpClient

ReleaseNotes.md

@@ -3,6 +3,13 @@
 ## Release 0.3.8 (NOT RELEASED YET)
 
 
+ * [Fixed Issue 301][issue-301]
+
+   Decoupled JenkinsServer and JenkinsHttpClient by extracting JenkinsHttpClient 
+   methods into public interface so that different implementations can be plugged
+   into JenkinsServer if required
+
+
  * [Fixed Issue 298][issue-298]
 
    Added Closeable support to JenkinsServer and JenkinsHttpClient so that

jenkins-client/src/main/java/com/offbytwo/jenkins/JenkinsServer.java

@@ -25,6 +25,7 @@
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Maps;
 import com.offbytwo.jenkins.client.JenkinsHttpClient;
+import com.offbytwo.jenkins.client.JenkinsHttpConnection;
 import com.offbytwo.jenkins.client.util.EncodingUtils;
 import com.offbytwo.jenkins.client.util.UrlUtils;
 import com.offbytwo.jenkins.helper.JenkinsVersion;
@@ -51,7 +52,10 @@
 public class JenkinsServer implements Closeable {
     private final Logger LOGGER = LoggerFactory.getLogger(getClass());
 
-    private final JenkinsHttpClient client;
+    /**
+     * The transport client instance to use.
+     */
+    private final JenkinsHttpConnection client;
 
     /**
      * Create a new Jenkins server reference given only the server address
@@ -80,7 +84,7 @@ public JenkinsServer(URI serverUri, String username, String passwordOrToken) {
      *
      * @param client Specialized client to use.
      */
-    public JenkinsServer(JenkinsHttpClient client) {
+    public JenkinsServer(final JenkinsHttpConnection client) {
         this.client = client;
     }