Compare commits

...

1 Commits

Author SHA1 Message Date
yyh
10af8f8831 fix(web): name plugin credential actions 2026-08-09 15:02:55 +08:00
2 changed files with 19 additions and 20 deletions

View File

@@ -191,8 +191,7 @@ describe('Item Component', () => {
)
const enterRenameMode = () => {
const firstButton = result.container.querySelectorAll('button')[0] as HTMLElement
fireEvent.click(firstButton)
fireEvent.click(screen.getByRole('button', { name: 'common.operation.rename' }))
}
return { ...result, onRename, enterRenameMode }
@@ -337,7 +336,7 @@ describe('Item Component', () => {
credentials: { api_key: 'secret' },
})
const { container } = render(
render(
<Item
credential={credential}
onEdit={onEdit}
@@ -348,10 +347,7 @@ describe('Item Component', () => {
/>,
)
const editButton = container
.querySelector('.i-ri-equalizer-2-line')
?.closest('button') as HTMLElement
fireEvent.click(editButton)
fireEvent.click(screen.getByRole('button', { name: 'common.operation.edit' }))
expect(onEdit).toHaveBeenCalledWith('edit-test-id', {
api_key: 'secret',
__name__: 'Edit Test',
@@ -403,7 +399,7 @@ describe('Item Component', () => {
const onDelete = vi.fn()
const credential = createCredential({ id: 'delete-test-id' })
const { container } = render(
render(
<Item
credential={credential}
onDelete={onDelete}
@@ -414,10 +410,7 @@ describe('Item Component', () => {
/>,
)
const deleteButton = container
.querySelector('.i-ri-delete-bin-line')
?.closest('button') as HTMLElement
fireEvent.click(deleteButton)
fireEvent.click(screen.getByRole('button', { name: 'common.operation.delete' }))
expect(onDelete).toHaveBeenCalledWith('delete-test-id')
})
@@ -521,7 +514,7 @@ describe('Item Component', () => {
const onDelete = vi.fn()
const credential = createCredential()
const { container } = render(
render(
<Item
credential={credential}
onItemClick={onItemClick}
@@ -533,10 +526,7 @@ describe('Item Component', () => {
/>,
)
const deleteButton = container
.querySelector('.i-ri-delete-bin-line')
?.closest('button') as HTMLElement
fireEvent.click(deleteButton)
fireEvent.click(screen.getByRole('button', { name: 'common.operation.delete' }))
expect(onDelete).toHaveBeenCalled()
})
})

View File

@@ -177,6 +177,7 @@ const Item = ({
<TooltipTrigger
render={
<ActionButton
aria-label={t(($) => $['operation.rename'], { ns: 'common' })}
disabled={disabled || !canManageCredential}
onClick={(e) => {
e.stopPropagation()
@@ -184,7 +185,7 @@ const Item = ({
setRenameValue(credential.name)
}}
>
<span className="i-ri-edit-line size-4 text-text-tertiary" />
<span aria-hidden className="i-ri-edit-line size-4 text-text-tertiary" />
</ActionButton>
}
/>
@@ -200,6 +201,7 @@ const Item = ({
<TooltipTrigger
render={
<ActionButton
aria-label={t(($) => $['operation.edit'], { ns: 'common' })}
disabled={disabled || !canManageCredential}
onClick={(e) => {
e.stopPropagation()
@@ -210,7 +212,10 @@ const Item = ({
})
}}
>
<span className="i-ri-equalizer-2-line size-4 text-text-tertiary" />
<span
aria-hidden
className="i-ri-equalizer-2-line size-4 text-text-tertiary"
/>
</ActionButton>
}
/>
@@ -222,6 +227,7 @@ const Item = ({
<TooltipTrigger
render={
<ActionButton
aria-label={t(($) => $['operation.delete'], { ns: 'common' })}
className="hover:bg-transparent"
disabled={disabled || !canManageCredential}
onClick={(e) => {
@@ -229,7 +235,10 @@ const Item = ({
onDelete?.(credential.id)
}}
>
<span className="i-ri-delete-bin-line size-4 text-text-tertiary hover:text-text-destructive" />
<span
aria-hidden
className="i-ri-delete-bin-line size-4 text-text-tertiary hover:text-text-destructive"
/>
</ActionButton>
}
/>