test - address some typos

Author: biffgaut

source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/package.json

@@ -55,7 +55,7 @@
   "dependencies": {
     "@aws-cdk/aws-ec2": "0.0.0",
     "@aws-cdk/aws-s3": "0.0.0",
-    "@aws-cdk/aws-s3-deployment": "0.0.0",
+    "@aws-cdk/aws-s3-assets": "0.0.0",
     "@aws-cdk/aws-iam": "0.0.0",
     "@aws-cdk/aws-lambda": "0.0.0",
     "@aws-cdk/aws-sagemaker": "0.0.0",
@@ -85,7 +85,7 @@
   "peerDependencies": {
     "@aws-cdk/aws-ec2": "0.0.0",
     "@aws-cdk/aws-s3": "0.0.0",
-    "@aws-cdk/aws-s3-deployment": "0.0.0",
+    "@aws-cdk/aws-s3-assets": "0.0.0",
     "@aws-cdk/aws-iam": "0.0.0",
     "@aws-cdk/aws-lambda": "0.0.0",
     "@aws-cdk/aws-sagemaker": "0.0.0",

source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/test/test-helper.ts

@@ -11,11 +11,8 @@
  *  and limitations under the License.
  */
 
-// import * as s3 from '@aws-cdk/aws-s3';
-// import * as lambda from '@aws-cdk/aws-lambda';
 import * as s3a from '@aws-cdk/aws-s3-assets';
 import { Stack, CfnMapping } from '@aws-cdk/core';
-// import { CreateScrapBucket } from '@aws-solutions-constructs/core';
 
 // linear learner ECR images can be found here:
 // https://github.com/awsdocs/amazon-sagemaker-developer-guide/blob/master/doc_source/sagemaker-algo-docker-registry-paths.md
@@ -89,46 +86,10 @@ export function getSagemakerModel(stack: Stack): [CfnMapping, s3a.Asset ] {
     }
   });
 
-  // const sourceBucket = new s3.Bucket(stack, 'source-bucket', {
-  //   removalPolicy: RemovalPolicy.DESTROY,
-  //   autoDeleteObjects: true
-  // });
-
-  // const sourceBucket = CreateScrapBucket(stack, { autoDeleteObjects: true });
-
-  // const customResourceLambda = stack.node.findChild('Custom::S3AutoDeleteObjectsCustomResourceProvider').node.findChild('Handler') as lambda.CfnFunction;
-
-  // customResourceLambda.cfnOptions.metadata = {
-  //   cfn_nag: {
-  //     rules_to_suppress: [{
-  //       id: 'W58',
-  //       reason: `Lambda functions has the required permission to write CloudWatch Logs. It uses custom policy instead of arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole with tighter permissions.`
-  //     },
-  //     {
-  //       id: 'W89',
-  //       reason: `This is not a rule for the general case, just for specific use cases/industries`
-  //     },
-  //     {
-  //       id: 'W92',
-  //       reason: `Impossible for us to define the correct concurrency for clients`
-  //     }]
-  //   }
-  // };
-
-  // const modelDeployment = new s3d.BucketDeployment(stack, 'bucket-deployment', {
-  //   destinationBucket: sourceBucket,
-  //   sources: [
-  //     s3d.Source.asset(`${__dirname}/model`)
-  //   ]
-  // });
-
-//   return [containerMap, sourceBucket, modelDeployment];
-
   const modelAsset = new s3a.Asset(stack, 'SampleAsset', {
     path: 'model/model.tar.gz',
   });
 
-
-  return [containerMap, modelAsset]
+  return [containerMap, modelAsset];
 
 }