8000 Apigw add tests fot vtl bracket assignment by cloutierMat · Pull Request #12727 · localstack/localstack · GitHub
[go: up one dir, main page]

Skip to content

Apigw add tests fot vtl bracket assignment #12727

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

Merged
merged 4 commits into from
Jun 6, 2025
Merged

Conversation

cloutierMat
Copy link
Contributor
@cloutierMat cloutierMat commented Jun 6, 2025

Motivation

Follow up to localstack/airspeed#22 and #12621. Adding tests to validate bracket assignment in vtl template.

@cloutierMat cloutierMat self-assigned this Jun 6, 2025
@cloutierMat cloutierMat added aws:apigateway Amazon API Gateway semver: patch Non-breaking changes which can be included in patch releases labels Jun 6, 2025
Copy link
github-actions bot commented Jun 6, 2025

Test Results - Preflight, Unit

21 595 tests  ±0   19 940 ✅ ±0   6m 12s ⏱️ -23s
     1 suites ±0    1 655 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 10e14bc. ± Comparison against base commit ed6c94a.

♻️ This comment has been updated with latest results.

Copy link
github-actions bot commented Jun 6, 2025

LocalStack Community integration with Pro

    2 files      2 suites   18m 24s ⏱️
1 135 tests 1 070 ✅ 65 💤 0 ❌
1 137 runs  1 070 ✅ 67 💤 0 ❌

Results for commit 10e14bc.

♻️ This comment has been updated with latest results.

@cloutierMat cloutierMat changed the title Apigw vtl bracket Apigw add tests fot vtl bracket assignment Jun 6, 2025
Copy link
github-actions bot commented Jun 6, 2025

Test Results (amd64) - Acceptance

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

Results for commit 10e14bc. ± Comparison against base commit ed6c94a.

♻️ This comment has been updated with latest results.

Copy link
github-actions bot commented Jun 6, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   36m 19s ⏱️
1 159 tests 1 094 ✅ 65 💤 0 ❌
1 165 runs  1 094 ✅ 71 💤 0 ❌

Results for commit 10e14bc.

Copy link
github-actions bot commented Jun 6, 2025

Test Results - Alternative Providers

987 tests   584 ✅  23m 5s ⏱️
  4 suites  403 💤
  4 files      0 ❌

Results for commit 10e14bc.

@cloutierMat cloutierMat marked this pull request as ready for review June 6, 2025 21:52
@cloutierMat cloutierMat requested a review from bentsku as a code owner June 6, 2025 21:52
Copy link
Contributor
@bentsku bentsku left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for following up! 🚀

@cloutierMat cloutierMat merged commit 6487dbf into master Jun 6, 2025
56 checks passed
@cloutierMat cloutierMat deleted the apigw-vtl-bracket branch June 6, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws:apigateway Amazon API Gateway 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.

2 participants
0