diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.test.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.test.tsx index da37689574e..0122c6c424d 100644 --- a/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.test.tsx @@ -11,7 +11,7 @@ * limitations under the License. */ -import { findByTestId, render } from '@testing-library/react'; +import { findByTestId, fireEvent, render } from '@testing-library/react'; import React from 'react'; import { MemoryRouter } from 'react-router-dom'; import UserCard from './UserCard'; @@ -101,4 +101,25 @@ describe('Test userCard component', () => { expect(svgIcon).toBeInTheDocument(); expect(datasetLink).toBeInTheDocument(); }); + + it('If isOwner is passed it should allow delete action', async () => { + const { container } = render( + , + { + wrapper: MemoryRouter, + } + ); + + const remove = await findByTestId(container, 'remove'); + + fireEvent.click(remove); + + expect(remove).toBeInTheDocument(); + expect(mockRemove).toBeCalled(); + }); }); diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.tsx index da42a98f2b7..08d56110356 100644 --- a/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/pages/teams/UserCard.tsx @@ -42,6 +42,7 @@ interface Props { isIconVisible?: boolean; isDataset?: boolean; isCheckBoxes?: boolean; + isOwner?: boolean; onSelect?: (value: string) => void; onRemove?: (value: string) => void; } @@ -52,6 +53,7 @@ const UserCard = ({ isIconVisible = false, isDataset = false, isCheckBoxes = false, + isOwner = false, onSelect, onRemove, }: Props) => { @@ -202,6 +204,7 @@ const UserCard = ({ ) : ( You do not have permission to update the team.} + isOwner={isOwner} permission={Operation.UpdateTeam} position="bottom"> {

There are no users added yet.

{isAdminUser || isAuthDisabled || + isOwner() || userPermissions[Operation.UpdateTeam] ? ( <>

Would like to start adding some?

@@ -476,6 +477,7 @@ const TeamsPage = () => {