Add request time to output (#94)

src/main/java/willitconnect/model/CheckedEntry.java

@@ -11,11 +11,13 @@
  * Holds a hostname or url that is checked to see if we can connect to it.
  */
 public class CheckedEntry {
+    public static long DEFAULT_RESPONSE_TIME = -1L;
     private Date lastChecked;
     private String entry;
     private boolean canConnect;
     private int httpStatus;
     private String httpProxy;
+    private long responseTime = DEFAULT_RESPONSE_TIME;
 
     public boolean isCanConnect() {
         return canConnect();
@@ -88,4 +90,13 @@ public void setHttpProxy(String proxy) {
     public String getHttpProxy() {
         return httpProxy;
     }
+
+
+    public long getResponseTime() {
+        return responseTime;
+    }
+
+    public void setResponseTime(long responseTime) {
+        this.responseTime = responseTime;
+    }
 }

src/main/java/willitconnect/service/EntryChecker.java

@@ -1,6 +1,7 @@
 package willitconnect.service;
 
 import org.apache.log4j.Logger;
+import org.apache.tomcat.jni.Time;
 import org.springframework.http.ResponseEntity;
 import org.springframework.http.client.ClientHttpRequestFactory;
 import org.springframework.http.client.ClientHttpResponse;
@@ -45,13 +46,15 @@ public EntryChecker(RestTemplate restTemplate) {
 
     public CheckedEntry check(CheckedEntry e) {
         log.info("checking " + e.getEntry());
+        long startTime = Instant.now().toEpochMilli();
         if (e.isValidUrl()) {
             checkUrl(e);
         } else if (e.isValidHostname()) {
             checkHostname(e);
         } else {
             log.error(e.getEntry() + " is not a valid hostname");
         }
+        e.setResponseTime(Instant.now().toEpochMilli() - startTime);
         return e;
     }
 

src/main/script/Entry.js

@@ -67,9 +67,10 @@ const Entry = ({
   success,
   httpStatus,
   time,
+  responseTime,
 }) =>
   <ul>
-    <li>  On {time}, I {success ? 'could' : 'could not'} connect.
+    <li>  On {time}, I {success ? 'could' : 'could not'} connect. It took {responseTime} ms.
       {httpStatus !== 0 && <span>Http Status: {httpStatus}</span>}
     </li>
   </ul>;
@@ -97,6 +98,7 @@ const StatelessEntry = (props) => {
           success={attempt.canConnect}
           httpStatus={attempt.httpStatus}
           time={(new Date(attempt.time)).toLocaleString()}
+          responseTime={attempt.responseTime}
         />
       );
     }).reverse()}

src/test/java/willitconnect/service/EntryCheckerTest.java

@@ -16,6 +16,7 @@
 import java.time.Instant;
 
 import static org.hamcrest.CoreMatchers.equalTo;
+import static org.hamcrest.CoreMatchers.not;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.greaterThan;
 import static org.hamcrest.core.Is.is;
@@ -24,6 +25,7 @@
 import static org.springframework.test.web.client.match.MockRestRequestMatchers.requestTo;
 import static org.springframework.test.web.client.response.MockRestResponseCreators.withServerError;
 import static org.testng.AssertJUnit.assertEquals;
+import static willitconnect.model.CheckedEntry.DEFAULT_RESPONSE_TIME;
 
 public class EntryCheckerTest {
 
@@ -83,8 +85,14 @@ public void itShouldConnectToAURLThroughAProxy() {
         entry.setHttpProxy("proxy.example.com:8080");
         CheckedEntry returnedEntry = checker.check(entry);
 
-
         assertThat(returnedEntry.getHttpProxy(),
                 is(equalTo("proxy.example.com:8080")));
     }
+
+    @Test
+    public void itShouldSayHowLongItTookToConnect() {
+        CheckedEntry returnedEntry = checker.check(entry);
+
+        assertThat(returnedEntry.getResponseTime(), is(not(equalTo(DEFAULT_RESPONSE_TIME))));
+    }
 }