Skip to content

Commit 39a6f80

Browse files
authored
chore(test): use manifest for env integration test (#3795)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.
1 parent 5e5fff3 commit 39a6f80

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

internal/pkg/deploy/cloudformation/cloudformation_integration_test.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ import (
2828
"github.com/aws/copilot-cli/internal/pkg/deploy/cloudformation"
2929
"github.com/aws/copilot-cli/internal/pkg/deploy/cloudformation/stack"
3030
"github.com/aws/copilot-cli/internal/pkg/deploy/upload/customresource"
31+
"github.com/aws/copilot-cli/internal/pkg/manifest"
3132
"github.com/aws/copilot-cli/internal/pkg/template"
3233
"github.com/stretchr/testify/require"
3334
)
@@ -492,6 +493,12 @@ func Test_Environment_Deployment_Integration(t *testing.T) {
492493
environmentToDeploy.CustomResourcesURLs = urls
493494
environmentToDeploy.ArtifactBucketKeyARN = "arn:aws:kms:us-west-2:111122223333:key/1234abcd-12ab-34cd-56ef-1234567890ab"
494495
environmentToDeploy.ArtifactBucketARN = fmt.Sprintf("arn:aws:s3:::%s", bucketName)
496+
environmentToDeploy.Mft = &manifest.Environment{
497+
Workload: manifest.Workload{
498+
Name: aws.String(envName),
499+
Type: aws.String("Environment"),
500+
},
501+
}
495502

496503
// Deploy the environment and wait for it to be complete.
497504
oldParams, err := deployer.EnvironmentParameters(environmentToDeploy.App.Name, environmentToDeploy.Name)
@@ -513,7 +520,7 @@ func Test_Environment_Deployment_Integration(t *testing.T) {
513520
deployedStack := output.Stacks[0]
514521
expectedResultsForKey := map[string]func(*awsCF.Output){
515522
"EnabledFeatures": func(output *awsCF.Output) {
516-
require.Equal(t, ",,,", aws.StringValue(output.OutputValue), "no env features enabled by default")
523+
require.Equal(t, ",,,,", aws.StringValue(output.OutputValue), "no env features enabled by default")
517524
},
518525
"EnvironmentManagerRoleARN": func(output *awsCF.Output) {
519526
require.Equal(t,

0 commit comments

Comments
 (0)