Tooltip2: Make the text prop required (#3925)

* Tooltip: make text prop required

* changeset

.changeset/young-trees-move.md

@@ -0,0 +1,5 @@
+---
+'@primer/react': patch
+---
+
+Tooltip2: make the text prop required

src/drafts/Tooltip/Tooltip.docs.json

@@ -13,6 +13,7 @@
     },
     {
       "name": "text",
+      "required": true,
       "type": "string",
       "description": "The text to be displayed in the tooltip"
     },

src/drafts/Tooltip/Tooltip.playground.stories.tsx

@@ -18,10 +18,10 @@ export default {
 export const Playground: StoryFn = args => {
   // this is a hack to remove the `type` prop from the args because for this example type label is not a valid choice and violates accessibility
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  const {type, ...rest} = args
+  const {text, type, ...rest} = args
   return (
     <Box sx={{p: 6}}>
-      <Tooltip type="description" {...rest}>
+      <Tooltip text={text} type="description" {...rest}>
         <Button>Delete</Button>
       </Tooltip>
     </Box>

src/drafts/Tooltip/Tooltip.tsx

@@ -126,7 +126,7 @@ type TooltipDirection = 'nw' | 'n' | 'ne' | 'e' | 'se' | 's' | 'sw' | 'w'
 export type TooltipProps = React.PropsWithChildren<
   {
     direction?: TooltipDirection
-    text?: string
+    text: string
     type?: 'label' | 'description'
   } & SxProp &
     ComponentProps<typeof StyledTooltip>

src/drafts/Tooltip/__tests__/Tooltip.test.tsx

@@ -5,7 +5,7 @@ import {render as HTMLRender} from '@testing-library/react'
 import theme from '../../../theme'
 import {Button, ActionMenu, ActionList, ThemeProvider, SSRProvider, BaseStyles} from '../../../'
 
-const TooltipComponent = (props: TooltipProps) => (
+const TooltipComponent = (props: Omit<TooltipProps, 'text'> & {text?: string}) => (
   <Tooltip text="Tooltip text" {...props}>
     <Button>Button Text</Button>
   </Tooltip>