ivandika3 commented on code in PR #6607:
URL: https://github.com/apache/ozone/pull/6607#discussion_r1592589144


##########
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectPut.java:
##########
@@ -193,6 +204,99 @@ void testPutObjectContentLengthForStreaming()
     assertEquals(15, getKeyDataSize());
   }
 
+  @Test
+  public void testPutObjectWithTags() throws IOException, OS3Exception {

Review Comment:
   Thank you for the review. Updated.



##########
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectPut.java:
##########
@@ -438,6 +542,92 @@ public void 
testCopyObjectMessageDigestResetDuringException() throws IOException
     }
   }
 
+  @Test
+  public void testCopyObjectWithTags() throws IOException, OS3Exception {

Review Comment:
   Updated.



##########
hadoop-ozone/dist/src/main/smoketest/s3/objectputget.robot:
##########
@@ -189,6 +196,16 @@ Create file with user defined metadata size larger than 2 
KB
                                 Should contain                        
${result}   MetadataTooLarge
                                 Should not contain                    
${result}   custom-key1: ${custom_metadata_value}
 
+Create files invalid tags
+    ${result} =                 Execute AWSS3APICli and checkrc       
put-object --bucket ${BUCKET} --key ${PREFIX}/putobject/custom-metadata/key2 
--body /tmp/testfile2 --tagging="tag-key1=tag-value1&tag-key1=tag-value2"    255
+                                Should contain                        
${result}   InvalidTag
+    ${long_tag_key} =           Execute                               printf 
'v%.0s' {1..129}
+    ${result} =                 Execute AWSS3APICli and checkrc       
put-object --bucket ${BUCKET} --key ${PREFIX}/putobject/custom-metadata/key2 
--body /tmp/testfile2 --tagging="${long_tag_key}=tag-value1"    255
+                                Should contain                        
${result}   InvalidTag
+    ${long_tag_value} =         Execute                               printf 
'v%.0s' {1..257}
+    ${result} =                 Execute AWSS3APICli and checkrc       
put-object --bucket ${BUCKET} --key ${PREFIX}/putobject/custom-metadata/key2 
--body /tmp/testfile2 --tagging="tag-key1=${long_tag_value}"    255
+                                Should contain                        
${result}   InvalidTag
+

Review Comment:
   Updated.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to