@dxc-technology/halstack-react 0.0.0-cb610df → 0.0.0-cc01d9c

package/test/FileInput.test.js

@@ -0,0 +1,201 @@
+import React from "react";
+import { render, fireEvent, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+
+import DxcFileInput from "../src/file-input/FileInput";
+
+describe("FileInput component tests", () => {
+  const file1 = new File(["file1"], "file1.png", { type: "image/png" });
+  const file2 = new File(["file2"], "file2.txt", {
+    type: "text/plain",
+  });
+
+  const files = [
+    {
+      error: null,
+      preview: null,
+      file: file1,
+    },
+    {
+      error: "Error message",
+      preview: null,
+      file: file2,
+    },
+  ];
+
+  test("Renders with correct labels and helper text in file mode", () => {
+    const { getByText } = render(<DxcFileInput label="File input label" helperText="File input helper text" />);
+    expect(getByText("File input label")).toBeTruthy();
+    expect(getByText("File input helper text")).toBeTruthy();
+  });
+  test("Renders with correct button label in file mode", () => {
+    const { getByText } = render(<DxcFileInput label="File input label" helperText="File input helper text" />);
+    expect(getByText("Select files")).toBeTruthy();
+  });
+  test("Renders with correct labels in filedrop mode", () => {
+    const { getByText } = render(
+      <DxcFileInput label="File input label" helperText="File input helper text" mode="filedrop" />
+    );
+    expect(getByText("Select")).toBeTruthy();
+    expect(getByText("or drop files")).toBeTruthy();
+  });
+  test("Renders with correct labels in dropzone mode", () => {
+    const { getByText } = render(
+      <DxcFileInput label="File input label" helperText="File input helper text" mode="dropzone" />
+    );
+    expect(getByText("Select")).toBeTruthy();
+    expect(getByText("or drop files")).toBeTruthy();
+  });
+  test("Disabled file input", () => {
+    const { getByText, getByRole } = render(
+      <DxcFileInput label="File input label" helperText="File input helper text" disabled />
+    );
+    expect(getByText("Select files")).toBeTruthy();
+    const button = getByRole("button");
+    expect(button.disabled).toBeTruthy();
+  });
+  test("Renders file items passed in value when multiple file input", () => {
+    const callbackFile = jest.fn();
+    const { getByText } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        value={files}
+        callbackFile={callbackFile}
+      />
+    );
+    expect(getByText("file1.png")).toBeTruthy();
+    expect(getByText("file2.txt")).toBeTruthy();
+    expect(getByText("Error message")).toBeTruthy();
+  });
+  test("Renders file items when single file input", () => {
+    const callbackFile = jest.fn();
+    const { getByText } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        multiple={false}
+        value={files}
+        callbackFile={callbackFile}
+      />
+    );
+    expect(getByText("file1.png")).toBeTruthy();
+    expect(getByText("file2.txt")).toBeTruthy();
+    expect(getByText("Error message")).toBeTruthy();
+  });
+  test("Add file item when single file input", async () => {
+    const newFile = new File(["newFile"], "newFile.pdf", { type: "pdf" });
+    const callbackFile = jest.fn();
+    const { getByText, getByLabelText } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        value={files}
+        callbackFile={callbackFile}
+        multiple={false}
+      />
+    );
+    expect(getByText("file1.png")).toBeTruthy();
+    expect(getByText("file2.txt")).toBeTruthy();
+    expect(getByText("Error message")).toBeTruthy();
+    const inputFile = getByLabelText("File input label", { hidden: true });
+    fireEvent.change(inputFile, { target: { files: [newFile] } });
+    await waitFor(() => {
+      expect(callbackFile).toHaveBeenCalledWith([
+        {
+          error: undefined,
+          file: newFile,
+          preview: (
+            <svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="currentColor">
+              <path fill="none" d="M0 0h24v24H0V0z" />
+              <path d="M14 2H6c-1.1 0-1.99.9-1.99 2L4 20c0 1.1.89 2 1.99 2H18c1.1 0 2-.9 2-2V8l-6-6zM6 20V4h7v5h5v11H6z" />
+            </svg>
+          ),
+        },
+      ]);
+    });
+  });
+  test("Renders file items and delete one file", async () => {
+    const callbackFile = jest.fn();
+    const { getByText, getAllByRole } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        value={files}
+        callbackFile={callbackFile}
+      />
+    );
+    expect(getByText("file1.png")).toBeTruthy();
+    expect(getByText("file2.txt")).toBeTruthy();
+    expect(getByText("Error message")).toBeTruthy();
+    const removeBtn = getAllByRole("button")[1];
+    userEvent.click(removeBtn);
+    await waitFor(() => {
+      expect(callbackFile).toHaveBeenCalledWith([{ error: "Error message", file: file2, preview: null }]);
+    });
+  });
+
+  test("CallbackFile is correctly called", async () => {
+    const newFile = new File(["newFile"], "newFile.pdf", { type: "pdf" });
+    const callbackFile = jest.fn();
+    const { getByLabelText } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        value={files}
+        callbackFile={callbackFile}
+      />
+    );
+    const inputFile = getByLabelText("File input label", { hidden: true });
+    fireEvent.change(inputFile, { target: { files: [newFile] } });
+    await waitFor(() => {
+      expect(callbackFile).toHaveBeenCalledWith([
+        { error: null, preview: null, file: file1 },
+        { error: "Error message", preview: null, file: file2 },
+        {
+          error: undefined,
+          file: newFile,
+          preview: (
+            <svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="currentColor">
+              <path fill="none" d="M0 0h24v24H0V0z" />
+              <path d="M14 2H6c-1.1 0-1.99.9-1.99 2L4 20c0 1.1.89 2 1.99 2H18c1.1 0 2-.9 2-2V8l-6-6zM6 20V4h7v5h5v11H6z" />
+            </svg>
+          ),
+        },
+      ]);
+    });
+  });
+
+  test("Callback called correctly when a file size does not comply minSize or maxSize", async () => {
+    const newFile = new File(["newFile"], "newFile.pdf", { type: "pdf" });
+    const callbackFile = jest.fn();
+    const { getByLabelText } = render(
+      <DxcFileInput
+        label="File input label"
+        helperText="File input helper text"
+        minSize={1000}
+        maxSize={20000}
+        value={files}
+        callbackFile={callbackFile}
+      />
+    );
+    const inputFile = getByLabelText("File input label", { hidden: true });
+    fireEvent.change(inputFile, { target: { files: [newFile] } });
+    await waitFor(() => {
+      expect(callbackFile).toHaveBeenCalledWith([
+        { error: null, preview: null, file: file1 },
+        { error: "Error message", preview: null, file: file2 },
+        {
+          error: "File size must be greater than min size.",
+          file: newFile,
+          preview: (
+            <svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="currentColor">
+              <path fill="none" d="M0 0h24v24H0V0z" />
+              <path d="M14 2H6c-1.1 0-1.99.9-1.99 2L4 20c0 1.1.89 2 1.99 2H18c1.1 0 2-.9 2-2V8l-6-6zM6 20V4h7v5h5v11H6z" />
+            </svg>
+          ),
+        },
+      ]);
+    });
+  });
+});

