Skip to content

Commit

Permalink
Comment the test with a todo
Browse files Browse the repository at this point in the history
  • Loading branch information
waahm7 committed Dec 4, 2023
1 parent 1ba0e47 commit 2d1dc27
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions tests/s3_data_plane_tests.c
Original file line number Diff line number Diff line change
Expand Up @@ -3762,6 +3762,19 @@ static int s_test_s3_download_multi_part_file_with_checksum(struct aws_allocator
client = aws_s3_client_release(client);
tester.bound_to_client = false;

/*** GET FILE with part_size > fileSize ***/
/* TODO: Enable this test once the checksum issue is resolved. Currently, when the S3 GetObject API is called with
* the range 0-contentLength, it returns a checksum of checksums without the -numParts portion. This leads to a
* checksum mismatch error, as it is incorrectly validated as a part checksum. */
/*
client_options.part_size = MB_TO_BYTES(20);
ASSERT_SUCCESS(aws_s3_tester_client_new(&tester, &client_options, &client));
get_options.client = client;
ASSERT_SUCCESS(aws_s3_tester_send_meta_request_with_options(&tester, &get_options, NULL));
client = aws_s3_client_release(client);
tester.bound_to_client = false;
*/
aws_byte_buf_clean_up(&path_buf);
aws_s3_tester_clean_up(&tester);

Expand Down

0 comments on commit 2d1dc27

Please sign in to comment.