Merge pull request #131 from microsoft/user/aubreyquinn/testCoverage

increased unit test coverage for the has tool tip parent function

Author: aubreyquinn

lib/util/hasTooltipParent.js

@@ -0,0 +1,24 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+import { elementType } from "jsx-ast-utils";
+import { TSESLint } from "@typescript-eslint/utils"; // Assuming context comes from TSESLint
+import { JSXOpeningElement } from "estree-jsx";
+
+const hasToolTipParent = (context: TSESLint.RuleContext<string, unknown[]>): boolean => {
+    const ancestors = context.getAncestors();
+
+    if (!ancestors || ancestors.length === 0) {
+        return false;
+    }
+
+    return ancestors.some(
+        item =>
+            item.type === "JSXElement" &&
+            item.openingElement &&
+            item.openingElement.type === "JSXOpeningElement" &&
+            elementType(item.openingElement as unknown as JSXOpeningElement) === "Tooltip"
+    );
+};
+
+export { hasToolTipParent };

lib/util/hasTooltipParent.ts

@@ -0,0 +1,70 @@
+import { hasToolTipParent } from "../../../../lib/util/hasTooltipParent";
+import { TSESLint } from "@typescript-eslint/utils";
+
+// Mocking the elementType utility
+jest.mock("jsx-ast-utils", () => ({
+    elementType: (openingElement: any) => openingElement.name.name
+}));
+
+describe("hasToolTipParent", () => {
+    let mockContext: TSESLint.RuleContext<string, unknown[]>;
+
+    beforeEach(() => {
+        mockContext = {
+            getAncestors: jest.fn()
+        } as unknown as TSESLint.RuleContext<string, unknown[]>;
+    });
+
+    test("should return false when there are no ancestors", () => {
+        (mockContext.getAncestors as jest.Mock).mockReturnValue([]);
+
+        const result = hasToolTipParent(mockContext);
+        expect(result).toBe(false);
+    });
+
+    test("should return false when no Tooltip ancestor exists", () => {
+        const mockAncestors = [
+            {
+                type: "JSXElement",
+                openingElement: {
+                    type: "JSXOpeningElement",
+                    name: { name: "Button" } // Not a Tooltip
+                }
+            },
+            {
+                type: "JSXElement",
+                openingElement: {
+                    type: "JSXOpeningElement",
+                    name: { name: "Div" } // Not a Tooltip
+                }
+            }
+        ];
+        (mockContext.getAncestors as jest.Mock).mockReturnValue(mockAncestors);
+
+        const result = hasToolTipParent(mockContext);
+        expect(result).toBe(false);
+    });
+
+    test("should return true when a Tooltip ancestor exists", () => {
+        const mockAncestors = [
+            {
+                type: "JSXElement",
+                openingElement: {
+                    type: "JSXOpeningElement",
+                    name: { name: "Div" } // Not a Tooltip
+                }
+            },
+            {
+                type: "JSXElement",
+                openingElement: {
+                    type: "JSXOpeningElement",
+                    name: { name: "Tooltip" } // This is a Tooltip
+                }
+            }
+        ];
+        (mockContext.getAncestors as jest.Mock).mockReturnValue(mockAncestors);
+
+        const result = hasToolTipParent(mockContext);
+        expect(result).toBe(true);
+    });
+});