Skip to content

add S3 presigned POST test cases #12042

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bentsku
Copy link
Contributor

@bentsku bentsku commented Dec 16, 2024

Motivation

This PR adds tests to validate the scenario reported with #5269 (comment)

For now, we're unable to reproduce the user's issues, but I'll update the PR if we get more information.

Changes

  • add test cases validating eq bad conditions (missing or invalid field)

@bentsku bentsku added aws:s3 Amazon Simple Storage Service semver: patch Non-breaking changes which can be included in patch releases labels Dec 16, 2024
@bentsku bentsku added this to the 4.1 milestone Dec 16, 2024
@bentsku bentsku self-assigned this Dec 16, 2024
Copy link

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   4m 14s ⏱️
442 tests 392 ✅  50 💤 0 ❌
884 runs  784 ✅ 100 💤 0 ❌

Results for commit 0f86a12.

Copy link

LocalStack Community integration with Pro

    2 files      2 suites   58m 44s ⏱️
1 657 tests 1 497 ✅ 160 💤 0 ❌
1 659 runs  1 497 ✅ 162 💤 0 ❌

Results for commit 0f86a12.

@bentsku bentsku modified the milestones: 4.1, 4.2 Jan 29, 2025
@bentsku bentsku modified the milestones: 4.2, Playground Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws:s3 Amazon Simple Storage Service semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant