Implement error log api and fix minor issues

Author: rubengees

library/src/main/java/me/proxer/library/api/ProxerApi.java

@@ -6,6 +6,7 @@
 import lombok.experimental.Accessors;
 import me.proxer.library.BuildConfig;
 import me.proxer.library.api.anime.AnimeApi;
+import me.proxer.library.api.apps.AppsApi;
 import me.proxer.library.api.forum.ForumApi;
 import me.proxer.library.api.info.InfoApi;
 import me.proxer.library.api.list.ListApi;
@@ -102,6 +103,11 @@ public final class ProxerApi {
      */
     private final MediaApi media;
 
+    /**
+     * Returns the respective API.
+     */
+    private final AppsApi apps;
+
     private ProxerApi(final Moshi moshi, final OkHttpClient client, final Retrofit retrofit) {
         this.moshi = moshi;
         this.client = client;
@@ -117,6 +123,7 @@ private ProxerApi(final Moshi moshi, final OkHttpClient client, final Retrofit r
         manga = new MangaApi(retrofit);
         forum = new ForumApi(retrofit);
         media = new MediaApi(retrofit);
+        apps = new AppsApi(retrofit);
     }
 
     /**

library/src/main/java/me/proxer/library/api/apps/AppsApi.java

@@ -0,0 +1,27 @@
+package me.proxer.library.api.apps;
+
+import retrofit2.Retrofit;
+
+/**
+ * Api for the Media class.
+ *
+ * @author Ruben Gees
+ */
+public final class AppsApi {
+
+    private final InternalApi internalApi;
+
+    /**
+     * Only for internal use.
+     */
+    public AppsApi(final Retrofit retrofit) {
+        this.internalApi = retrofit.create(InternalApi.class);
+    }
+
+    /**
+     * Returns the respective endpoint.
+     */
+    public ErrorLogEndpoint errorLog(final String id, final String message) {
+        return new ErrorLogEndpoint(internalApi, id, message);
+    }
+}

library/src/main/java/me/proxer/library/api/apps/ErrorLogEndpoint.java

@@ -0,0 +1,40 @@
+package me.proxer.library.api.apps;
+
+import lombok.Setter;
+import lombok.experimental.Accessors;
+import me.proxer.library.api.Endpoint;
+import me.proxer.library.api.ProxerCall;
+
+import javax.annotation.Nullable;
+
+/**
+ * Endpoint for sending error logs.
+ *
+ * @author Ruben Gees
+ */
+@Accessors(fluent = true)
+public final class ErrorLogEndpoint implements Endpoint<Void> {
+
+    private final InternalApi internalApi;
+
+    private final String id;
+    private final String message;
+
+    /**
+     * Sets if the error log should be sent anonymously. If not set, this behaves as true.
+     */
+    @Nullable
+    @Setter
+    private Boolean anonym;
+
+    ErrorLogEndpoint(final InternalApi internalApi, final String id, final String message) {
+        this.internalApi = internalApi;
+        this.id = id;
+        this.message = message;
+    }
+
+    @Override
+    public ProxerCall<Void> build() {
+        return internalApi.errorLog(id, message, anonym);
+    }
+}

library/src/main/java/me/proxer/library/api/apps/InternalApi.java

@@ -0,0 +1,21 @@
+package me.proxer.library.api.apps;
+
+import me.proxer.library.api.ProxerCall;
+import retrofit2.http.Field;
+import retrofit2.http.FormUrlEncoded;
+import retrofit2.http.POST;
+
+import javax.annotation.ParametersAreNullableByDefault;
+
+/**
+ * @author Ruben Gees
+ */
+@ParametersAreNullableByDefault
+public interface InternalApi {
+
+    @FormUrlEncoded
+    @POST("apps/errorlog")
+    ProxerCall<Void> errorLog(@Field("id") String id,
+                              @Field("message") String message,
+                              @Field("anonym") Boolean anonym);
+}

library/src/main/java/me/proxer/library/api/media/CalendarEndpoint.java

@@ -8,6 +8,8 @@
 import java.util.List;
 
 /**
+ * Endpoint for retrieving the anime calendar/schedule.
+ *
  * @author Ruben Gees
  */
 @Accessors(fluent = true)

library/src/main/java/me/proxer/library/api/ucp/BookmarksEndpoint.java

@@ -24,22 +24,22 @@ public final class BookmarksEndpoint implements PagingLimitEndpoint<List<Bookmar
      * {@inheritDoc}
      */
     @Nullable
-    @Setter
-    private Category category;
+    @Setter(onMethod = @__({@Override}))
+    private Integer page;
 
     /**
      * {@inheritDoc}
      */
     @Nullable
     @Setter(onMethod = @__({@Override}))
-    private Integer page;
+    private Integer limit;
 
     /**
-     * {@inheritDoc}
+     * Sets the type of category to return.
      */
     @Nullable
-    @Setter(onMethod = @__({@Override}))
-    private Integer limit;
+    @Setter
+    private Category category;
 
     /**
      * Sets if only available or only not available bookmarks should be returned.

library/src/test/java/me/proxer/library/api/apps/ErrorLogEndpointTest.java

@@ -0,0 +1,48 @@
+package me.proxer.library.api.apps;
+
+import me.proxer.library.ProxerTest;
+import me.proxer.library.api.ProxerException;
+import okhttp3.mockwebserver.MockResponse;
+import org.junit.Test;
+
+import java.io.IOException;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+public class ErrorLogEndpointTest extends ProxerTest {
+
+    @Test
+    public void testDefault() throws ProxerException, IOException {
+        server.enqueue(new MockResponse().setBody(fromResource("empty.json")));
+
+        final Void result = api.apps()
+                .errorLog("3", "test message")
+                .build()
+                .execute();
+
+        assertThat(result).isNull();
+    }
+
+    @Test
+    public void testPath() throws ProxerException, IOException, InterruptedException {
+        server.enqueue(new MockResponse().setBody(fromResource("empty.json")));
+
+        api.apps().errorLog("3", "test message")
+                .build()
+                .execute();
+
+        assertThat(server.takeRequest().getPath()).isEqualTo("/api/v1/apps/errorlog");
+    }
+
+    @Test
+    public void testParameters() throws IOException, ProxerException, InterruptedException {
+        server.enqueue(new MockResponse().setBody(fromResource("empty.json")));
+
+        api.apps().errorLog("3", "test message")
+                .anonym(false)
+                .build()
+                .execute();
+
+        assertThat(server.takeRequest().getBody().readUtf8()).isEqualTo("id=3&message=test%20message&anonym=false");
+    }
+}