[ 
https://issues.apache.org/jira/browse/HDDS-12978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ivan Andika updated HDDS-12978:
-------------------------------
    Description: 
TestMultipartObjectGet integration test the ranged GET for multipart upload 
part. Currently it uses a ObjectEndpoint which still needs to mock HttpHeaders 
and ContainerRequestContext (although integration test should minimize mocking 
as much as possible). Additionally, this integration test only contains one 
test. 

Seems HDDS-11699 has already added S3 SDK integration test  for multipart 
upload range get (AbstractS3SDKV1Tests#, so we might be able to remove the test 
entirely. We need to check the test logic first and migrate any test logic not 
already covered by the S3 SDK integration tests.

  was:
TestMultipartObjectGet integration test the ranged GET for multipart upload 
part. Currently it uses a ObjectEndpoint which still needs to mock HttpHeaders 
and ContainerRequestContext (although integration test should minimize mocking 
as much as possible). Additionally, this integration test only contains one 
test. 

We can move the test logic to the current S3 SDK tests (AbstractS3SDKV1Tests 
and AbstractS3SDKV2Tests). If the test logic already exists in the integration 
tests, we can simply remove this class.


> Remove TestMultipartObjectGet
> -----------------------------
>
>                 Key: HDDS-12978
>                 URL: https://issues.apache.org/jira/browse/HDDS-12978
>             Project: Apache Ozone
>          Issue Type: Sub-task
>            Reporter: Ivan Andika
>            Priority: Major
>
> TestMultipartObjectGet integration test the ranged GET for multipart upload 
> part. Currently it uses a ObjectEndpoint which still needs to mock 
> HttpHeaders and ContainerRequestContext (although integration test should 
> minimize mocking as much as possible). Additionally, this integration test 
> only contains one test. 
> Seems HDDS-11699 has already added S3 SDK integration test  for multipart 
> upload range get (AbstractS3SDKV1Tests#, so we might be able to remove the 
> test entirely. We need to check the test logic first and migrate any test 
> logic not already covered by the S3 SDK integration tests.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to