MINOR: fix the reviewer placeholder container multiple no (#20886)

* fix the reviewer placeholder container multiple no

* fix unit test
This commit is contained in:
Ashish Gupta 2025-04-17 19:11:47 +05:30 committed by GitHub
parent 598699eef3
commit b17c9be06d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 6 additions and 5 deletions

View File

@ -284,7 +284,9 @@ const IncidentManagerPageHeader = ({
team: false, team: false,
}} }}
owners={details?.assignee ? [details.assignee] : []} owners={details?.assignee ? [details.assignee] : []}
placeHolder={t('label.assignee')} placeHolder={t('label.no-entity', {
entity: t('label.assignee'),
})}
tooltipText={t('label.edit-entity', { tooltipText={t('label.edit-entity', {
entity: t('label.assignee'), entity: t('label.assignee'),
})} })}

View File

@ -276,7 +276,7 @@ describe('Incident Manager Page Header component', () => {
screen.getByText('TestCaseIncidentManagerStatus.component') screen.getByText('TestCaseIncidentManagerStatus.component')
).toBeInTheDocument(); ).toBeInTheDocument();
// Assignee // Assignee
expect(screen.getByText('label.assignee')).toBeInTheDocument(); expect(screen.getByTestId('assignee')).toBeInTheDocument();
// Severity // Severity
expect(screen.getByText('label.severity')).toBeInTheDocument(); expect(screen.getByText('label.severity')).toBeInTheDocument();
expect(screen.getByText('Severity.component')).toBeInTheDocument(); expect(screen.getByText('Severity.component')).toBeInTheDocument();

View File

@ -86,9 +86,8 @@ export const NoOwnerFound: React.FC<NoOwnerFoundProps> = ({
{!isCompactView && ( {!isCompactView && (
<div className="no-owner-text text-sm font-medium"> <div className="no-owner-text text-sm font-medium">
{t('label.no-entity', { {placeHolder ??
entity: placeHolder ?? t('label.owner-plural'), t('label.no-entity', { entity: t('label.owner-plural') })}
})}
</div> </div>
)} )}
</div> </div>