Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: move flexible checksums integration tests #6752

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

trivikr
Copy link
Member

@trivikr trivikr commented Dec 19, 2024

Issue

Refs: #6750 (review)

Description

Moves flexible checksums integration tests inside middleware-flexible-checksums

Testing

CI

Verified that integration tests are successful

$ middleware-flexible-checksums> yarn test:integration

 RUN  v0.34.6 /local/home/trivikr/workspace/aws-sdk-js-v3/packages/middleware-flexible-checksums

 ✓ src/middleware-flexible-checksums.integ.spec.ts (29)
   ✓ middleware-flexible-checksums (29)
     ✓ S3 (29)
       ✓ putObject (12)
         ✓ sets x-amz-checksum-crc32="AAAAAA=="" for checksum="CRC32"
         ✓ sets x-amz-checksum-crc32="NSRBwg=="" for checksum="CRC32"
         ✓ sets x-amz-checksum-crc32="i9aeUg=="" for checksum="CRC32"
         ✓ sets x-amz-checksum-crc32c="AAAAAA=="" for checksum="CRC32C"
         ✓ sets x-amz-checksum-crc32c="Nks/tw=="" for checksum="CRC32C"
         ✓ sets x-amz-checksum-crc32c="crUfeA=="" for checksum="CRC32C"
         ✓ sets x-amz-checksum-sha1="2jmj7l5rSw0yVb/vlWAYkK/YBwk="" for checksum="SHA1"
         ✓ sets x-amz-checksum-sha1="qZk+NkcGgWq6PiVxeFDCbJzQ2J0="" for checksum="SHA1"
         ✓ sets x-amz-checksum-sha1="e1AsOh9IyGCa4hLN+2Od7jlnP14="" for checksum="SHA1"
         ✓ sets x-amz-checksum-sha256="47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU="" for checksum="SHA256"
         ✓ sets x-amz-checksum-sha256="ungWv48Bz+pBQUDeXa4iI7ADYaOWF3qctBD/YfIAFa0="" for checksum="SHA256"
         ✓ sets x-amz-checksum-sha256="ZOyIygCyaOW6GjVnihtTFtIS9PNmskdyMlNKiuyjfzw="" for checksum="SHA256"
       ✓ getObject (12)
         ✓ validates x-amz-checksum-crc32="AAAAAA=="" set for checksum="CRC32"
         ✓ validates x-amz-checksum-crc32="NSRBwg=="" set for checksum="CRC32"
         ✓ validates x-amz-checksum-crc32="i9aeUg=="" set for checksum="CRC32"
         ✓ validates x-amz-checksum-crc32c="AAAAAA=="" set for checksum="CRC32C"
         ✓ validates x-amz-checksum-crc32c="Nks/tw=="" set for checksum="CRC32C"
         ✓ validates x-amz-checksum-crc32c="crUfeA=="" set for checksum="CRC32C"
         ✓ validates x-amz-checksum-sha1="2jmj7l5rSw0yVb/vlWAYkK/YBwk="" set for checksum="SHA1"
         ✓ validates x-amz-checksum-sha1="qZk+NkcGgWq6PiVxeFDCbJzQ2J0="" set for checksum="SHA1"
         ✓ validates x-amz-checksum-sha1="e1AsOh9IyGCa4hLN+2Od7jlnP14="" set for checksum="SHA1"
         ✓ validates x-amz-checksum-sha256="47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU="" set for checksum="SHA256"
         ✓ validates x-amz-checksum-sha256="ungWv48Bz+pBQUDeXa4iI7ADYaOWF3qctBD/YfIAFa0="" set for checksum="SHA256"
         ✓ validates x-amz-checksum-sha256="ZOyIygCyaOW6GjVnihtTFtIS9PNmskdyMlNKiuyjfzw="" set for checksum="SHA256"
       ✓ should not set binary file content length
       ✓ features (4)
         ✓ should feature-detect checksum SHA256=Y
         ✓ should feature-detect checksum SHA1=X
         ✓ should feature-detect checksum CRC32=U
         ✓ should feature-detect checksum CRC32C=V

 Test Files  1 passed (1)
      Tests  29 passed (29)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr trivikr force-pushed the test-integ-flexible-checksums branch from 39647f7 to ae65cbe Compare December 19, 2024 23:25
@trivikr trivikr force-pushed the test-integ-flexible-checksums branch from ae65cbe to 5108467 Compare December 19, 2024 23:40
@trivikr trivikr marked this pull request as ready for review December 19, 2024 23:42
@trivikr trivikr requested a review from a team as a code owner December 19, 2024 23:42
@trivikr trivikr merged commit 3317a92 into aws:main Dec 20, 2024
4 checks passed
@trivikr trivikr deleted the test-integ-flexible-checksums branch December 20, 2024 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants