chore: fix tests

Author: anuraghazra

packages/blade/src/components/ButtonGroup/__tests__/ButtonGroup.web.test.tsx

@@ -7,7 +7,6 @@ import { Button } from '~components/Button/Button';
 import { ChevronDownIcon, PlusIcon } from '~components/Icons';
 import { Dropdown, DropdownButton, DropdownOverlay } from '~components/Dropdown';
 import { ActionList, ActionListItem } from '~components/ActionList';
-import { AutoComplete } from '~components/Input/DropdownInputTriggers';
 
 beforeAll(() => jest.spyOn(console, 'error').mockImplementation());
 afterAll(() => jest.restoreAllMocks());
@@ -168,40 +167,7 @@ describe('<ButtonGroup />', () => {
         </ButtonGroup>,
       ),
     ).toThrowError(
-      '[Blade: ButtonGroup]: Only "Button" or "Dropdown" component with Button trigger are allowed as children.',
-    );
-  });
-
-  it('should throw error with invalid dropdown children', () => {
-    expect(() =>
-      renderWithTheme(
-        <ButtonGroup>
-          <Button icon={PlusIcon}>Payout</Button>
-          <Dropdown selectionType="single">
-            <AutoComplete
-              label="City"
-              placeholder="Select your City"
-              name="action"
-              onChange={({ name, values }) => {
-                console.log({ name, values });
-              }}
-              onInputValueChange={({ name, value }) => {
-                console.log({ name, value });
-              }}
-            />
-            <DropdownOverlay>
-              <ActionList>
-                <ActionListItem title="Mumbai" value="mumbai" />
-                <ActionListItem title="Pune" value="pune" />
-                <ActionListItem title="Bangalore" value="bangalore" />
-                <ActionListItem title="Mysore" value="mysore" />
-              </ActionList>
-            </DropdownOverlay>
-          </Dropdown>
-        </ButtonGroup>,
-      ),
-    ).toThrowError(
-      '[Blade: ButtonGroup]: Only "Button" or "Dropdown" component with Button trigger are allowed as children.',
+      '[Blade: ButtonGroup]: Only `Button, Dropdown, Tooltip, Popover` components are accepted in `ButtonGroup` children',
     );
   });
 });

packages/blade/src/utils/useControllable.web.test.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import userEvent from '@testing-library/user-event';
 import React from 'react';
 import renderWithTheme from './testing/renderWithTheme.web';
@@ -10,18 +11,20 @@ const Example = ({
 }: {
   value?: number;
   defaultValue?: number;
-  onChange?: (value: number) => void;
+  onChange?: (value: number, extra: any) => void;
 }): React.ReactElement => {
   const [_value, setValue] = useControllableState({
     value,
     defaultValue: defaultValue ?? 0,
-    onChange,
+    onChange: (value, extraData) => {
+      onChange?.(value, extraData);
+    },
   });
 
   return (
     <button
       onClick={() => {
-        setValue((prev) => prev + 1);
+        setValue((prev) => prev + 1, false, 'extra');
       }}
     >
       {_value}
@@ -87,11 +90,11 @@ describe('useControllableState', () => {
     const user = userEvent.setup();
     await user.click(getByRole('button'));
 
-    expect(onChange).toHaveBeenCalledWith(2);
+    expect(onChange).toHaveBeenCalledWith(2, 'extra');
     expect(onChange).toHaveBeenCalledTimes(1);
 
     await user.click(getByRole('button'));
-    expect(onChange).toHaveBeenCalledWith(3);
+    expect(onChange).toHaveBeenCalledWith(3, 'extra');
     expect(onChange).toHaveBeenCalledTimes(2);
   });
 });