From f8fd84a2da44614353848cc31b8367861748cd5b Mon Sep 17 00:00:00 2001
From: Binij Shrestha <shresthabinij@gmail.com>
Date: Thu, 18 May 2023 17:19:51 +0545
Subject: [PATCH] Omit children from IconProps

---
 .../avatar-icon/avatar-icon.test.tsx                | 13 +++----------
 .../avatar-icon/avatar-icon.types.ts                |  2 +-
 2 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/ui/components/component-library/avatar-icon/avatar-icon.test.tsx b/ui/components/component-library/avatar-icon/avatar-icon.test.tsx
index 84d709de5ad0..9f92f3a00fc8 100644
--- a/ui/components/component-library/avatar-icon/avatar-icon.test.tsx
+++ b/ui/components/component-library/avatar-icon/avatar-icon.test.tsx
@@ -15,7 +15,6 @@ describe('AvatarIcon', () => {
       <AvatarIcon
         iconName={IconName.SwapHorizontal}
         data-testid="avatar-icon"
-        children={undefined}
       />,
     );
     expect(getByTestId('avatar-icon')).toBeDefined();
@@ -29,31 +28,29 @@ describe('AvatarIcon', () => {
           iconName={IconName.SwapHorizontal}
           size={AvatarIconSize.Xs}
           data-testid={AvatarIconSize.Xs}
-          children={undefined}
         />
         <AvatarIcon
           iconName={IconName.SwapHorizontal}
           size={AvatarIconSize.Sm}
           data-testid={AvatarIconSize.Sm}
-          children={undefined}
         />
         <AvatarIcon
           iconName={IconName.SwapHorizontal}
           size={AvatarIconSize.Md}
           data-testid={AvatarIconSize.Md}
-          children={undefined}
+
         />
         <AvatarIcon
           iconName={IconName.SwapHorizontal}
           size={AvatarIconSize.Lg}
           data-testid={AvatarIconSize.Lg}
-          children={undefined}
+
         />
         <AvatarIcon
           iconName={IconName.SwapHorizontal}
           size={AvatarIconSize.Xl}
           data-testid={AvatarIconSize.Xl}
-          children={undefined}
+
         />
       </>,
     );
@@ -80,7 +77,6 @@ describe('AvatarIcon', () => {
         iconName={IconName.SwapHorizontal}
         className="mm-avatar-icon--test"
         data-testid="classname"
-        children={undefined}
       />,
     );
     expect(getByTestId('classname')).toHaveClass('mm-avatar-icon--test');
@@ -91,7 +87,6 @@ describe('AvatarIcon', () => {
       <AvatarIcon
         iconName={IconName.SwapHorizontal}
         iconProps={{ 'data-testid': 'avatar-icon' }}
-        children={undefined}
       />,
     );
 
@@ -105,7 +100,6 @@ describe('AvatarIcon', () => {
         color={IconColor.successDefault}
         backgroundColor={BackgroundColor.successMuted}
         data-testid="success"
-        children={undefined}
       />,
     );
 
@@ -120,7 +114,6 @@ describe('AvatarIcon', () => {
       <AvatarIcon
         iconName={IconName.SwapHorizontal}
         ref={ref}
-        children={undefined}
       />,
     );
     expect(ref.current).not.toBeNull();
diff --git a/ui/components/component-library/avatar-icon/avatar-icon.types.ts b/ui/components/component-library/avatar-icon/avatar-icon.types.ts
index b0f9597a46d4..1bf677686288 100644
--- a/ui/components/component-library/avatar-icon/avatar-icon.types.ts
+++ b/ui/components/component-library/avatar-icon/avatar-icon.types.ts
@@ -10,7 +10,7 @@ export enum AvatarIconSize {
   Xl = 'xl',
 }
 
-export interface AvatarIconProps extends Omit<AvatarBaseProps, 'color'> {
+export interface AvatarIconProps extends Omit<AvatarBaseProps, 'color'| 'children'> {
   /**
    * The name of the icon to display. Should be one of IconName
    */