package/test/Footer.test.js

@@ -5,13 +5,8 @@ import icon from "../../app/src/images/linkedin.svg";
 
 describe("Footer component tests", () => {
   test("Footer renders with default logo", () => {
-    const { getByRole } = render(<DxcFooter></DxcFooter>);
-    expect(getByRole("img")).toBeTruthy();
-  });
-
-  test("Footer renders with logo", () => {
-    const { getByRole } = render(<DxcFooter logoSrc={icon}></DxcFooter>);
-    expect(getByRole("img")).toBeTruthy();
+    const { getByTitle } = render(<DxcFooter></DxcFooter>);
+    expect(getByTitle("DXC Logo")).toBeTruthy();
   });
 
   test("Footer renders with social links", () => {

package/test/Header.test.js

@@ -5,25 +5,20 @@ import icon from "../../app/src/images/linkedin.svg";
 
 describe("Header component tests", () => {
   test("Header renders with default logo", () => {
-    const { getByRole } = render(<DxcHeader></DxcHeader>);
-    expect(getByRole("img")).toBeTruthy();
-  });
-
-  test("Header renders with logo", () => {
-    const { getByRole } = render(<DxcHeader logoSrc={icon}></DxcHeader>);
-    expect(getByRole("img")).toBeTruthy();
+    const { getByTitle } = render(<DxcHeader></DxcHeader>);
+    expect(getByTitle("DXC Logo")).toBeTruthy();
   });
 
   test("Call correct function on logo click", () => {
     const onClick = jest.fn();
-    const { getByRole } = render(<DxcHeader onClick={onClick}></DxcHeader>);
-    const logo = getByRole("img");
+    const { getByTitle } = render(<DxcHeader onClick={onClick}></DxcHeader>);
+    const logo = getByTitle("DXC Logo");
     fireEvent.click(logo);
     expect(onClick).toHaveBeenCalled();
   });
 
   test("Header renders with correct children", () => {
-    //We need to force the offsetWidth value
+    // We need to force the offsetWidth value
     Object.defineProperty(HTMLElement.prototype, "offsetWidth", { configurable: true, value: 1024 });
 
     const { getByText } = render(<DxcHeader content={<p>header-child-text</p>}></DxcHeader>);

package/test/Heading.test.js

@@ -3,33 +3,81 @@ import { render } from "@testing-library/react";
 import DxcHeading from "../src/heading/Heading";
 
 describe("Heading component tests", () => {
-  test("Footer renders with default level", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test"></DxcHeading>);
+  test("Heading renders with default level", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test"></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 1 })).toBeTruthy();
   });
 
-  test("Footer renders with level 1", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test" level={1}></DxcHeading>);
+  test("Heading renders with level 1", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={1}></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 1 })).toBeTruthy();
   });
 
-  test("Footer renders with level 2", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test" level={2}></DxcHeading>);
+  test("Heading renders with level 2", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={2}></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 2 })).toBeTruthy();
   });
 
-  test("Footer renders with level 3", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test" level={3}></DxcHeading>);
+  test("Heading renders with level 3", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={3}></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 3 })).toBeTruthy();
   });
 
-  test("Footer renders with level 4", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test" level={4}></DxcHeading>);
+  test("Heading renders with level 4", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={4}></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 4 })).toBeTruthy();
   });
 
