From 324aab71a42a7204c0eb8a13bbd46faaebdc02f7 Mon Sep 17 00:00:00 2001
From: Shrushti Polekar <shrushtipolekar@gmail.com>
Date: Fri, 30 May 2025 10:46:35 +0530
Subject: [PATCH] Fix #21468 :  Profile picture display issue (#21474)

* fix profile url issue

* added unit test
---
 .../ProfileSectionUserDetailsCard.component.tsx |  2 +-
 .../ProfilePicture/ProfilePicture.test.tsx      | 17 ++++++++++++++++-
 2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/ProfileCard/ProfileSectionUserDetailsCard.component.tsx b/openmetadata-ui/src/main/resources/ui/src/components/ProfileCard/ProfileSectionUserDetailsCard.component.tsx
index 0558fbe4ea2..14afa95cb65 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/ProfileCard/ProfileSectionUserDetailsCard.component.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/ProfileCard/ProfileSectionUserDetailsCard.component.tsx
@@ -243,7 +243,7 @@ const ProfileSectionUserDetailsCard = ({
           <div className="d-flex items-center">
             <ProfilePicture
               data-testid="replied-user"
-              name={getEntityName(userData)}
+              name={userData?.name}
               width="80"
             />
           </div>
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/common/ProfilePicture/ProfilePicture.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/common/ProfilePicture/ProfilePicture.test.tsx
index 0005f4b9f30..5121a95a33c 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/common/ProfilePicture/ProfilePicture.test.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/common/ProfilePicture/ProfilePicture.test.tsx
@@ -28,8 +28,9 @@ jest.mock('../../../utils/UserDataUtils', () => {
   };
 });
 
+const mockUseUserProfile = jest.fn().mockReturnValue(['', false, {}]);
 jest.mock('../../../hooks/user-profile/useUserProfile', () => ({
-  useUserProfile: jest.fn().mockImplementation(() => ['', false, {}]),
+  useUserProfile: () => mockUseUserProfile(),
 }));
 
 const mockData = {
@@ -44,4 +45,18 @@ describe('Test ProfilePicture component', () => {
 
     expect(avatar).toBeInTheDocument();
   });
+
+  it('should render with profile image when profileURL is available', async () => {
+    mockUseUserProfile.mockReturnValue([
+      'https://example.com/profile.jpg',
+      false,
+      {},
+    ]);
+
+    const { container } = render(<ProfilePicture {...mockData} width="36" />);
+
+    const profileImage = await findByTestId(container, 'profile-image');
+
+    expect(profileImage).toBeInTheDocument();
+  });
 });