Fix lambda test following the removal of the asset grants

Fixes #664 (asset grants not needed)
aws · Oct 17, 2018 · 5feac95 · 5feac95
1 parent fd30bee
commit 5feac95
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 88 deletions.
diff --git a/packages/@aws-cdk/aws-lambda/test/integ.assets.file.expected.json b/packages/@aws-cdk/aws-lambda/test/integ.assets.file.expected.json
@@ -31,91 +31,6 @@
         ]
       }
     },
-    "MyLambdaServiceRoleDefaultPolicy5BBC6F68": {
-      "Type": "AWS::IAM::Policy",
-      "Properties": {
-        "PolicyDocument": {
-          "Version": "2012-10-17",
-          "Statement": [
-            {
-              "Effect": "Allow",
-              "Action": [
-                "s3:GetBucket*",
-                "s3:GetObject*",
-                "s3:List*"
-              ],
-              "Resource": [
-                {
-                  "Fn::Join": [
-                    "",
-                    [
-                      "arn:",
-                      {
-                        "Ref": "AWS::Partition"
-                      },
-                      ":s3:::",
-                      {
-                        "Ref": "MyLambdaCodeS3BucketC82A5870"
-                      }
-                    ]
-                  ]
-                },
-                {
-                  "Fn::Join": [
-                    "",
-                    [
-                      {
-                        "Fn::Join": [
-                          "",
-                          [
-                            "arn:",
-                            {
-                              "Ref": "AWS::Partition"
-                            },
-                            ":s3:::",
-                            {
-                              "Ref": "MyLambdaCodeS3BucketC82A5870"
-                            }
-                          ]
-                        ]
-                      },
-                      "/",
-                      {
-                        "Fn::Join": [
-                          "",
-                          [
-                            {
-                              "Fn::Select": [
-                                0,
-                                {
-                                  "Fn::Split": [
-                                    "||",
-                                    {
-                                      "Ref": "MyLambdaCodeS3VersionKey47762537"
-                                    }
-                                  ]
-                                }
-                              ]
-                            },
-                            "*"
-                          ]
-                        ]
-                      }
-                    ]
-                  ]
-                }
-              ]
-            }
-          ]
-        },
-        "PolicyName": "MyLambdaServiceRoleDefaultPolicy5BBC6F68",
-        "Roles": [
-          {
-            "Ref": "MyLambdaServiceRole4539ECB6"
-          }
-        ]
-      }
-    },
     "MyLambdaCCE802FB": {
       "Type": "AWS::Lambda::Function",
       "Properties": {
@@ -167,8 +82,7 @@
         "Runtime": "python3.6"
       },
       "DependsOn": [
-        "MyLambdaServiceRole4539ECB6",
-        "MyLambdaServiceRoleDefaultPolicy5BBC6F68"
+        "MyLambdaServiceRole4539ECB6"
       ]
     }
   },

diff --git a/packages/@aws-cdk/aws-lambda/test/test.code.ts b/packages/@aws-cdk/aws-lambda/test/test.code.ts
@@ -9,7 +9,6 @@ export = {
     'fails if used with unsupported runtimes'(test: Test) {
       test.throws(() => defineFunction(lambda.Code.inline('boom'), lambda.Runtime.Go1x), /Inline source not allowed for go1\.x/);
       test.throws(() => defineFunction(lambda.Code.inline('boom'), lambda.Runtime.Java8), /Inline source not allowed for java8/);
-      test.throws(() => defineFunction(lambda.Code.inline('boom'), lambda.Runtime.NodeJS810), /Inline source not allowed for nodejs8\.10/);
       test.done();
     },
     'fails if larger than 4096 bytes'(test: Test) {