-  test("Footer renders with level 5", () => {
-    const { getByText } = render(<DxcHeading text="my-heading-test" level={5}></DxcHeading>);
+  test("Heading renders with level 5", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={5}></DxcHeading>);
     expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 5 })).toBeTruthy();
+  });
+
+  test("Heading renders with default level and as h5", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" as="h5"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 5 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 1 and as h5", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={1} as="h5"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 5 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 2 and as h4", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={2} as="h4"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 4 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 3 and as h2", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={3} as="h2"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 2 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 4 and as h3", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={4} as="h3"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 3 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 5 as h4", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={5} as="h4"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 4 })).toBeTruthy();
+  });
+
+  test("Heading renders with level 5 and invalid as", () => {
+    const { getByText, getByRole } = render(<DxcHeading text="my-heading-test" level={5} as="h4afdssf"></DxcHeading>);
+    expect(getByText("my-heading-test")).toBeTruthy();
+    expect(getByRole("heading", { level: 5 })).toBeTruthy();
   });
 });

package/test/InputText.test.js

@@ -1,5 +1,7 @@
 import React from "react";
 import { render, fireEvent, waitFor, screen, act, waitForElementToBeRemoved } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+
 import icon from "../../app/src/images/invision.svg";
 
 import DxcInputText from "../src/input-text/InputText";
@@ -18,6 +20,7 @@ jest.mock("popper.js", () => {
       return {
         destroy: () => {},
         scheduleUpdate: () => {},
+        update: () => {},
       };
     }
   };
