add feedback from Jimmy

Author: msambol

Diff for: packages/@aws-cdk/aws-pipes-targets-alpha/lib/sns.ts

@@ -34,7 +34,7 @@ export class SnsTarget implements ITarget {
   }
 
   bind(pipe: IPipe): TargetConfig {
-    if (!this.topicParameters) {
+    if (!this.topicParameters?.inputTransformation) {
       return {
         targetParameters: {},
       };

Diff for: packages/@aws-cdk/aws-pipes-targets-alpha/test/sns.test.ts

@@ -59,6 +59,28 @@ describe('sns', () => {
     });
   });
 
+  it('should handle an empty parameter object', () => {
+    // ARRANGE
+    const app = new App();
+    const stack = new Stack(app, 'TestStack');
+    const topic = new Topic(stack, 'MyTopic', {});
+
+    const target = new SnsTarget(topic, {});
+
+    new Pipe(stack, 'MyPipe', {
+      source: new TestSource(),
+      target,
+    });
+
+    // ACT
+    const template = Template.fromStack(stack);
+
+    // ASSERT
+    template.hasResourceProperties('AWS::Pipes::Pipe', {
+      TargetParameters: {},
+    });
+  });
+
   it('should grant pipe role publish access', () => {
     // ARRANGE
     const app = new App();