bug(UIKIT-1691,PageHeader): Исправлена работа onClick у SecondaryVisi…

…bleAction (#1088)
kaluga-astral · Aug 12, 2024 · 46bde0a · 46bde0a
1 parent a09a70b
commit 46bde0a
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 3 deletions.
diff --git a/...der/ButtonGroup/ButtonGroupSecondaryVisibleActions/ButtonGroupSecondaryVisibleActions.tsx b/...der/ButtonGroup/ButtonGroupSecondaryVisibleActions/ButtonGroupSecondaryVisibleActions.tsx
@@ -22,8 +22,7 @@ export const ButtonGroupSecondaryVisibleActions = ({
   actions,
 }: ButtonGroupMainActionProps) => {
   return actions.map((action) => {
-    const { name, icon, disableReason, disabled, onClick, ...mainProps } =
-      action;
+    const { name, icon, disableReason, disabled, ...mainProps } = action;
 
     return (
       <Tooltip

diff --git a/packages/components/src/PageHeader/PageHeader.test.tsx b/packages/components/src/PageHeader/PageHeader.test.tsx
@@ -1,4 +1,4 @@
-import { describe, expect, it } from 'vitest';
+import { describe, expect, it, vi } from 'vitest';
 import { renderWithTheme, screen, userEvents } from '@astral/tests';
 
 import { PageHeader } from './PageHeader';
@@ -165,4 +165,28 @@ describe('PageHeader', () => {
     expect(tooltip).toBeVisible();
     expect(tooltip).toHaveTextContent('Отправка по маршруту');
   });
+
+  it('OnClick вызывается при нажатии на вторичные видимые действия', async () => {
+    const onClickSpy = vi.fn();
+
+    renderWithTheme(
+      <PageHeader
+        title="Черновик"
+        actions={{
+          secondaryVisible: [
+            {
+              name: 'Отправка по маршруту',
+              icon: <svg />,
+              onClick: onClickSpy,
+            },
+          ],
+        }}
+      />,
+    );
+
+    const button = screen.getByRole('button');
+
+    await userEvents.click(button);
+    expect(onClickSpy).toBeCalled();
+  });
 });