Skip to content

Commit

Permalink
Merge branch 'master' into contributing-guide
Browse files Browse the repository at this point in the history
  • Loading branch information
SoManyHs committed May 14, 2021
2 parents 3d700d6 + 348e11e commit 40b3b3a
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 2 additions & 0 deletions packages/@aws-cdk/aws-ecs/lib/base/base-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -415,6 +415,8 @@ export abstract class BaseService extends Resource
if (props.cloudMapOptions) {
this.enableCloudMap(props.cloudMapOptions);
}

this.node.defaultChild = this.resource;
}

/**
Expand Down
4 changes: 3 additions & 1 deletion packages/@aws-cdk/aws-ecs/test/ec2/ec2-service.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ nodeunitShim({
memoryLimitMiB: 512,
});

new ecs.Ec2Service(stack, 'Ec2Service', {
const service = new ecs.Ec2Service(stack, 'Ec2Service', {
cluster,
taskDefinition,
});
Expand All @@ -47,6 +47,8 @@ nodeunitShim({
EnableECSManagedTags: false,
}));

test.notEqual(service.node.defaultChild, undefined);

test.done();
},

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ nodeunitShim({
image: ecs.ContainerImage.fromRegistry('amazon/amazon-ecs-sample'),
});

new ecs.FargateService(stack, 'FargateService', {
const service = new ecs.FargateService(stack, 'FargateService', {
cluster,
taskDefinition,
});
Expand Down Expand Up @@ -79,6 +79,8 @@ nodeunitShim({
},
}));

test.notEqual(service.node.defaultChild, undefined);

test.done();
},

Expand Down

0 comments on commit 40b3b3a

Please sign in to comment.