Uncomment test cases after the recent merge.

Author: sidhsharma1992

tests/lib/rules/buttons/menu-button-needs-labelling.test.ts

@@ -15,19 +15,12 @@ import rule from "../../../../lib/rules/buttons/menu-button-needs-labelling";
 
 ruleTester.run("menu-button-needs-labelling", rule as unknown as Rule.RuleModule, {
     valid: [
-        // MenuButton with aria-label
         `<MenuButton aria-label="Menu options" />`,
-        // MenuButton with text content
         `<MenuButton>Options</MenuButton>`,
-        // MenuButton with aria-labelledby that references existing element
         `<><Label id="menu-label">Menu</Label><MenuButton aria-labelledby="menu-label" /></>`,
-        // MenuButton wrapped in Tooltip
-        `<Tooltip content="Menu options" relationship="label"><MenuButton /></Tooltip>`
-        // TODO: Uncomment when hasLabeledChild is implemented
-        // MenuButton with labeled child
-        // `<MenuButton><img alt="Menu icon" /></MenuButton>`,
-        // MenuButton with Icon child
-        // `<MenuButton><MenuIcon /></MenuButton>`
+        `<Tooltip content="Menu options" relationship="label"><MenuButton /></Tooltip>`,
+        `<MenuButton><img alt="Menu icon" /></MenuButton>`,
+        `<MenuButton><MenuIcon /></MenuButton>`
     ],
     invalid: [
         {

tests/lib/rules/card-needs-accessible-name.test.ts

@@ -15,15 +15,10 @@ import rule from "../../../lib/rules/card-needs-accessible-name";
 
 ruleTester.run("card-needs-accessible-name", rule as unknown as Rule.RuleModule, {
     valid: [
-        // Card with aria-label
         `<Card aria-label="Product details" />`,
-        // Card with aria-labelledby that references existing element
-        `<><Label id="card-label">Product</Label><Card aria-labelledby="card-label" /></>`
-        // TODO: Uncomment when hasLabeledChild is implemented
-        // Card with labeled child
-        // `<Card><img alt="Product image" /></Card>`,
-        // Card with Icon child
-        // `<Card><ProductIcon /></Card>`
+        `<><Label id="card-label">Product</Label><Card aria-labelledby="card-label" /></>`,
+        `<Card><img alt="Product image" /></Card>`,
+        `<Card><ProductIcon /></Card>`
     ],
     invalid: [
         {

tests/lib/rules/infolabel-needs-labelling.test.ts

@@ -15,21 +15,14 @@ import rule from "../../../lib/rules/infolabel-needs-labelling";
 
 ruleTester.run("infolabel-needs-labelling", rule as unknown as Rule.RuleModule, {
     valid: [
-        // InfoLabel with aria-label
         `<InfoLabel aria-label="Additional information" />`,
-        // InfoLabel with text content
         `<InfoLabel>Help text</InfoLabel>`,
         `<InfoLabel>?</InfoLabel>`,
         `<InfoLabel>ℹ️</InfoLabel>`,
-        // InfoLabel with aria-labelledby that references existing element
         `<><Label id="info-label">Information</Label><InfoLabel aria-labelledby="info-label" /></>`,
-        // InfoLabel wrapped in Tooltip
-        `<Tooltip content="Help information" relationship="label"><InfoLabel /></Tooltip>`
-        // TODO: Uncomment when hasLabeledChild is implemented
-        // InfoLabel with labeled child
-        // `<InfoLabel><img alt="Help icon" /></InfoLabel>`,
-        // InfoLabel with Icon child
-        // `<InfoLabel><InfoIcon /></InfoLabel>`
+        `<Tooltip content="Help information" relationship="label"><InfoLabel /></Tooltip>`,
+        `<InfoLabel><img alt="Help icon" /></InfoLabel>`,
+        `<InfoLabel><InfoIcon /></InfoLabel>`
     ],
     invalid: [
         {
@@ -49,9 +42,16 @@ ruleTester.run("infolabel-needs-labelling", rule as unknown as Rule.RuleModule,
             errors: [{ messageId: "infoLabelNeedsLabelling" }]
         },
         {
-            // aria-labelledby references ID that doesn't exist
             code: `<><Label id="wrong-id">Information</Label><InfoLabel aria-labelledby="info-label" /></>`,
             errors: [{ messageId: "infoLabelNeedsLabelling" }]
+        },
+        {
+            code: `<InfoLabel><img /></InfoLabel>`,
+            errors: [{ messageId: "infoLabelNeedsLabelling" }]
+        },
+        {
+            code: `<InfoLabel><img alt="" /></InfoLabel>`,
+            errors: [{ messageId: "infoLabelNeedsLabelling" }]
         }
     ]
 });