update the entry point and add additional fixes

Author: broccolinisoup

src/rules/__tests__/use-next-tooltip.test.js

@@ -13,30 +13,41 @@ const ruleTester = new RuleTester({
 
 ruleTester.run('use-next-tooltip', rule, {
   valid: [
-    `import {Tooltip} from '@primer/react/next';`,
+    `import {Tooltip} from '@primer/react/experimental';`,
     `import {UnderlineNav, Button} from '@primer/react';
-   import {Tooltip} from '@primer/react/next';`,
+   import {Tooltip} from '@primer/react/experimental';`,
     `import {UnderlineNav, Button} from '@primer/react';
-   import {Tooltip, SelectPanel} from '@primer/react/next';`,
+   import {Tooltip, SelectPanel} from '@primer/react/experimental';`,
   ],
   invalid: [
     {
       code: `import {Tooltip} from '@primer/react';`,
       errors: [{messageId: 'useNextTooltip'}],
-      output: `import {Tooltip} from '@primer/react/next';`,
+      output: `import {Tooltip} from '@primer/react/experimental';`,
     },
     {
       code: `import {Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text"><Button>Button</Button></Tooltip>`,
       errors: [{messageId: 'useNextTooltip'}, {messageId: 'useTextProp'}],
-      output: `import {Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
+      output: `import {Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
     },
     {
       code: `import {ActionList, ActionMenu, Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text"><Button>Button</Button></Tooltip>`,
       errors: [
         {messageId: 'useNextTooltip', line: 1},
         {messageId: 'useTextProp', line: 2},
       ],
-      output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
+      output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
+    },
+    {
+      code: `import {ActionList, ActionMenu, Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text" noDelay={true} wrap={true} align="left"><Button>Button</Button></Tooltip>`,
+      errors: [
+        {messageId: 'useNextTooltip', line: 1},
+        {messageId: 'useTextProp', line: 2},
+        {messageId: 'noDelayRemoved', line: 2},
+        {messageId: 'wrapRemoved', line: 2},
+        {messageId: 'alignRemoved', line: 2},
+      ],
+      output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text" ><Button>Button</Button></Tooltip>`,
     },
   ],
 })

src/rules/use-next-tooltip.js

@@ -4,15 +4,18 @@ module.exports = {
   meta: {
     type: 'suggestion',
     docs: {
-      description: 'recommends the use of @primer/react/next Tooltip component',
+      description: 'recommends the use of @primer/react/experimental Tooltip component',
       category: 'Best Practices',
       recommended: true,
     },
     fixable: true,
     schema: [],
     messages: {
-      useNextTooltip: 'Please use @primer/react/next Tooltip component that has accessibility improvements',
+      useNextTooltip: 'Please use @primer/react/experimental Tooltip component that has accessibility improvements',
       useTextProp: 'Please use the text prop instead of aria-label',
+      noDelayRemoved: 'noDelay prop is removed. Tooltip now has no delay by default',
+      wrapRemoved: 'wrap prop is removed. Tooltip now wraps by default',
+      alignRemoved: 'align prop is removed. Please use the direction prop instead.',
     },
   },
   create(context) {
@@ -37,7 +40,7 @@ module.exports = {
           fix(fixer) {
             // If Tooltip is the only import, replace the whole import statement
             if (!hasOtherImports) {
-              return fixer.replaceText(node.source, `'@primer/react/next'`)
+              return fixer.replaceText(node.source, `'@primer/react/experimental'`)
             } else {
               // Otherwise, remove Tooltip from the import statement and add a new import statement with the correct path
               const tooltipSpecifier = node.specifiers.find(
@@ -49,7 +52,7 @@ module.exports = {
                 // fixer.remove(tooltipSpecifier),
                 fixer.insertTextAfterRange(
                   [node.range[1], node.range[1]],
-                  `\nimport {Tooltip} from '@primer/react/next';`,
+                  `\nimport {Tooltip} from '@primer/react/experimental';`,
                 ),
               ]
             }
@@ -66,6 +69,33 @@ module.exports = {
             },
           })
         }
+        if (node.name.name === 'noDelay') {
+          context.report({
+            node,
+            messageId: 'noDelayRemoved',
+            fix(fixer) {
+              return fixer.remove(node)
+            },
+          })
+        }
+        if (node.name.name === 'wrap') {
+          context.report({
+            node,
+            messageId: 'wrapRemoved',
+            fix(fixer) {
+              return fixer.remove(node)
+            },
+          })
+        }
+        if (node.name.name === 'align') {
+          context.report({
+            node,
+            messageId: 'alignRemoved',
+            fix(fixer) {
+              return fixer.remove(node)
+            },
+          })
+        }
       },
     }
   },