Use spread operator for lambda action, test

* use spread operator for lambda invoke action; * revert `integ.pipeline-cfn.expected.json`
aws · Jan 11, 2019 · 96b1628 · 96b1628
1 parent 17e2fbc
commit 96b1628
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/packages/@aws-cdk/aws-codepipeline/test/integ.pipeline-cfn.expected.json b/packages/@aws-cdk/aws-codepipeline/test/integ.pipeline-cfn.expected.json
@@ -147,6 +147,12 @@
     "PipelineC660917D": {
       "Type": "AWS::CodePipeline::Pipeline",
       "Properties": {
+        "ArtifactStore": {
+          "Location": {
+            "Ref": "PipelineArtifactsBucket22248F97"
+          },
+          "Type": "S3"
+        },
         "RoleArn": {
           "Fn::GetAtt": [
             "PipelineRoleD68726F7",
@@ -215,13 +221,7 @@
             ],
             "Name": "CFN"
           }
-        ],
-        "ArtifactStore": {
-          "Location": {
-            "Ref": "PipelineArtifactsBucket22248F97"
-          },
-          "Type": "S3"
-        }
+        ]
       },
       "DependsOn": [
         "PipelineRoleD68726F7",
@@ -254,4 +254,4 @@
       }
     }
   }
-}
+}
diff --git a/packages/@aws-cdk/aws-lambda/lib/pipeline-action.ts b/packages/@aws-cdk/aws-lambda/lib/pipeline-action.ts
@@ -83,9 +83,7 @@ export interface PipelineInvokeActionProps extends CommonPipelineInvokeActionPro
 export class PipelineInvokeAction extends codepipeline.Action {
   constructor(scope: cdk.Construct, id: string, props: PipelineInvokeActionProps) {
     super(scope, id, {
-      stage: props.stage,
-      role: props.role,
-      runOrder: props.runOrder,
+      ...props,
       category: codepipeline.ActionCategory.Invoke,
       provider: 'Lambda',
       artifactBounds: codepipeline.defaultBounds(),