From f30ecd65b81abe7ccf998f0155afdb5963be3b81 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20=C4=8Curda?= <jakub.curda@almacareer.com>
Date: Fri, 27 Dec 2024 12:02:30 +0100
Subject: [PATCH] fixup! Feat(web-react): Additional parameter for styleProps
 and Box refactor

---
 .../src/hooks/__tests__/styleProps.test.ts    |  42 ++++++++
 .../hooks/__tests__/useStyleUtilities.test.ts | 101 ++++++++++++++++++
 2 files changed, 143 insertions(+)

diff --git a/packages/web-react/src/hooks/__tests__/styleProps.test.ts b/packages/web-react/src/hooks/__tests__/styleProps.test.ts
index cafb96fb93..bc5e28190e 100644
--- a/packages/web-react/src/hooks/__tests__/styleProps.test.ts
+++ b/packages/web-react/src/hooks/__tests__/styleProps.test.ts
@@ -131,4 +131,46 @@ describe('styleProps', () => {
       expect(result.current.styleProps).toEqual(expected);
     });
   });
+
+  it('should process style props with additional utilities', () => {
+    const mockProps = {
+      margin: 'space-100',
+      marginX: 'space-200',
+      marginY: 'space-400',
+      padding: 'space-500',
+      paddingX: 'space-600',
+      paddingY: 'space-700',
+    };
+    const additionalUtilities = {
+      padding: 'p',
+      paddingX: 'px',
+      paddingY: 'py',
+    };
+
+    const { result } = renderHook(() => useStyleProps(mockProps as StyleProps, additionalUtilities));
+
+    expect(result.current.styleProps).toEqual({
+      className: 'm-100 mx-200 my-400 p-500 px-600 py-700',
+      style: undefined,
+    });
+  });
+
+  it('should process style props with responsive additional utilities', () => {
+    const mockProps = {
+      margin: 'space-100',
+      marginX: 'space-200',
+      marginY: 'space-400',
+      padding: { mobile: 'space-500', tablet: 'space-600', desktop: 'space-700' },
+    };
+    const additionalUtilities = {
+      padding: 'p',
+    };
+
+    const { result } = renderHook(() => useStyleProps(mockProps as StyleProps, additionalUtilities));
+
+    expect(result.current.styleProps).toEqual({
+      className: 'm-100 mx-200 my-400 p-500 p-tablet-600 p-desktop-700',
+      style: undefined,
+    });
+  });
 });
diff --git a/packages/web-react/src/hooks/__tests__/useStyleUtilities.test.ts b/packages/web-react/src/hooks/__tests__/useStyleUtilities.test.ts
index a283b428b2..c6a769c81f 100644
--- a/packages/web-react/src/hooks/__tests__/useStyleUtilities.test.ts
+++ b/packages/web-react/src/hooks/__tests__/useStyleUtilities.test.ts
@@ -40,4 +40,105 @@ describe('useStyleUtilities hook', () => {
     expect(result.current.styleUtilities).toEqual(['m-100', 'mx-200', 'mx-tablet-auto', 'mx-desktop-300']);
     expect(result.current.props).toEqual({});
   });
+
+  it('should process style utilities correctly with responsive values', () => {
+    const mockProps = {
+      margin: { mobile: 'space-100', tablet: 'space-200', desktop: 'space-300' },
+      marginX: { mobile: 'space-200', tablet: 'space-200', desktop: 'space-300' },
+    };
+    const mockPrefix = 'test-prefix';
+
+    const { result } = renderHook(() => useStyleUtilities(mockProps as StyleProps, mockPrefix));
+
+    expect(result.current.styleUtilities).toEqual([
+      'test-prefix-m-100',
+      'test-prefix-m-tablet-200',
+      'test-prefix-m-desktop-300',
+      'test-prefix-mx-200',
+      'test-prefix-mx-tablet-200',
+      'test-prefix-mx-desktop-300',
+    ]);
+    expect(result.current.props).toEqual({});
+  });
+
+  it('should process style utilities correctly with responsive values without prefix', () => {
+    const mockProps = {
+      margin: { mobile: 'space-100', tablet: 'space-200', desktop: 'space-300' },
+      marginX: { mobile: 'space-200', tablet: 'space-200', desktop: 'space-300' },
+    };
+
+    const { result } = renderHook(() => useStyleUtilities(mockProps as StyleProps));
+
+    expect(result.current.styleUtilities).toEqual([
+      'm-100',
+      'm-tablet-200',
+      'm-desktop-300',
+      'mx-200',
+      'mx-tablet-200',
+      'mx-desktop-300',
+    ]);
+    expect(result.current.props).toEqual({});
+  });
+
+  it('should process style utilities with additional spacing props', () => {
+    const mockProps = {
+      margin: 'space-100',
+      marginX: 'space-200',
+      marginY: 'space-400',
+      padding: 'space-500',
+      paddingX: 'space-600',
+      paddingY: 'space-700',
+    };
+    const additionalSpacingProps = {
+      padding: 'p',
+      paddingX: 'px',
+      paddingY: 'py',
+    };
+
+    const { result } = renderHook(() =>
+      useStyleUtilities(mockProps as StyleProps, 'test-prefix', additionalSpacingProps),
+    );
+
+    expect(result.current.styleUtilities).toEqual([
+      'test-prefix-m-100',
+      'test-prefix-mx-200',
+      'test-prefix-my-400',
+      'test-prefix-p-500',
+      'test-prefix-px-600',
+      'test-prefix-py-700',
+    ]);
+  });
+
+  it('should process style utilities with responsive additional spacing props', () => {
+    const mockProps = {
+      margin: 'space-100',
+      marginX: 'space-200',
+      marginY: 'space-400',
+      padding: { mobile: 'space-500', tablet: 'space-600', desktop: 'space-700' },
+      paddingX: { mobile: 'space-600', tablet: 'space-700', desktop: 'space-800' },
+      paddingY: { mobile: 'space-700', tablet: 'space-800', desktop: 'space-900' },
+    };
+    const additionalSpacingProps = {
+      padding: 'p',
+      paddingX: 'px',
+      paddingY: 'py',
+    };
+
+    const { result } = renderHook(() => useStyleUtilities(mockProps as StyleProps, '', additionalSpacingProps));
+
+    expect(result.current.styleUtilities).toEqual([
+      'm-100',
+      'mx-200',
+      'my-400',
+      'p-500',
+      'p-tablet-600',
+      'p-desktop-700',
+      'px-600',
+      'px-tablet-700',
+      'px-desktop-800',
+      'py-700',
+      'py-tablet-800',
+      'py-desktop-900',
+    ]);
+  });
 });