Fixed #176 (The defaultAllowed parameter is not added to request for thumbnails service)

Author: TanyaEf

src/main/java/com/jaspersoft/jasperserver/jaxrs/client/apiadapters/thumbnails/SingleThumbnailAdapter.java

@@ -18,13 +18,16 @@
 public class SingleThumbnailAdapter extends AbstractAdapter {
 
     private final MultivaluedHashMap<String, String> params = new MultivaluedHashMap<String, String>();
+    private String reportName;
 
     public SingleThumbnailAdapter(SessionStorage sessionStorage) {
         super(sessionStorage);
     }
 
     public SingleThumbnailAdapter report(String uri) {
-        params.add("uri", uri);
+        if (uri != null && !uri.equals("")) {
+            reportName = uri;
+        }
         return this;
     }
 
@@ -34,14 +37,17 @@ public SingleThumbnailAdapter defaultAllowed(Boolean value) {
     }
 
     public OperationResult<InputStream> get() {
-        return request().setAccept("image/jpeg").get();
+        return request().get();
     }
 
     private JerseyRequest<InputStream> request() {
-        if (params.size() == 0) {
+        if (reportName == null) {
             throw new MandatoryParameterNotFoundException("URI of report should be specified");
         }
-        return JerseyRequest.buildRequest(sessionStorage, InputStream.class,
-                new String[]{"/thumbnails",  params.get("uri").get(0)}, new DefaultErrorHandler());
+        JerseyRequest<InputStream> request = JerseyRequest.buildRequest(sessionStorage, InputStream.class,
+                new String[]{"/thumbnails", reportName}, new DefaultErrorHandler());
+        request.setAccept("image/jpeg");
+        request.addParams(params);
+        return request;
     }
 }

src/test/java/com/jaspersoft/jasperserver/jaxrs/client/apiadapters/thumbnails/SingleThumbnailAdapterTest.java

@@ -74,12 +74,28 @@ public void should_set_report_uri() {
         // When
         SingleThumbnailAdapter retrieved = thumbnailAdapter.report("/public/Samples/Reports/07g.RevenueDetailReport");
 
+        // Then
+        String reportName = (String) Whitebox.getInternalState(retrieved, "reportName");
+        assertEquals(reportName, "/public/Samples/Reports/07g.RevenueDetailReport");
+    }
+
+    @Test
+    /**
+     * for {@link SingleThumbnailAdapter#report(String)}
+     */
+    public void should_set_report_uri_with_default_allowed() {
+        // Given
+        SingleThumbnailAdapter thumbnailAdapter = new SingleThumbnailAdapter(sessionStorageMock);
+
+        // When
+        SingleThumbnailAdapter retrieved = thumbnailAdapter.defaultAllowed(true).report("/public/Samples/Reports/07g.RevenueDetailReport");
+
         // Then
         MultivaluedHashMap<String, String> params =
-                (MultivaluedHashMap<String, String>) Whitebox.getInternalState(thumbnailAdapter, "params");
-        List<String> list = params.get("uri");
+                (MultivaluedHashMap<String, String>) Whitebox.getInternalState(retrieved, "params");
+        List<String> list = params.get("defaultAllowed");
         assertSame(retrieved, thumbnailAdapter);
-        assertEquals(list.get(0), "/public/Samples/Reports/07g.RevenueDetailReport");
+        assertEquals(list.get(0), "true");
     }
 
     @Test
@@ -115,8 +131,8 @@ public void should_return_proper_operation_result() {
                 eq(InputStream.class),
                 eq(new String[]{"/thumbnails", "/public/Samples/Reports/07g.RevenueDetailReport"}),
                 any(DefaultErrorHandler.class))).thenReturn(jerseyRequestMock);
-        when(jerseyRequestMock.setAccept("image/jpeg")).thenReturn(requestBuilderMock);
-        when(requestBuilderMock.get()).thenReturn(operationResultMock);
+        when(jerseyRequestMock.setAccept("image/jpeg")).thenReturn(jerseyRequestMock);
+        when(jerseyRequestMock.get()).thenReturn(operationResultMock);
         SingleThumbnailAdapter thumbnailAdapter = new SingleThumbnailAdapter(sessionStorageMock);
         thumbnailAdapter.report("/public/Samples/Reports/07g.RevenueDetailReport");
 
@@ -128,7 +144,7 @@ public void should_return_proper_operation_result() {
         assertNotNull(retrieved);
         assertSame(retrieved, operationResultMock);
         verify(jerseyRequestMock).setAccept(eq("image/jpeg"));
-        verify(requestBuilderMock).get();
+        verify(jerseyRequestMock).get();
         verifyStatic(times(1));
         buildRequest(
                 eq(sessionStorageMock),