refactor(codepipeline): change the API of cross-region replication Bu…

…ckets.

Users are now supposed to pass instances of IBucket instead of just Bucket names.

Changed the name and type of the Pipeline.crossRegionScaffold property.

BREAKING CHANGE: the Pipeline construction property crossRegionReplicationBuckets now takes values of type IBucket instead of string.
* The property Pipeline.crossRegionScaffoldStacks has been renamed to crossRegionSupport,
and its type changed from CrossRegionScaffoldStack to CrossRegionSupport.

packages/@aws-cdk/aws-codepipeline-actions/test/test.pipeline.ts

@@ -1,4 +1,4 @@
-import { expect, haveResource, haveResourceLike, not, SynthUtils } from '@aws-cdk/assert';
+import { countResources, expect, haveResource, haveResourceLike, not, SynthUtils } from '@aws-cdk/assert';
 import codebuild = require('@aws-cdk/aws-codebuild');
 import codecommit = require('@aws-cdk/aws-codecommit');
 import codepipeline = require('@aws-cdk/aws-codepipeline');
@@ -596,19 +596,19 @@ export = {
           account: pipelineAccount,
         },
       });
-      const bucket = new s3.Bucket(stack, 'MyBucket');
       const pipeline = new codepipeline.Pipeline(stack, 'MyPipeline', {
         crossRegionReplicationBuckets: {
-          'us-west-1': 'sfo-replication-bucket',
+          'us-west-1': s3.Bucket.fromBucketName(stack, 'ImportedBucket', 'sfo-replication-bucket'),
         },
       });
 
+      const sourceBucket = new s3.Bucket(stack, 'MyBucket');
       const sourceOutput = new codepipeline.Artifact('SourceOutput');
       const sourceAction = new cpactions.S3SourceAction({
         actionName: 'BucketSource',
         bucketKey: '/some/key',
         output: sourceOutput,
-        bucket,
+        bucket: sourceBucket,
       });
       pipeline.addStage({
         stageName: 'Stage1',
@@ -648,19 +648,25 @@ export = {
             "Region": "us-east-1",
             "ArtifactStore": {
               "Type": "S3",
+              "Location": "cdk-cross-region-codepipeline-replication-bucket-685c6feea5fb",
             },
           },
           {
             "Region": "us-west-1",
             "ArtifactStore": {
-              "Location": "sfo-replication-bucket",
               "Type": "S3",
+              "Location": "sfo-replication-bucket",
             },
           },
           {
             "Region": "us-west-2",
             "ArtifactStore": {
               "Type": "S3",
+              "EncryptionKey": {
+                "Type": "KMS",
+                "Id": {
+                },
+              },
             },
           },
         ],
@@ -685,18 +691,90 @@ export = {
               },
             ],
           },
-        ]
+        ],
       }));
 
