Skip to content

Commit bf49345

Browse files
committed
added cfn suppress rule on s3 logging bucket
1 parent 6fc6b3e commit bf49345

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

Diff for: source/patterns/@aws-solutions-constructs/aws-kinesisfirehose-s3-and-kinesisanalytics/test/integ.customLoggingBucket.ts

+11-1
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,15 @@ import { App, Stack, RemovalPolicy } from "@aws-cdk/core";
1616
import { BucketEncryption } from "@aws-cdk/aws-s3";
1717
import { KinesisFirehoseToAnalyticsAndS3 } from "../lib";
1818
import { generateIntegStackName } from '@aws-solutions-constructs/core';
19+
import * as s3 from "@aws-cdk/aws-s3";
20+
import * as defaults from '@aws-solutions-constructs/core';
1921

2022
const app = new App();
2123

2224
// Empty arguments
2325
const stack = new Stack(app, generateIntegStackName(__filename));
2426

25-
new KinesisFirehoseToAnalyticsAndS3(stack, 'test-kinesisfirehose-analytics-s3', {
27+
const construct = new KinesisFirehoseToAnalyticsAndS3(stack, 'test-kinesisfirehose-analytics-s3', {
2628
kinesisAnalyticsProps: {
2729
inputs: [{
2830
inputSchema: {
@@ -61,4 +63,12 @@ new KinesisFirehoseToAnalyticsAndS3(stack, 'test-kinesisfirehose-analytics-s3',
6163
versioned: true
6264
}
6365
});
66+
67+
const s3Bucket = construct.s3Bucket as s3.Bucket;
68+
69+
defaults.addCfnSuppressRules(s3Bucket, [
70+
{ id: 'W35',
71+
reason: 'This S3 bucket is created for unit/ integration testing purposes only.' },
72+
]);
73+
6474
app.synth();

0 commit comments

Comments
 (0)