From ccbd46ba9d438a04971ece6d301a489c75297bf3 Mon Sep 17 00:00:00 2001
From: Eric Olkowski <thatblindgeye@gmail.com>
Date: Fri, 16 Aug 2024 10:51:52 -0400
Subject: [PATCH] Added test for 1 valid and 1 invalid property

---
 .../card-updated-clickable-markup.test.ts       | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/packages/eslint-plugin-pf-codemods/src/rules/v6/cardUpdatedClickableMarkup/card-updated-clickable-markup.test.ts b/packages/eslint-plugin-pf-codemods/src/rules/v6/cardUpdatedClickableMarkup/card-updated-clickable-markup.test.ts
index d61619ae..01765df2 100644
--- a/packages/eslint-plugin-pf-codemods/src/rules/v6/cardUpdatedClickableMarkup/card-updated-clickable-markup.test.ts
+++ b/packages/eslint-plugin-pf-codemods/src/rules/v6/cardUpdatedClickableMarkup/card-updated-clickable-markup.test.ts
@@ -20,6 +20,7 @@ ruleTester.run("card-updated-clickable-markup", rule, {
     },
   ],
   invalid: [
+    // No invalid properties
     {
       code: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: "#"}} /></Card>`,
       output: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: "#"}} /></Card>`,
@@ -30,6 +31,19 @@ ruleTester.run("card-updated-clickable-markup", rule, {
         },
       ],
     },
+    // 1 valid property + 1 invalid property
+    {
+      code: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: '#', selectableActionId: 'Id'}} /></Card>`,
+      output: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: '#'}} /></Card>`,
+      errors: [
+        {
+          message:
+            "The markup for clickable-only cards has been updated.Additionally, the `selectableActions.selectableActionId` and `selectableActions.name` props are no longer necessary to pass to CardHeader for clickable-only cards.",
+          type: "JSXElement",
+        },
+      ],
+    },
+    // 2 invalid properties
     {
       code: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{name: 'Test', selectableActionId: 'Id'}} /></Card>`,
       output: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{}} /></Card>`,
@@ -41,6 +55,7 @@ ruleTester.run("card-updated-clickable-markup", rule, {
         },
       ],
     },
+    // 1 valid property + 2 invalid properties
     {
       code: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: "#", name: 'Test', selectableActionId: 'Id'}} /></Card>`,
       output: `import { Card, CardHeader } from '@patternfly/react-core'; <Card isClickable><CardHeader selectableActions={{to: "#"}} /></Card>`,
@@ -53,6 +68,7 @@ ruleTester.run("card-updated-clickable-markup", rule, {
         },
       ],
     },
+    // Passed as a variable reference
     {
       code: `import { Card, CardHeader } from '@patternfly/react-core'; const obj = {name: 'Test', selectableActionId: 'Id'}; <Card isClickable><CardHeader selectableActions={obj} /></Card>`,
       output: `import { Card, CardHeader } from '@patternfly/react-core'; const obj = {}; <Card isClickable><CardHeader selectableActions={obj} /></Card>`,
@@ -64,6 +80,7 @@ ruleTester.run("card-updated-clickable-markup", rule, {
         },
       ],
     },
+    // Passed as a variable reference with 2 valid and 2 invalid properties
     {
       code: `import { Card, CardHeader } from '@patternfly/react-core'; const obj = {to: "#", name: 'Test', extra: "thing", selectableActionId: 'Id'}; <Card isClickable><CardHeader selectableActions={obj} /></Card>`,
       output: `import { Card, CardHeader } from '@patternfly/react-core'; const obj = {to: "#", extra: "thing"}; <Card isClickable><CardHeader selectableActions={obj} /></Card>`,