Skip to content

Commit

Permalink
Actually fix CRT only tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jonathan343 committed Oct 8, 2024
1 parent 2aed29c commit ccbc9c0
Showing 1 changed file with 40 additions and 36 deletions.
76 changes: 40 additions & 36 deletions tests/functional/test_httpchecksum.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,23 +175,25 @@ def _request_checksum_calculation_cases():
),
]
if HAS_CRT:
cases.append(
(
"CRC32C",
request_payload,
{
"x-amz-request-algorithm": "CRC32C",
"x-amz-checksum-crc32c": "crUfeA==",
},
),
(
"CRC64NVME",
request_payload,
{
"x-amz-request-algorithm": "CRC64NVME",
"x-amz-checksum-crc64nvme": "OOJZ0D8xKts=",
},
),
cases.extend(
[
(
"CRC32C",
request_payload,
{
"x-amz-request-algorithm": "CRC32C",
"x-amz-checksum-crc32c": "crUfeA==",
},
),
(
"CRC64NVME",
request_payload,
{
"x-amz-request-algorithm": "CRC64NVME",
"x-amz-checksum-crc64nvme": "OOJZ0D8xKts=",
},
),
]
)
return cases

Expand Down Expand Up @@ -262,25 +264,27 @@ def _streaming_request_checksum_calculation_cases():
),
]
if HAS_CRT:
cases.append(
(
"CRC32C",
request_payload,
{
"content-encoding": "aws-chunked",
"x-amz-trailer": "x-amz-checksum-crc32c",
},
{"x-amz-checksum-crc32c": "crUfeA=="},
),
(
"CRC64NVME",
request_payload,
{
"content-encoding": "aws-chunked",
"x-amz-trailer": "x-amz-checksum-crc64nvme",
},
{"x-amz-checksum-crc64nvme": "OOJZ0D8xKts="},
),
cases.extend(
[
(
"CRC32C",
request_payload,
{
"content-encoding": "aws-chunked",
"x-amz-trailer": "x-amz-checksum-crc32c",
},
{"x-amz-checksum-crc32c": "crUfeA=="},
),
(
"CRC64NVME",
request_payload,
{
"content-encoding": "aws-chunked",
"x-amz-trailer": "x-amz-checksum-crc64nvme",
},
{"x-amz-checksum-crc64nvme": "OOJZ0D8xKts="},
),
]
)
return cases

Expand Down

0 comments on commit ccbc9c0

Please sign in to comment.