fix: prevent useTooltipTrigger from setting tabIndex (#8622)

* remove tabIndex set in useTooltipTrigger

* add test

* add the opposite test as well

---------

Co-authored-by: Robert Snow <[email protected]>

Author: abeljohn

packages/@react-aria/tooltip/src/useTooltipTrigger.ts

@@ -139,9 +139,9 @@ export function useTooltipTrigger(props: TooltipTriggerProps, state: TooltipTrig
       'aria-describedby': state.isOpen ? tooltipId : undefined,
       ...mergeProps(focusableProps, hoverProps, {
         onPointerDown: onPressStart,
-        onKeyDown: onPressStart,
-        tabIndex: undefined
-      })
+        onKeyDown: onPressStart
+      }),
+      tabIndex: undefined
     },
     tooltipProps: {
       id: tooltipId

packages/react-aria-components/test/Tooltip.test.js

@@ -312,4 +312,32 @@ describe('Tooltip', () => {
     let tooltip = getByRole('tooltip');
     expect(tooltip).toBeInTheDocument();
   });
+
+  it('should not override child properties for excludeFromTabOrder', async () => {
+    let {getByRole} = render(
+      <TooltipTrigger>
+        <Button excludeFromTabOrder>
+          Check my tabindex
+        </Button>
+        <Tooltip>hello world</Tooltip>
+      </TooltipTrigger>
+    );
+
+    let button = getByRole('button');
+    expect(button).toHaveAttribute('tabindex', '-1');
+  });
+
+  it('should not override child properties', async () => {
+    let {getByRole} = render(
+      <TooltipTrigger>
+        <Button>
+          Check my tabindex
+        </Button>
+        <Tooltip>hello world</Tooltip>
+      </TooltipTrigger>
+    );
+
+    let button = getByRole('button');
+    expect(button).toHaveAttribute('tabindex', '0');
+  });
 });