From 8f72511b8b3b0642684b472b241f886d8c2a1036 Mon Sep 17 00:00:00 2001
From: Eric Olkowski <thatblindgeye@gmail.com>
Date: Thu, 23 May 2024 08:22:25 -0400
Subject: [PATCH] PR feedback

---
 .../emptyState-nonExported-components.test.ts | 28 +++++++++++++++++++
 .../emptyState-nonExported-components.ts      |  5 +---
 2 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.test.ts b/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.test.ts
index bd2aa5084..745e49bca 100644
--- a/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.test.ts
+++ b/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.test.ts
@@ -117,5 +117,33 @@ ruleTester.run("emptyState-nonExported-components", rule, {
         },
       ],
     },
+    {
+      code: `import { EmptyStateHeader } from '@patternfly/react-core';export default EmptyStateHeader;`,
+      output: `import { EmptyStateHeader } from '@patternfly/react-core';`,
+      errors: [
+        {
+          message: `EmptyStateHeader is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports.`,
+          type: "ImportSpecifier",
+        },
+        {
+          message: `EmptyStateHeader is no longer exported by PatternFly.`,
+          type: "ExportDefaultDeclaration",
+        },
+      ],
+    },
+    {
+      code: `import { EmptyStateHeader as CustomHeader } from '@patternfly/react-core';export default CustomHeader;`,
+      output: `import { EmptyStateHeader as CustomHeader } from '@patternfly/react-core';`,
+      errors: [
+        {
+          message: `EmptyStateHeader is no longer exported by PatternFly. This rule will not fix any imports, as our cleanup rule handles removal of unused imports.`,
+          type: "ImportSpecifier",
+        },
+        {
+          message: `CustomHeader is no longer exported by PatternFly.`,
+          type: "ExportDefaultDeclaration",
+        },
+      ],
+    },
   ],
 });
diff --git a/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.ts b/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.ts
index 9b4a40bf4..8a197ac84 100644
--- a/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.ts
+++ b/packages/eslint-plugin-pf-codemods/src/rules/v6/emptyStateNonExportedComponents/emptyState-nonExported-components.ts
@@ -1,7 +1,6 @@
 import { Rule } from "eslint";
 import {
   ImportSpecifier,
-  ExportSpecifier,
   ExportNamedDeclaration,
   ExportDefaultDeclaration,
 } from "estree-jsx";
@@ -97,9 +96,7 @@ module.exports = {
         const exportName =
           node.declaration.type === "Identifier" && node.declaration.name;
         const isEmptyStateDefaultExport = emptyStateImports.some(
-          (imp) =>
-            node.declaration.type === "Identifier" &&
-            imp.local.name === exportName
+          (imp) => imp.local.name === exportName
         );
 
         if (isEmptyStateDefaultExport) {