-
Notifications
You must be signed in to change notification settings - Fork 18
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
chore: add end-to-end test for ApiGatewayV2 #644
base: main
Are you sure you want to change the base?
Conversation
tests/end-to-end.rs
Outdated
@@ -84,6 +84,7 @@ test_case!(cloudwatch, "CloudwatchStack", &["golang"]); | |||
test_case!(ecs, "EcsStack", &["java", "golang"]); | |||
test_case!(ec2, "Ec2Stack", &["java", "golang"]); | |||
test_case!(efs, "EfsStack", &["java", "golang"]); | |||
test_case!(apigatewayv2, "ApiGatewayV2Stack", &["csharp", "golang", "python"]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm concerned about not synthesizing python and csharp. If these aren't working you may have uncovered a bug. Could you remove these and we can debug the failing tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is going in the right direction! See my comment about python and csharp.
Codecov Report
Additional details and impacted files
@@ Coverage Diff @@
## main #644 +/- ##
=====================================
Coverage 82.1% 82.1%
=====================================
Files 27 27
Lines 5097 5097
Branches 5097 5097
=====================================
Hits 4186 4186
Misses 741 741
Partials 170 170 Continue to review full report in Codecov by Sentry.
|
36431a5
to
49e93a7
Compare
No description provided.