added test cases

Author: jepp3

src/main/java/org/codebrewery/AbstractRESTModel.java

@@ -2,7 +2,6 @@
 
 import com.ning.http.client.AsyncCompletionHandler;
 import com.ning.http.client.AsyncHttpClient;
-import com.ning.http.client.ListenableFuture;
 import com.ning.http.client.Response;
 import org.json.simple.JSONObject;
 import org.json.simple.parser.JSONParser;
@@ -26,6 +25,8 @@
  */
 public abstract class AbstractRESTModel implements CrudModelInterface {
 
+    private RestModelConfiguration config = new RestModelConfiguration();
+
     /**
      * This method returns the base url for a resource.
      *
@@ -56,32 +57,39 @@ public abstract class AbstractRESTModel implements CrudModelInterface {
     public abstract  AbstractRESTModel  parse(JSONObject response) throws IOException;
 
 
-    private String generateFullHttpUrl() {
+    protected String generateCollectionUrl() {
 
-        return "http://localhost:8081"+resourceUrl();
+        return config.getRequestBaseUrl() + "/" + resourceUrl();
 
     }
 
-    private JSONObject convertResponseToJSONObject(Response response) throws IOException, ParseException {
+    protected String generatedInstanceUrl() {
 
-        return (JSONObject)new JSONParser().parse(response.getResponseBody());
+        return generateCollectionUrl() + "/" + identifierValue();
 
     }
 
-    public void fetch(final ActionCompletedInterface actions) throws ExecutionException, InterruptedException {
+    public void setConfiguration( RestModelConfiguration config ) {
 
+        this.config = config;
 
-        AsyncHttpClient asyncHttpClient = new AsyncHttpClient();
-        String urlToUse = generateFullHttpUrl() + identifierValue();
-        asyncHttpClient.prepareGet(urlToUse).execute(new AsyncCompletionHandler() {
+    }
+    protected JSONObject convertResponseToJSONObject(Response response) throws IOException, ParseException {
+
+        return (JSONObject)new JSONParser().parse(response.getResponseBody());
+
+    }
+
+    protected AsyncCompletionHandler getAsyncCompletionHandler(ActionCompletedInterface actions) {
+
+        return new AsyncCompletionHandler(){
 
             @Override
             public Object onCompleted(Response response) throws Exception {
 
 
                 actions.onDone(parse(convertResponseToJSONObject(response)));
 
-                // is it possible to not return void?
                 return null;
             }
 
@@ -91,7 +99,18 @@ public void onThrowable(Throwable t) {
                 actions.onError(t);
 
             }
-        });
+        };
+
+    }
+
+
+    public void fetch(final ActionCompletedInterface actions) throws ExecutionException, InterruptedException {
+
+
+        AsyncHttpClient asyncHttpClient = new AsyncHttpClient();
+        String urlToUse = generatedInstanceUrl();
+        asyncHttpClient.prepareGet(urlToUse).execute(getAsyncCompletionHandler(actions));
+
 
     }
 

src/main/java/org/codebrewery/App.java

@@ -0,0 +1,32 @@
+package org.codebrewery;
+
+/**
+ * Created by jepp3 on 2015-09-06.
+ */
+public class RestModelConfiguration {
+
+
+    public String getRequestBaseUrl(){
+
+        return getBaseUrl() + ":" + getPort() + "/" + getApiLocation();
+
+    }
+
+    public String getBaseUrl() {
+
+        return "http://localhost";
+
+    }
+
+    public String getPort() {
+
+        return "8081";
+
+    }
+
+    public String getApiLocation() {
+
+        return "api";
+
+    }
+}

src/main/java/org/codebrewery/ClientConfigurer.java

@@ -15,11 +15,6 @@ public class DogModel extends AbstractRESTModel {
 
     public final int age;
 
-    public DogModel() {
-
-        name = "pluto";
-        age = 5;
-    }
 
     public DogModel(String name, int age) {
         this.name = name;
@@ -46,6 +41,7 @@ public final AbstractRESTModel  parse(JSONObject responseJson) {
         return new DogModel(name,123);
 
     }
+
     // dedicated method
     public String bark() {
         return "VOFF says" + this.name;

src/main/java/org/codebrewery/ClientConfigurerInterface.java

@@ -0,0 +1,118 @@
+package org.codebrewery;
+
+import com.ning.http.client.FluentCaseInsensitiveStringsMap;
+import com.ning.http.client.Response;
+import com.ning.http.client.cookie.Cookie;
+import com.ning.http.client.uri.Uri;
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.nio.ByteBuffer;
+import java.util.List;
+
+/**
+ * Created by jepp3 on 2015-09-06.
+ */
+public class MockResponse implements Response {
+
+    @Override
+    public int getStatusCode() {
+        return 0;
+    }
+
+    @Override
+    public String getStatusText() {
+        return null;
+    }
+
+    @Override
+    public byte[] getResponseBodyAsBytes() throws IOException {
+        return new byte[0];
+    }
+
+    @Override
+    public ByteBuffer getResponseBodyAsByteBuffer() throws IOException {
+        return null;
+    }
+
+    @Override
+    public InputStream getResponseBodyAsStream() throws IOException {
+        return null;
+    }
+
+    @Override
+    public String getResponseBodyExcerpt(int i, String s) throws IOException {
+        return null;
+    }
+
+    @Override
+    public String getResponseBody(String s) throws IOException {
+        return null;
+    }
+
+    @Override
+    public String getResponseBodyExcerpt(int i) throws IOException {
+        return null;
+    }
+
+    @Override
+    public String getResponseBody() throws IOException {
+        return "{\"name\":\"pluto\"}";
+    }
+
+    @Override
+    public Uri getUri() {
+        return null;
+    }
+
+    @Override
+    public String getContentType() {
+        return null;
+    }
+
+    @Override
+    public String getHeader(String s) {
+        return null;
+    }
+
+    @Override
+    public List<String> getHeaders(String s) {
+        return null;
+    }
+
+    @Override
+    public FluentCaseInsensitiveStringsMap getHeaders() {
+        return null;
+    }
+
+    @Override
+    public boolean isRedirected() {
+        return false;
+    }
+
+    @Override
+    public String toString() {
+        return null;
+    }
+
+    @Override
+        public List<Cookie> getCookies() {
+            return null;
+        }
+
+        @Override
+        public boolean hasResponseStatus() {
+            return false;
+        }
+
+        @Override
+        public boolean hasResponseHeaders() {
+            return false;
+        }
+
+        @Override
+        public boolean hasResponseBody() {
+            return false;
+        }
+
+}

src/main/java/org/codebrewery/RestModelConfiguration.java

@@ -0,0 +1,31 @@
+package org.codebrewery;
+
+import com.ning.http.client.AsyncHttpClient;
+import org.json.simple.JSONObject;
+
+import java.io.IOException;
+
+/**
+ * Created by jepp3 on 2015-09-06.
+ */
+public class MockRestModel extends AbstractRESTModel{
+    @Override
+    String resourceUrl() {
+        return "dogs";
+    }
+
+    @Override
+    String identifierValue() {
+
+        return "identifier";
+
+    }
+
+    @Override
+    public AbstractRESTModel parse(JSONObject response) throws IOException {
+
+        return new MockRestModel();
+    }
+
+
+}