-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add integration tests for CloudFront construct library #208
Conversation
@@ -7,7 +7,7 @@ | |||
"@pulumi/aws": "^6.0.0", | |||
"@pulumi/aws-native": "^1.0.0", | |||
"@pulumi/pulumi": "^3.0.0", | |||
"aws-cdk-lib": "2.149.0", | |||
"aws-cdk-lib": "2.156.0", |
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.
Updating the dev dependency for a new CloudFront feature.
26a72d0
to
55a41c8
Compare
func TestCloudFront(t *testing.T) { | ||
test := getJSBaseOptions(t). | ||
With(integration.ProgramTestOptions{ | ||
Dir: filepath.Join(getCwd(t), "cloudfront"), |
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.
Would it be possible to add a quick assert, even if it's hello-world style, that some output of the stack got returned correctly? Thanks!
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.
LGTM, but tests don't seem to be happy
db3bd00
to
fb94cc3
Compare
re #183