Upload APIs rename call should return metadata and context

Author: YomesInc

cloudinary-core/src/main/java/com/cloudinary/Uploader.java

@@ -247,6 +247,8 @@ public Map rename(String fromPublicId, String toPublicId, Map options) throws IO
         params.put("to_public_id", toPublicId);
         params.put("invalidate", ObjectUtils.asBoolean(options.get("invalidate"), false).toString());
         params.put("to_type", options.get("to_type"));
+        params.put("context", ObjectUtils.asBoolean(options.get("context"), false).toString());
+        params.put("metadata", ObjectUtils.asBoolean(options.get("metadata"), false).toString());
         return callApi("rename", params, options, null);
     }
 

cloudinary-test-common/src/main/java/com/cloudinary/test/AbstractStructuredMetadataTest.java

@@ -302,17 +302,12 @@ private SetMetadataField createSetField(String labelPrefix) {
     }
 
     private StringMetadataField newFieldInstance(String labelPrefix) throws Exception {
-        StringMetadataField field = new StringMetadataField();
         String label = labelPrefix + "_" + SUFFIX;
-        field.setLabel(label);
-        field.setMandatory(true);
-        field.setValidation(new MetadataValidation.StringLength(3, 9));
-        field.setDefaultValue("val_test");
-        return field;
+        return MetadataTestHelper.newFieldInstance(label);
     }
 
     private ApiResponse addFieldToAccount(MetadataField field) throws Exception {
-        ApiResponse apiResponse = api.addMetadataField(field);
+        ApiResponse apiResponse = MetadataTestHelper.addFieldToAccount(api, field);
         metadataFieldExternalIds.add(apiResponse.get("external_id").toString());
         return apiResponse;
     }

cloudinary-test-common/src/main/java/com/cloudinary/test/AbstractUploaderTest.java

@@ -219,6 +219,31 @@ public void testRename() throws Exception {
         assertEquals(cloudinary.api().resource(publicId2, ObjectUtils.emptyMap()).get("format"), "ico");
     }
 
+    @Test
+    public void testRenameShouldReturnContext() throws Exception {
+        Map result = cloudinary.uploader().upload(SRC_TEST_IMAGE, asMap("tags", Arrays.asList(SDK_TEST_TAG, UPLOADER_TAG), "context", asMap("foo", "boo")));
+
+        String publicId = result.get("public_id").toString();
+        String publicId2 = "folder/" + publicId + "2";
+        Map renameResult = cloudinary.uploader().rename(publicId, publicId2, asMap("context", true));
+        assertNotNull(renameResult.get("context"));
+    }
+
+    @Test
+    public void testRenameShouldReturnMetadata() throws Exception {
+        String label = "test" + SUFFIX;
+        StringMetadataField f = MetadataTestHelper.newFieldInstance(label);
+        Map fieldResult = MetadataTestHelper.addFieldToAccount(cloudinary.api(), f);
+        String fieldId = fieldResult.get("external_id").toString();
+        Map<String, Object> metadata = Collections.<String, Object>singletonMap(fieldId, "123456");
+        Map result = cloudinary.uploader().upload(SRC_TEST_IMAGE, asMap("tags", Arrays.asList(SDK_TEST_TAG, UPLOADER_TAG), "metadata", metadata));
+
+        String publicId = result.get("public_id").toString();
+        String publicId2 = "folder/" + publicId + "2";
+        Map renameResult = cloudinary.uploader().rename(publicId, publicId2, asMap("metadata", true));
+        assertNotNull(renameResult.get("metadata"));
+    }
+
     @Test
     public void testUniqueFilename() throws Exception {
         Map result = cloudinary.uploader().upload(SRC_TEST_IMAGE, asMap("use_filename", true, "tags", Arrays.asList(SDK_TEST_TAG, UPLOADER_TAG)));

cloudinary-test-common/src/main/java/com/cloudinary/test/MetadataTestHelper.java

@@ -0,0 +1,24 @@
+package com.cloudinary.test;
+
+import com.cloudinary.Api;
+import com.cloudinary.api.ApiResponse;
+import com.cloudinary.metadata.MetadataField;
+import com.cloudinary.metadata.MetadataValidation;
+import com.cloudinary.metadata.StringMetadataField;
+
+public class MetadataTestHelper {
+    public static StringMetadataField newFieldInstance(String label) throws Exception {
+        StringMetadataField field = new StringMetadataField();
+        field.setLabel(label);
+        field.setMandatory(true);
+        field.setValidation(new MetadataValidation.StringLength(3, 9));
+        field.setDefaultValue("val_test");
+        return field;
+    }
+
+    public static ApiResponse addFieldToAccount(Api api, MetadataField field) throws Exception {
+        ApiResponse apiResponse = api.addMetadataField(field);
+        return apiResponse;
+    }
+}
+