-      test.equal(pipeline.crossRegionScaffolding[pipelineRegion], undefined);
-      test.equal(pipeline.crossRegionScaffolding['us-west-1'], undefined);
+      test.equal(pipeline.crossRegionSupport[pipelineRegion], undefined);
+      test.equal(pipeline.crossRegionSupport['us-west-1'], undefined);
+
+      const usEast1Support = pipeline.crossRegionSupport['us-east-1'];
+      test.notEqual(usEast1Support, undefined);
+      test.equal(usEast1Support.stack.region, 'us-east-1');
+      test.equal(usEast1Support.stack.account, pipelineAccount);
+      test.ok(usEast1Support.stack.node.id.indexOf('us-east-1') !== -1,
+        `expected '${usEast1Support.stack.node.id}' to contain 'us-east-1'`);
+
+      test.done();
+    },
+
+    'allows specifying only one of artifactBucket and crossRegionReplicationBuckets'(test: Test) {
+      const stack = new Stack();
+
+      test.throws(() => {
+        new codepipeline.Pipeline(stack, 'Pipeline', {
+          artifactBucket: new s3.Bucket(stack, 'Bucket'),
+          crossRegionReplicationBuckets: {
+            // even an empty map should trigger this validation...
+          },
+        });
+      }, /Only one of artifactBucket and crossRegionReplicationBuckets can be specified!/);
+      test.done();
+    },
+
+    'does not create a new artifact Bucket if one was provided in the cross-region Buckets for the Pipeline region'(test: Test) {
+      const pipelineRegion = 'us-west-2';
+
+      const stack = new Stack(undefined, undefined, {
+        env: {
+          region: pipelineRegion,
+        },
+      });
+      const sourceOutput = new codepipeline.Artifact();
+      new codepipeline.Pipeline(stack, 'Pipeline', {
+        crossRegionReplicationBuckets: {
+          [pipelineRegion]: new s3.Bucket(stack, 'Bucket', {
+            bucketName: PhysicalName.of('my-pipeline-bucket'),
+          })
+        },
+        stages: [
+          {
+            stageName: 'Source',
+            actions: [
+              new cpactions.CodeCommitSourceAction({
+                actionName: 'Source',
+                output: sourceOutput,
+                repository: new codecommit.Repository(stack, 'Repo', { repositoryName: PhysicalName.of('Repo'), }),
+              }),
+            ],
+          },
+          {
+            stageName: 'Build',
+            actions: [
+              new cpactions.CodeBuildAction({
+                actionName: 'Build',
+                input: sourceOutput,
+                project: new codebuild.PipelineProject(stack, 'Project'),
+              }),
+            ],
+          },
+        ],
+      });
+
+      expect(stack).to(countResources('AWS::S3::Bucket', 1));
 
-      const usEast1ScaffoldStack = pipeline.crossRegionScaffolding['us-east-1'];
-      test.notEqual(usEast1ScaffoldStack, undefined);
-      test.equal(usEast1ScaffoldStack.region, 'us-east-1');
-      test.equal(usEast1ScaffoldStack.account, pipelineAccount);
-      test.ok(usEast1ScaffoldStack.node.id.indexOf('us-east-1') !== -1,
-        `expected '${usEast1ScaffoldStack.node.id}' to contain 'us-east-1'`);
+      expect(stack).to(haveResourceLike('AWS::CodePipeline::Pipeline', {
+        "ArtifactStores": [
+          {
+            "Region": pipelineRegion,
+            "ArtifactStore": {
+              "Type": "S3",
+              "Location": {
+                "Ref": "Bucket83908E77",
+              },
+            },
+          },
+        ],
+      }));
 
       test.done();
     },

packages/@aws-cdk/aws-codepipeline/lib/cross-region-scaffold-stack.ts → packages/@aws-cdk/aws-codepipeline/lib/cross-region-support-stack.ts

@@ -1,12 +1,12 @@
 import s3 = require('@aws-cdk/aws-s3');
 import cdk = require('@aws-cdk/cdk');
 import crypto = require('crypto');
-import { CrossRegionScaffolding } from './pipeline';
 
 /**
- * Construction properties for {@link CrossRegionScaffoldStack}.
+ * Construction properties for {@link CrossRegionSupportStack}.
+ * This interface is private to the aws-codepipeline package.
  */
-export interface CrossRegionScaffoldStackProps {
+export interface CrossRegionSupportStackProps {
   /**
    * The AWS region this Stack resides in.
    */
@@ -22,14 +22,15 @@ export interface CrossRegionScaffoldStackProps {
 
 /**
  * A Stack containing resources required for the cross-region CodePipeline functionality to work.
+ * This class is private to the aws-codepipeline package.
  */
-export class CrossRegionScaffoldStack extends CrossRegionScaffolding {
+export class CrossRegionSupportStack extends cdk.Stack {
   /**
    * The name of the S3 Bucket used for replicating the Pipeline's artifacts into the region.
    */
-  public readonly replicationBucketName: string;
+  public readonly replicationBucket: s3.IBucket;
 
-  constructor(scope: cdk.Construct, id: string, props: CrossRegionScaffoldStackProps) {
+  constructor(scope: cdk.Construct, id: string, props: CrossRegionSupportStackProps) {
     super(scope, id, {
       stackName: generateStackName(props),
       env: {
@@ -41,14 +42,13 @@ export class CrossRegionScaffoldStack extends CrossRegionScaffolding {
     const replicationBucketName = generateUniqueName('cdk-cross-region-codepipeline-replication-bucket-',
       props.region, props.account, false, 12);
 
-    new s3.Bucket(this, 'CrossRegionCodePipelineReplicationBucket', {
+    this.replicationBucket = new s3.Bucket(this, 'CrossRegionCodePipelineReplicationBucket', {
       bucketName: cdk.PhysicalName.of(replicationBucketName),
     });
-    this.replicationBucketName = replicationBucketName;
   }
 }
 
-function generateStackName(props: CrossRegionScaffoldStackProps): string {
+function generateStackName(props: CrossRegionSupportStackProps): string {
   return `aws-cdk-codepipeline-cross-region-scaffolding-${props.region}`;
 }