@@ -26,7 +29,6 @@ jest.mock("popper.js", () => {
 describe("InputText component tests", () => {
   test("Input renders with correct label", () => {
     const { getByText } = render(<DxcInputText label="Input label" />);
-
     expect(getByText("Input label")).toBeTruthy();
   });
 
@@ -34,7 +36,7 @@ describe("InputText component tests", () => {
     const onChange = jest.fn();
     const { getByRole } = render(<DxcInputText label="Input label" onChange={onChange} />);
     const input = getByRole("textbox");
-    fireEvent.change(input, { target: { value: "20000" } });
+    userEvent.type(input, "20000");
     expect(onChange).toHaveBeenCalled();
     expect(onChange).toHaveBeenCalledWith("20000");
   });
@@ -44,7 +46,7 @@ describe("InputText component tests", () => {
 
     const { getByRole } = render(<DxcInputText label="Input label" onChange={onChange} onBlur={onBlur} />);
     const input = getByRole("textbox");
-    fireEvent.change(input, { target: { value: "Test value" } });
+    userEvent.type(input, "Test value");
     fireEvent.blur(input);
     expect(onBlur).toHaveBeenCalled();
     expect(onBlur).toHaveBeenCalledWith("Test value");
@@ -54,7 +56,7 @@ describe("InputText component tests", () => {
     const onChange = jest.fn();
     const { getByRole } = render(<DxcInputText label="Input label" onChange={onChange} />);
     const input = getByRole("textbox");
-    fireEvent.change(input, { target: { value: "20000" } });
+    userEvent.type(input, "20000");
     expect(onChange).toHaveBeenCalled();
     expect(onChange).toHaveBeenCalledWith("20000");
     expect(input.value).toBe("20000");
@@ -67,9 +69,8 @@ describe("InputText component tests", () => {
       <DxcInputText label="Input label" value="Test value" onChange={onChange} onBlur={onBlur} />
     );
     const input = getByRole("textbox");
-    fireEvent.change(input, { target: { value: "20000" } });
+    userEvent.type(input, "20000");
     expect(onChange).toHaveBeenCalled();
-    expect(onChange).toHaveBeenCalledWith("20000");
     expect(input.value).toBe("Test value");
     fireEvent.blur(input);
     expect(onBlur).toHaveBeenCalled();
@@ -79,14 +80,14 @@ describe("InputText component tests", () => {
     const onClick = jest.fn();
     const { getByRole } = render(<DxcInputText label="Input label" prefixIconSrc={icon} onClickPrefix={onClick} />);
     const prefixIcon = getByRole("img");
-    fireEvent.click(prefixIcon);
+    userEvent.click(prefixIcon);
     expect(onClick).toHaveBeenCalled();
   });
   test("Suffix icon onClick", () => {
     const onClick = jest.fn();
     const { getByRole } = render(<DxcInputText label="Input label" suffixIconSrc={icon} onClickSuffix={onClick} />);
     const suffixIcon = getByRole("img");
-    fireEvent.click(suffixIcon);
+    userEvent.click(suffixIcon);
     expect(onClick).toHaveBeenCalled();
   });
 });
@@ -98,9 +99,9 @@ describe("Autocomplete component Synchronous tests", () => {
       <DxcInputText label="Autocomplete Countries" autocompleteOptions={countries} onChange={onChangeAtocomplete} />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
+    fireEvent.focus(input);
     expect(screen.getByText("Andorra")).toBeTruthy();
-    fireEvent.change(screen.getByRole("textbox"), { target: { value: "Spain" } });
+    userEvent.type(input, "X");
     expect(screen.getByText("No matches found.")).toBeTruthy();
   });
 
@@ -110,7 +111,7 @@ describe("Autocomplete component Synchronous tests", () => {
       <DxcInputText label="Autocomplete Countries" autocompleteOptions={countries} onChange={onChangeAtocomplete} />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
+    fireEvent.focus(input);
     expect(screen.getByText("Andorra")).toBeTruthy();
   });
 
@@ -120,14 +121,14 @@ describe("Autocomplete component Synchronous tests", () => {
       <DxcInputText label="Autocomplete Countries" autocompleteOptions={countries} onChange={onChangeAtocomplete} />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
+    fireEvent.focus(input);
     expect(screen.getByText("Andorra")).toBeTruthy();
-    fireEvent.change(screen.getByRole("textbox"), { target: { value: "Alb" } });
+    userEvent.type(screen.getByRole("textbox"), "Alb");
     expect(onChangeAtocomplete).toHaveBeenCalled();
     expect(screen.getByText("Albania")).toBeTruthy();
     const suggestion1 = screen.getByText("Albania");
     act(() => {
-      fireEvent.click(suggestion1);
+      userEvent.click(suggestion1);
     });
 
     // expect(onChangeAtocomplete).toHaveBeenCalledWith("Albania");
@@ -144,19 +145,20 @@ describe("Autocomplete component Synchronous tests", () => {
         onChange={onChangeAtocomplete}
       />
     );
-    screen.getByRole("textbox").focus();
-
+    const input = screen.getByRole("textbox");
+    fireEvent.focus(input);
     expect(screen.getByText("Andorra")).toBeTruthy();
 
     expect(screen.getByText("Andorra")).toBeTruthy();
     const suggestion1 = screen.getByText("Andorra");
     act(() => {
-      fireEvent.click(suggestion1);
+      userEvent.click(suggestion1);
     });
 
     expect(onChangeAtocomplete).toHaveBeenCalledWith("Andorra");
   });
 });
+
 describe("InputText component Asynchronous Autocomplete tests", () => {
   test("Autocomplete Searching is shown", async () => {
     const callbackFunc = jest.fn(() => new Promise((resolve) => setTimeout(resolve(["Italy", "Spain"]), 1000)));
@@ -166,7 +168,7 @@ describe("InputText component Asynchronous Autocomplete tests", () => {
       <DxcInputText label="Autocomplete Countries" autocompleteOptions={callbackFunc} onChange={onChangeAtocomplete} />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
+    fireEvent.focus(input);
     await waitForElementToBeRemoved(() => screen.getByText(/Searching.../i));
     expect(screen.getByText("Italy")).toBeTruthy();
     expect(screen.getByText("Spain")).toBeTruthy();
@@ -179,18 +181,19 @@ describe("InputText component Asynchronous Autocomplete tests", () => {
       <DxcInputText label="Autocomplete Countries" onChange={onChangeAtocomplete} autocompleteOptions={callbackFunc} />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
-    fireEvent.change(input, { target: { value: "And" } });
+    fireEvent.focus(input);
+    userEvent.type(input, "And");
     await waitForElementToBeRemoved(() => screen.getByText(/Searching.../i));
     expect(screen.getByText("Italy")).toBeTruthy();
     expect(screen.getByText("Spain")).toBeTruthy();
     const suggestion1 = screen.getByText("Spain");
     act(() => {
-      fireEvent.click(suggestion1);
+      userEvent.click(suggestion1);
     });
     expect(onChangeAtocomplete).toHaveBeenCalledWith("Spain");
     expect(screen.getByRole("textbox").value).toBe("Spain");
   });
+
   test("Asynchronous CONTROLLED autocomplete tests", async () => {
     const callbackFunc = jest.fn(() => new Promise((resolve) => setTimeout(resolve(["Italy", "Spain"]), 3000)));
     const onChangeAtocomplete = jest.fn();
@@ -203,18 +206,19 @@ describe("InputText component Asynchronous Autocomplete tests", () => {
       />
     );
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
-    fireEvent.change(input, { target: { value: "And" } });
+    fireEvent.focus(input);
+    userEvent.type(input, "And");
     await waitForElementToBeRemoved(() => screen.getByText(/Searching.../i));
     expect(screen.getByText("Italy")).toBeTruthy();
     expect(screen.getByText("Spain")).toBeTruthy();
     const suggestion1 = screen.getByText("Spain");
     act(() => {
-      fireEvent.click(suggestion1);
+      userEvent.click(suggestion1);
     });
     expect(onChangeAtocomplete).toHaveBeenCalledWith("Spain");
     expect(screen.getByRole("textbox").value).toBe("test value");
   });
+
   test("Asynchronous autocomplete request failed", async () => {
     const errorCallbackFunc = jest.fn(() => new Promise((resolve, reject) => setTimeout(reject("E"), 3000)));
     const onChangeAtocomplete = jest.fn();
@@ -229,7 +233,7 @@ describe("InputText component Asynchronous Autocomplete tests", () => {
     );
 
     const input = screen.getByRole("textbox");
-    screen.getByRole("textbox").focus();
+    fireEvent.focus(input);
     await waitForElementToBeRemoved(() => screen.getByText(/Searching.../i));
     expect(screen.getByText("Error fetching data")).toBeTruthy();
   });

package/test/Link.test.js

@@ -13,9 +13,11 @@ describe("Link component tests", () => {
     expect(getByRole("link").getAttribute("href")).toEqual("/testPage");
   });
 
-  test("Link renders with disable attribute", () => {
+  test("Link renders with correct disabled state", () => {
     const { getByText } = render(<DxcLink text="Link" href="/testPage" disabled={true} />);
-    expect(getByText("Link").hasAttribute("disabled")).toBeTruthy();
+    expect(getByText("Link").hasAttribute("href")).toBeFalsy();
+    const { getByText: getByTextLinkButton } = render(<DxcLink text="LinkButton" onClick={() => console.log("Andorra")} disabled={true} />);
+    expect(getByTextLinkButton("LinkButton").hasAttribute("onclick")).toBeFalsy();
   });
 
   test("Link open new tab", () => {
@@ -30,4 +32,12 @@ describe("Link component tests", () => {
     fireEvent.click(link);
     expect(onClick).toHaveBeenCalled();
   });
+
+  test("Disabled link onClick not called", () => {
+    const onClick = jest.fn();
+    const { getByText } = render(<DxcLink text="Link" onClick={onClick} disabled={true} />);
+    const link = getByText("Link");
+    fireEvent.click(link);
+    expect(onClick).toHaveBeenCalledTimes(0);
+  });
 });