Merge pull request #56234 from bernhardoj/fix/56015-unhover-doesnt-work

Fix subscript avatar color doesn't reset when unhovered

src/components/SelectionList/BaseListItem.tsx

@@ -48,6 +48,7 @@ function BaseListItem<TItem extends ListItem>({
     // Sync focus on an item
     useSyncFocus(pressableRef, !!isFocused, shouldSyncFocus);
     const handleMouseLeave = (e: React.MouseEvent<Element, MouseEvent>) => {
+        bind.onMouseLeave();
         e.stopPropagation();
         setMouseUp();
     };

tests/ui/BaseListItemTest.tsx

@@ -0,0 +1,35 @@
+import {fireEvent, render, screen} from '@testing-library/react-native';
+import BaseListItem from '@components/SelectionList/BaseListItem';
+import useHover from '@hooks/useHover';
+import CONST from '@src/CONST';
+
+jest.mock('@hooks/useHover', () => jest.fn());
+
+const mockedUseHover = useHover as jest.MockedFunction<typeof useHover>;
+
+describe('BaseListItem', () => {
+    it('hover should work correctly', () => {
+        const mouseEnterMock = jest.fn();
+        const mouseLeaveMock = jest.fn();
+        mockedUseHover.mockReturnValue({
+            hovered: false,
+            bind: {
+                onMouseEnter: mouseEnterMock,
+                onMouseLeave: mouseLeaveMock,
+            },
+        });
+        render(
+            <BaseListItem
+                item={{keyForList: '1'}}
+                onSelectRow={() => {}}
+                showTooltip={false}
+                isFocused={false}
+            />,
+        );
+        const testID = `${CONST.BASE_LIST_ITEM_TEST_ID}1`;
+        fireEvent(screen.getByTestId(testID), 'mouseEnter');
+        expect(mouseEnterMock).toBeCalled();
+        fireEvent(screen.getByTestId(testID), 'mouseLeave', {stopPropagation: jest.fn()});
+        expect(mouseLeaveMock).toBeCalled();
+    });
+});