#36 Decompose method to make code testable

Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.com>

Author: Weltraumschaf

src/main/java/io/securecodebox/persistence/defectdojo/http/Foo.java

@@ -69,7 +69,7 @@ public RestTemplate createRestTemplate() {
 
             final var builder = HttpClientBuilder.create();
             builder.useSystemProperties();
-            builder.setProxy(new HttpHost(proxyConfig.getHost(), proxyConfig.getPort()));
+            builder.setProxy(createHttpHost());
             builder.setDefaultCredentialsProvider(credentials);
             builder.setProxyAuthenticationStrategy(new ProxyAuthenticationStrategy());
 
@@ -88,4 +88,8 @@ AuthScope createAuthScope() {
     Credentials createCredentials() {
         return new UsernamePasswordCredentials(proxyConfig.getUser(), proxyConfig.getPassword());
     }
+
+    HttpHost createHttpHost() {
+        return new HttpHost(proxyConfig.getHost(), proxyConfig.getPort());
+    }
 }

src/test/java/io/securecodebox/persistence/defectdojo/http/FooTest.java

@@ -84,6 +84,14 @@ void createCredentials() {
         assertAll(
             () -> assertThat(result.getUserPrincipal().getName(), is(proxyConfig.getUser())),
             () -> assertThat(result.getPassword(), is(proxyConfig.getPassword()))
+
+    @Test
+    void createHttpHost() {
+        final var result = sut.createHttpHost();
+
+        assertAll(
+                () -> assertThat(result.getHostName(), is(proxyConfig.getHost())),
+                () -> assertThat(result.getPort(), is(proxyConfig.getPort()))
         );
     }
 }