Added tests to increase coverage.

Author: sidhsharma1992

tests/lib/rules/utils/hasLabelledChilImage.test.ts

@@ -156,3 +156,157 @@ describe("hasLabelledChildImage", () => {
         expect(hasLabelledChildImage(node)).toBe(false);
     });
 });
+
+describe("hasLabelledChildImage - missing coverage", () => {
+    it("covers line 34 - handles non-JSXElement children", () => {
+        // This test covers the case where children exist but are not JSXElement types
+        const mockTextChild: TSESTree.JSXText = {
+            type: AST_NODE_TYPES.JSXText,
+            value: "Some text content",
+            raw: "Some text content",
+            ...createMockLocRange()
+        };
+
+        const mockExpressionChild: TSESTree.JSXExpressionContainer = {
+            type: AST_NODE_TYPES.JSXExpressionContainer,
+            expression: {
+                type: AST_NODE_TYPES.Literal,
+                value: "expression",
+                raw: '"expression"',
+                ...createMockLocRange()
+            },
+            ...createMockLocRange()
+        };
+
+        const node: TSESTree.JSXElement = {
+            type: AST_NODE_TYPES.JSXElement,
+            openingElement: {
+                type: AST_NODE_TYPES.JSXOpeningElement,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "Container", ...createMockLocRange() },
+                attributes: [],
+                selfClosing: false,
+                ...createMockLocRange()
+            },
+            closingElement: null,
+            children: [mockTextChild, mockExpressionChild], // Non-JSXElement children
+            ...createMockLocRange()
+        };
+
+        // This should execute line 34 and return false since no JSXElement children are found
+        expect(hasLabelledChildImage(node)).toBe(false);
+    });
+
+    it("covers line 34 - JSXElement child with non-image component name", () => {
+        // This covers the case where we have JSXElement children but they're not image components
+        const mockDivChild: TSESTree.JSXElement = {
+            type: AST_NODE_TYPES.JSXElement,
+            openingElement: {
+                type: AST_NODE_TYPES.JSXOpeningElement,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "div", ...createMockLocRange() }, // Not an image component
+                attributes: [
+                    {
+                        type: AST_NODE_TYPES.JSXAttribute,
+                        name: { type: AST_NODE_TYPES.JSXIdentifier, name: "alt", ...createMockLocRange() },
+                        value: { type: AST_NODE_TYPES.Literal, value: "description", raw: '"description"', ...createMockLocRange() },
+                        ...createMockLocRange()
+                    }
+                ],
+                selfClosing: false,
+                ...createMockLocRange()
+            },
+            closingElement: null,
+            children: [],
+            ...createMockLocRange()
+        };
+
+        const node: TSESTree.JSXElement = {
+            type: AST_NODE_TYPES.JSXElement,
+            openingElement: {
+                type: AST_NODE_TYPES.JSXOpeningElement,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "Container", ...createMockLocRange() },
+                attributes: [],
+                selfClosing: false,
+                ...createMockLocRange()
+            },
+            closingElement: null,
+            children: [mockDivChild],
+            ...createMockLocRange()
+        };
+
+        // This should execute line 34 and return false since child is not an image component
+        expect(hasLabelledChildImage(node)).toBe(false);
+    });
+});
+
+describe("isImageHidden - missing coverage for aria-label handling", () => {
+    it("covers lines 70-72 - returns false when aria-label has non-empty value", () => {
+        // This covers the case where aria-label is present and has a value (lines 70-72)
+        const attributes: TSESTree.JSXOpeningElement["attributes"] = [
+            {
+                type: AST_NODE_TYPES.JSXAttribute,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "aria-label", ...createMockLocRange() },
+                value: {
+                    type: AST_NODE_TYPES.Literal,
+                    value: "Descriptive label",
+                    raw: '"Descriptive label"',
+                    ...createMockLocRange()
+                },
+                ...createMockLocRange()
+            }
+        ];
+
+        // Should return false because aria-label has a value, meaning image is not hidden
+        expect(isImageHidden(attributes)).toBe(false);
+    });
+
+    it("covers lines 70-72 - returns false when aria-labelledby has non-empty value", () => {
+        // This covers the case where aria-labelledby is present and has a value
+        const attributes: TSESTree.JSXOpeningElement["attributes"] = [
+            {
+                type: AST_NODE_TYPES.JSXAttribute,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "aria-labelledby", ...createMockLocRange() },
+                value: {
+                    type: AST_NODE_TYPES.Literal,
+                    value: "label-id",
+                    raw: '"label-id"',
+                    ...createMockLocRange()
+                },
+                ...createMockLocRange()
+            }
+        ];
+
+        // Should return false because aria-labelledby has a value, meaning image is not hidden
+        expect(isImageHidden(attributes)).toBe(false);
+    });
+
+    it("handles case with both aria-label and aria-labelledby present", () => {
+        // Additional test to ensure proper handling when both attributes are present
+        const attributes: TSESTree.JSXOpeningElement["attributes"] = [
+            {
+                type: AST_NODE_TYPES.JSXAttribute,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "aria-label", ...createMockLocRange() },
+                value: {
+                    type: AST_NODE_TYPES.Literal,
+                    value: "Primary label",
+                    raw: '"Primary label"',
+                    ...createMockLocRange()
+                },
+                ...createMockLocRange()
+            },
+            {
+                type: AST_NODE_TYPES.JSXAttribute,
+                name: { type: AST_NODE_TYPES.JSXIdentifier, name: "aria-labelledby", ...createMockLocRange() },
+                value: {
+                    type: AST_NODE_TYPES.Literal,
+                    value: "secondary-label",
+                    raw: '"secondary-label"',
+                    ...createMockLocRange()
+                },
+                ...createMockLocRange()
+            }
+        ];
+
+        // Should return false because aria-label has a value
+        expect(isImageHidden(attributes)).toBe(false);
+    });
+});