Skip to content

SNS: v2 enable skipped tests #13544

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.

Sign up for GitHub

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

Jump to bottom
Draft
baermat wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

SNS: v2 enable skipped tests #13544

baermat wants to merge 1 commit into from

Conversation

@baermat
Copy link
Member

@baermat baermat commented Dec 18, 2025 •
edited
Loading

Motivation

This PR enables all but one remaining tests that were skipped for v2. The remaining test requires some clarification

Changes

Tests

Related

@baermat baermat added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 18, 2025
@github-actions
Copy link

github-actions bot commented Dec 18, 2025

Test Results - Alternative Providers

208 tests   165 ✅  2m 13s ⏱️
  1 suites   43 💤
  1 files      0 ❌

Results for commit d67db37.

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 3s ⏱️ +2s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit d67db37. ± Comparison against base commit b613fb9.

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   1h 29m 57s ⏱️
3 145 tests 2 975 ✅ 170 💤 0 ❌
3 154 runs  2 978 ✅ 176 💤 0 ❌

Results for commit d67db37.

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

LocalStack Community integration with Pro

    2 files  ±    0      2 suites  ±0   1h 2m 41s ⏱️ - 52m 24s
3 121 tests  - 2 032  2 948 ✅  - 1 808  173 💤  - 224  0 ❌ ±0 
3 126 runs   - 2 029  2 951 ✅  - 1 805  175 💤  - 224  0 ❌ ±0 

Results for commit d67db37. ± Comparison against base commit b613fb9.

This pull request removes 2032 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bentsku bentsku Awaiting requested review from bentsku bentsku will be requested when the pull request is marked ready for review bentsku is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@baermat