fix: remove CloudFormation property renames (#973)

The construct id parameter used to be a property called `name` in the 
props object, and so properties called `name` were renamed to `xxxName`
(where `xxx` is the name of the resource). Since we now pass names as
a constructor argument, we can get rid of the properly rename mechanism.

BREAKING CHANGE: if you use L1s, you may need to change some
`XxxName` properties back into `Name`. These will match the CloudFormation
property names.

Fixes #852.

Author: rix0rrr

packages/@aws-cdk/aws-apigateway/lib/restapi.ts

@@ -176,7 +176,7 @@ export class RestApi extends RestApiRef implements cdk.IDependable {
     super(parent, id);
 
     const resource = new cloudformation.RestApiResource(this, 'Resource', {
-      restApiName: props.restApiName || id,
+      name: props.restApiName || id,
       description: props.description,
       policy: props.policy,
       failOnWarnings: props.failOnWarnings,

packages/@aws-cdk/aws-codebuild/lib/project.ts

@@ -522,7 +522,7 @@ export class Project extends ProjectRef {
       encryptionKey: props.encryptionKey && props.encryptionKey.keyArn,
       badgeEnabled: props.badge,
       cache,
-      projectName: props.projectName,
+      name: props.projectName,
     });
 
     this.projectArn = resource.projectArn;

packages/@aws-cdk/aws-codepipeline/lib/pipeline.ts

@@ -97,7 +97,7 @@ export class Pipeline extends cdk.Construct implements events.IEventRuleTarget {
       stages: new cdk.Token(() => this.renderStages()) as any,
       roleArn: this.role.roleArn,
       restartExecutionOnUpdate: props && props.restartExecutionOnUpdate,
-      pipelineName: props && props.pipelineName,
+      name: props && props.pipelineName,
     });
 
     // this will produce a DependsOn for both the role and the policy resources.

packages/@aws-cdk/aws-events/lib/rule.ts

@@ -74,7 +74,7 @@ export class EventRule extends EventRuleRef {
     super(parent, name);
 
     const resource = new cloudformation.RuleResource(this, 'Resource', {
-      ruleName: props.ruleName,
+      name: props.ruleName,
       description: props.description,
       state: props.enabled == null ? 'ENABLED' : (props.enabled ? 'ENABLED' : 'DISABLED'),
       scheduleExpression: new Token(() => this.scheduleExpression),

packages/@aws-cdk/aws-kinesis/lib/stream.ts

@@ -297,7 +297,7 @@ export class Stream extends StreamRef {
     const { streamEncryption, encryptionKey } = this.parseEncryption(props);
 
     this.stream = new cloudformation.StreamResource(this, "Resource", {
-      streamName: props.streamName,
+      name: props.streamName,
       retentionPeriodHours,
       shardCount,
       streamEncryption

packages/@aws-cdk/aws-lambda/lib/alias.ts

@@ -86,7 +86,7 @@ export class Alias extends FunctionRef {
     this.underlyingLambda = props.version.lambda;
 
     const alias = new cloudformation.AliasResource(this, 'Resource', {
-      aliasName: props.aliasName,
+      name: props.aliasName,
       description: props.description,
       functionName: this.underlyingLambda.functionName,
       functionVersion: props.version.functionVersion,

packages/@aws-cdk/aws-route53/lib/hosted-zone.ts

@@ -125,9 +125,9 @@ function toVpcProperty(vpc: ec2.VpcNetworkRef): cloudformation.HostedZoneResourc
 }
 
 function determineHostedZoneProps(props: PublicHostedZoneProps) {
-  const hostedZoneName = props.zoneName + '.';
+  const name = props.zoneName + '.';
   const hostedZoneConfig = props.comment ? { comment: props.comment } : undefined;
   const queryLoggingConfig = props.queryLogsLogGroupArn ? { cloudWatchLogsLogGroupArn: props.queryLogsLogGroupArn } : undefined;
 
-  return { hostedZoneName, hostedZoneConfig, queryLoggingConfig };
+  return { name, hostedZoneConfig, queryLoggingConfig };
 }

packages/@aws-cdk/aws-route53/lib/records/txt.ts

@@ -23,7 +23,7 @@ export class TXTRecord extends Construct {
 
     new cloudformation.RecordSetResource(this, 'Resource', {
       hostedZoneId: parent.hostedZoneId,
-      recordSetName: determineFullyQualifiedDomainName(props.recordName, parent),
+      name: determineFullyQualifiedDomainName(props.recordName, parent),
       type: 'TXT',
       resourceRecords: [recordValue],
       ttl: ttl.toString()

packages/@aws-cdk/aws-route53/lib/records/zone-delegation.ts

@@ -40,7 +40,7 @@ export class ZoneDelegationRecord extends Construct {
 
     new cloudformation.RecordSetResource(this, 'Resource', {
       hostedZoneId: parent.hostedZoneId,
-      recordSetName: determineFullyQualifiedDomainName(props.delegatedZoneName, parent),
+      name: determineFullyQualifiedDomainName(props.delegatedZoneName, parent),
       type: 'NS',
       ttl: ttl.toString(),
       comment: props.comment,

packages/@aws-cdk/aws-ssm/test/test.ssm-document.ts

@@ -0,0 +1,22 @@
+import { expect, haveResource } from '@aws-cdk/assert';
+import cdk = require('@aws-cdk/cdk');
+import { Test } from 'nodeunit';
+import ssm = require('../lib');
+
+export = {
+  'association name is rendered properly in L1 construct'(test: Test) {
+    // GIVEN
+    const stack = new cdk.Stack();
+
+    // WHEN
+    new ssm.cloudformation.AssociationResource(stack, 'Assoc', {
+      name: 'document',
+    });
+
+    // THEN
+    expect(stack).to(haveResource('AWS::SSM::Association', {
+      Name: 'document',
+    }));
+    test.done();
+  }
+};