Merge pull request #25 from RestComm/feat/bs-131

New naming convention for query methods in RestEndpoints

Author: agafox

src/main/java/org/restcomm/sdk/endpoints/RestEndpoints.java

@@ -7,6 +7,7 @@
 
 /**
  * @author [email protected] (Oleg Agafonov)
+ * @author [email protected] (Henrique Rosa)
  */
 public class RestEndpoints<T> extends Endpoints {
 
@@ -18,31 +19,31 @@ public RestEndpoints(String url, HttpClient httpClient, Class<T> type) {
     }
 
     public List<T> getAll() {
-        return getAll(null);
+        return find(null);
     }
 
-    public List<T> getAll(Object query) {
-        return httpClient.get(url, query, TypeFactory
-                .defaultInstance().constructCollectionType(List.class, type));
+    public T get(String id) {
+        return httpClient.get(url + "/" + id, type);
     }
 
-    public T get(Object query) {
+    public List<T> find(Object query) {
+        return httpClient.get(url, query, TypeFactory.defaultInstance().constructCollectionType(List.class, type));
+    }
+
+    public T findOne(Object query) {
         return httpClient.get(url, query, TypeFactory.defaultInstance().constructType(type));
     }
 
     public T create(T entity) {
         return httpClient.post(url, entity, type);
     }
 
-    public T get(String id) {
-        return httpClient.get(url + "/" + id, type);
-    }
-
     public T update(T entity) {
         return httpClient.put(url, entity, type);
     }
 
     public void delete(String id) {
         httpClient.delete(url + "/" + id, type);
     }
+
 }

src/test/java/org/restcomm/sdk/endpoints/CallsEndpointTest.java

@@ -44,7 +44,7 @@ public void testGetCallPage() throws IOException {
                 .willReturn(okJson(response)));
 
         // when
-        final CallPage callPage = callsEndpoint.get(Collections.emptyMap());
+        final CallPage callPage = callsEndpoint.findOne(Collections.emptyMap());
 
         // then
         assertNotNull(callPage);