refactor: remove redundant default callbacks and failing test

Author: lposen

src/embedded/components/IterableEmbeddedBanner/IterableEmbeddedBanner.test.tsx

@@ -303,19 +303,6 @@ describe('IterableEmbeddedBanner', () => {
         }
       );
     });
-
-    it('should call useEmbeddedView with default callbacks when not provided', () => {
-      render(<IterableEmbeddedBanner message={baseMessage} />);
-
-      expect(mockUseEmbeddedView).toHaveBeenCalledWith(
-        IterableEmbeddedViewType.Banner,
-        expect.objectContaining({
-          message: baseMessage,
-          onButtonClick: expect.any(Function),
-          onMessageClick: expect.any(Function),
-        })
-      );
-    });
   });
 
   describe('Edge cases', () => {

src/embedded/components/IterableEmbeddedBanner/IterableEmbeddedBanner.tsx

@@ -18,17 +18,15 @@ import {
   IMAGE_WIDTH,
 } from './IterableEmbeddedBanner.styles';
 
-const noop = () => {};
-
 /**
  * TODO: figure out how default action works.
  */
 
 export const IterableEmbeddedBanner = ({
   config,
   message,
-  onButtonClick = noop,
-  onMessageClick = noop,
+  onButtonClick,
+  onMessageClick,
 }: IterableEmbeddedComponentProps) => {
   const { parsedStyles, media, handleButtonClick, handleMessageClick } =
     useEmbeddedView(IterableEmbeddedViewType.Banner, {

src/embedded/components/IterableEmbeddedNotification/IterableEmbeddedNotification.test.tsx

@@ -278,19 +278,6 @@ describe('IterableEmbeddedNotification', () => {
         }
       );
     });
-
-    it('should call useEmbeddedView with default callbacks when not provided', () => {
-      render(<IterableEmbeddedNotification message={baseMessage} />);
-
-      expect(mockUseEmbeddedView).toHaveBeenCalledWith(
-        IterableEmbeddedViewType.Notification,
-        expect.objectContaining({
-          message: baseMessage,
-          onButtonClick: expect.any(Function),
-          onMessageClick: expect.any(Function),
-        })
-      );
-    });
   });
 
   describe('Edge cases', () => {