Skip to content

Conversation

pinzon
Copy link
Member

@pinzon pinzon commented Sep 25, 2025

Motivation

  • This PR fixes an issue in the pipeline caused by perform_dynamic_reference_lookup returning a number in certain occasions.

Changes

  • convert returning type to string as the type annotation suggests

Copy link

Test Results - Preflight, Unit

22 277 tests  ±0   20 536 ✅ ±0   16m 20s ⏱️ +9s
     1 suites ±0    1 741 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 3fa534e. ± Comparison against base commit 339ad5c.

Copy link

Test Results (amd64) - Acceptance

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

Results for commit 3fa534e. ± Comparison against base commit 339ad5c.

Copy link

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   35m 20s ⏱️ - 1h 26m 3s
573 tests  - 4 208  458 ✅  - 3 987  115 💤  - 221  0 ❌ ±0 
575 runs   - 4 208  458 ✅  - 3 987  117 💤  - 221  0 ❌ ±0 

Results for commit 3fa534e. ± Comparison against base commit 339ad5c.

This pull request removes 4208 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]
…

Copy link

Test Results (amd64) - Integration, Bootstrap

  5 files    5 suites   47m 34s ⏱️
597 tests 483 ✅ 114 💤 0 ❌
603 runs  483 ✅ 120 💤 0 ❌

Results for commit 3fa534e.

@pinzon pinzon marked this pull request as ready for review September 25, 2025 19:03
@pinzon pinzon requested a review from cloutierMat September 25, 2025 19:03
Copy link
Contributor

@cloutierMat cloutierMat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. Thanks for the jumping in on this!

Copy link

Test Results - Alternative Providers

572 tests   329 ✅  25m 53s ⏱️
  1 suites  243 💤
  1 files      0 ❌

Results for commit 3fa534e.

@pinzon pinzon merged commit f93ee95 into main Sep 25, 2025
43 checks passed
@pinzon pinzon deleted the cp/cfn/hot-fix/dynamic-ref-typing branch September 25, 2025 19:14
@alexrashed alexrashed added this to the 4.9 milestone Sep 30, 2025
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.

3 participants