@dxc-technology/halstack-react 0.0.0-90f64ff → 0.0.0-9427b76

package/test/NewTextarea.test.js

@@ -1,6 +1,6 @@
 import React from "react";
 import { render, fireEvent } from "@testing-library/react";
-
+import userEvent from "@testing-library/user-event";
 import DxcNewTextarea from "../src/new-textarea/NewTextarea";
 
 describe("NewTextarea component tests", () => {
@@ -14,29 +14,75 @@ describe("NewTextarea component tests", () => {
     expect(getByText("Example helper text")).toBeTruthy();
   });
   test("Renders with correct label and optional", () => {
-    const { getByText } = render(<DxcNewTextarea label="Example label" helperText="Example helper text" optional />);
+    const { getByText, getByRole } = render(
+      <DxcNewTextarea label="Example label" helperText="Example helper text" optional />
+    );
+    const textarea = getByRole("textbox");
     expect(getByText("Example label")).toBeTruthy();
     expect(getByText("(Optional)")).toBeTruthy();
     expect(getByText("Example helper text")).toBeTruthy();
+    expect(textarea.getAttribute("aria-required")).toBe("false");
   });
   test("Renders with correct placeholder", () => {
     const { getByRole } = render(<DxcNewTextarea placeholder="Placeholder" />);
-    const input = getByRole("textbox");
-    expect(input.getAttribute("placeholder")).toBe("Placeholder");
+    const textarea = getByRole("textbox");
+    expect(textarea.getAttribute("placeholder")).toBe("Placeholder");
   });
   test("Renders with error message", () => {
-    const { getByText } = render(<DxcNewTextarea error="Error message." />);
+    const { getByText, getByRole } = render(<DxcNewTextarea error="Error message." />);
+    const textarea = getByRole("textbox");
     expect(getByText("Error message.")).toBeTruthy();
+    expect(textarea.getAttribute("aria-invalid")).toBe("true");
+    expect(textarea.getAttribute("aria-describedBy")).not.toBeNull();
   });
   test("Renders with correct default rows", () => {
     const { getByLabelText } = render(<DxcNewTextarea label="Example label" rows={10} />);
     const textarea = getByLabelText("Example label");
     expect(textarea.rows).toBe(10);
   });
-  test("Strict mode - Pattern constraint", () => {
+  test("Renders with correct accesibility attributes", () => {
+    const { getByLabelText } = render(<DxcNewTextarea label="Example label" />);
+    const textarea = getByLabelText("Example label");
+    expect(textarea.getAttribute("aria-invalid")).toBe("false");
+    expect(textarea.getAttribute("aria-describedBy")).toBeNull();
+    expect(textarea.getAttribute("aria-required")).toBe("true");
+  });
+  test("Not optional constraint (onBlur)", () => {
     const onChange = jest.fn();
     const onBlur = jest.fn();
-    const { getByLabelText, getByText, queryByText } = render(
+    const { getByLabelText } = render(
+      <DxcNewTextarea label="Example label" placeholder="Placeholder" onChange={onChange} onBlur={onBlur} />
+    );
+    const textarea = getByLabelText("Example label");
+
+    fireEvent.focus(textarea);
+    fireEvent.blur(textarea);
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "", error: "This field is required. Please, enter a value." });
+    fireEvent.change(textarea, { target: { value: "Test" } });
+    fireEvent.blur(textarea);
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "Test", error: null });
+  });
+  test("Not optional constraint (onChange)", () => {
+    const onChange = jest.fn();
+    const { getByLabelText } = render(
+      <DxcNewTextarea label="Example label" placeholder="Placeholder" onChange={onChange} />
+    );
+    const textarea = getByLabelText("Example label");
+
+    fireEvent.focus(textarea);
+    fireEvent.change(textarea, { target: { value: "Test" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "Test", error: null });
+    userEvent.clear(textarea);
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "", error: "This field is required. Please, enter a value." });
+  });
+  test("Pattern constraint", () => {
+    const onChange = jest.fn();
+    const onBlur = jest.fn();
+    const { getByLabelText } = render(
       <DxcNewTextarea
         label="Example label"
         placeholder="Placeholder"
@@ -47,17 +93,25 @@ describe("NewTextarea component tests", () => {
       />
     );
     const textarea = getByLabelText("Example label");
+
     fireEvent.change(textarea, { target: { value: "pattern test" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "pattern test", error: "Please match the format requested." });
     fireEvent.blur(textarea);
-    expect(getByText("Please match the format requested.")).toBeTruthy();
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "pattern test", error: "Please match the format requested." });
+    userEvent.clear(textarea);
     fireEvent.change(textarea, { target: { value: "pattern4&" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "pattern4&", error: null });
     fireEvent.blur(textarea);
-    expect(queryByText("Please match the format requested.")).toBeFalsy();
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "pattern4&", error: null });
   });
-  test("Strict mode - Length constraint", () => {
+  test("Length constraint", () => {
     const onChange = jest.fn();
     const onBlur = jest.fn();
-    const { getByLabelText, getByText, queryByText } = render(
+    const { getByLabelText } = render(
       <DxcNewTextarea
         label="Example label"
         placeholder="Placeholder"
@@ -68,14 +122,22 @@ describe("NewTextarea component tests", () => {
       />
     );
     const textarea = getByLabelText("Example label");
+
     fireEvent.change(textarea, { target: { value: "test" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "test", error: "Min length 5, max length 10." });
     fireEvent.blur(textarea);
-    expect(getByText("Min length 5, max length 10.")).toBeTruthy();
-    fireEvent.change(textarea, { target: { value: "test " } });
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "test", error: "Min length 5, max length 10." });
+    userEvent.clear(textarea);
+    fireEvent.change(textarea, { target: { value: "length" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "length", error: null });
     fireEvent.blur(textarea);
-    expect(queryByText(/Min length /)).toBeFalsy();
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "length", error: null });
   });
-  test("Strict mode - Pattern and length constraints", () => {
+  test("Pattern and length constraints", () => {
     const onChange = jest.fn();
     const onBlur = jest.fn();
     const { getByLabelText, getByText, queryByText } = render(
@@ -90,81 +152,25 @@ describe("NewTextarea component tests", () => {
       />
     );
     const textarea = getByLabelText("Example label");
+
     fireEvent.change(textarea, { target: { value: "test" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "test", error: "Min length 5, max length 10." });
     fireEvent.blur(textarea);
-    expect(getByText("Min length 5, max length 10.")).toBeTruthy();
-    fireEvent.change(textarea, { target: { value: "test " } });
-    fireEvent.blur(textarea);
-    expect(getByText("Please match the format requested.")).toBeTruthy();
-    fireEvent.change(textarea, { target: { value: "test 4" } });
-    fireEvent.blur(textarea);
-    expect(queryByText("Please match the format requested.")).toBeFalsy();
-  });
-  test("Non Strict mode - Pattern constraint", () => {
-    const onChange = jest.fn((value) => {
-      expect(value).toBe("Example value");
-    });
-    const onBlur = jest.fn(({ value, error }) => {
-      expect(value).toBe("Example value");
-      expect(error).toBe("Please match the format requested.");
-    });
-    const { getByLabelText } = render(
-      <DxcNewTextarea
-        label="Example label"
-        placeholder="Placeholder"
-        onChange={onChange}
-        onBlur={onBlur}
-        margin={{ left: "medium", right: "medium" }}
-        pattern='^.*(?=.*[a-zA-Z])(?=.*\d)(?=.*[!&$%&? "]).*$'
-      />
-    );
-    const textarea = getByLabelText("Example label");
-    fireEvent.change(textarea, { target: { value: "Example value" } });
-    fireEvent.blur(textarea);
-  });
-  test("Non Strict mode - Length constraint", () => {
-    const onChange = jest.fn((value) => {
-      expect(value).toBe("Example value");
-    });
-    const onBlur = jest.fn(({ value, error }) => {
-      expect(value).toBe("Example value");
-      expect(error).toBe("Min length 5, max length 10.");
-    });
-    const { getByLabelText } = render(
-      <DxcNewTextarea
-        label="Example label"
-        placeholder="Placeholder"
-        onChange={onChange}
-        onBlur={onBlur}
-        margin={{ left: "medium", right: "medium" }}
-        length={{ min: 5, max: 10 }}
-      />
-    );
-    const textarea = getByLabelText("Example label");
-    fireEvent.change(textarea, { target: { value: "Example value" } });
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "test", error: "Min length 5, max length 10." });
+    fireEvent.change(textarea, { target: { value: "tests" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "tests", error: "Please match the format requested." });
     fireEvent.blur(textarea);
-  });
-  test("Non Strict mode - Pattern and length constraints", () => {
-    const onChange = jest.fn((value) => {
-      expect(value).toBe("Example value");
-    });
-    const onBlur = jest.fn(({ value, error }) => {
-      expect(value).toBe("Example value");
-      expect(error).toBe("Min length 5, max length 10.");
-    });
-    const { getByLabelText } = render(
-      <DxcNewTextarea
-        label="Example label"
-        placeholder="Placeholder"
-        onChange={onChange}
-        onBlur={onBlur}
-        margin={{ left: "medium", right: "medium" }}
-        length={{ min: 5, max: 10 }}
-      />
-    );
-    const textarea = getByLabelText("Example label");
-    fireEvent.change(textarea, { target: { value: "Example value" } });
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "tests", error: "Please match the format requested." });
+    fireEvent.change(textarea, { target: { value: "tests4&" } });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "tests4&", error: null });
     fireEvent.blur(textarea);
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "tests4&", error: null });
   });
   test("onBlur function is called correctly", () => {
     const onBlur = jest.fn();
@@ -175,27 +181,15 @@ describe("NewTextarea component tests", () => {
     expect(onBlur).toHaveBeenCalled();
     expect(onBlur).toHaveBeenCalledWith({ value: "Blur test", error: null });
   });
-  test("Controlled textarea", () => {
+  test("onChange function is called correctly", () => {
     const onChange = jest.fn();
-    const onBlur = jest.fn();
     const { getByLabelText } = render(
-      <DxcNewTextarea label="Example label" value="Test value" onChange={onChange} onBlur={onBlur} />
+      <DxcNewTextarea label="Example label" value="Test value" onChange={onChange} />
     );
     const textarea = getByLabelText("Example label");
     fireEvent.change(textarea, { target: { value: "Controlled test" } });
     expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "Controlled test", error: null });
     expect(textarea.value).toBe("Test value");
-    fireEvent.blur(textarea);
-    expect(onBlur).toHaveBeenCalled();
-    expect(onBlur).toHaveBeenCalledWith({ value: "Test value", error: null });
-  });
-  test("Uncontrolled input", () => {
-    const onChange = jest.fn();
-    const { getByLabelText } = render(<DxcNewTextarea label="Example label" onChange={onChange} />);
-    const textarea = getByLabelText("Example label");
-    fireEvent.change(textarea, { target: { value: "Uncontrolled test" } });
-    expect(onChange).toHaveBeenCalled();
-    expect(onChange).toHaveBeenCalledWith("Uncontrolled test");
-    expect(textarea.value).toBe("Uncontrolled test");
   });
 });

package/test/Number.test.js

@@ -36,6 +36,21 @@ describe("Number component tests", () => {
     expect(getByText("(Optional)")).toBeTruthy();
   });
 
+  test("Number is not optional: required field, displays error if not filled in", () => {
+    const onBlur = jest.fn();
+    const onChange = jest.fn();
+    const { getByRole } = render(<DxcNumber onBlur={onBlur} onChange={onChange} />);
+    const input = getByRole("textbox");
+
+    userEvent.type(input, "1");
+    userEvent.clear(input);
+    fireEvent.blur(input);
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "", error: "This field is required. Please, enter a value." });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({ value: "", error: "This field is required. Please, enter a value." });
+  });
+
   test("Suffix and prefix must be shown)", () => {
     const { getByText } = render(<DxcNumber label="Number label" prefix="+34" suffix="USD" />);
     expect(getByText("+34")).toBeTruthy();
@@ -52,19 +67,29 @@ describe("Number component tests", () => {
     const { getByLabelText } = render(<DxcNumber label="Number label" onChange={onChange} />);
     const number = getByLabelText("Number label");
     userEvent.type(number, "t");
-    expect(onChange).not.toHaveBeenCalledWith("t");
+    expect(onChange).not.toHaveBeenCalledWith({ value: "t", error: null });
     expect(number.value).toBe("");
     userEvent.type(number, "1");
-    expect(onChange).toHaveBeenCalledWith("1");
+    expect(onChange).toHaveBeenCalledWith({ value: "1", error: null });
     expect(number.value).toBe("1");
   });
 
   test("Error message is shown if the value is less than the min value", () => {
-    const { getByLabelText, getByText } = render(<DxcNumber label="Number label" min={5} />);
+    const onChange = jest.fn(({ value, error }) => {
+      expect(value).toBe("1");
+      expect(error).toBe("Value must be greater than or equal to 5.");
+    });
+    const onBlur = jest.fn(({ value, error }) => {
+      expect(value).toBe("1");
+      expect(error).toBe("Value must be greater than or equal to 5.");
+    });
+    const { getByLabelText } = render(
+      <DxcNumber label="Number label" min={5} onBlur={onBlur} onChange={onChange} />
+    );
     const number = getByLabelText("Number label");
+
     userEvent.type(number, "1");
     fireEvent.blur(number);
-    expect(getByText("Value must be greater than or equal to 5.")).toBeTruthy();
   });
 
   test("Cannot decrement the value if it is less than the min value", () => {
@@ -90,11 +115,17 @@ describe("Number component tests", () => {
   });
 
   test("Error message is shown if the value is greater than the max value", () => {
-    const { getByLabelText, getByText } = render(<DxcNumber label="Number label" max={10} />);
+    const onChange = jest.fn();
+    const onBlur = jest.fn();
+    const { getByLabelText } = render(<DxcNumber label="Number label" max={10} onBlur={onBlur} onChange={onChange} />);
     const number = getByLabelText("Number label");
-    userEvent.type(number, "12");
+
+    userEvent.type(number, "12");    
+    expect(onChange).toHaveBeenCalledTimes(2);
+    expect(onChange).toHaveBeenCalledWith({ value: "12", error: "Value must be less than or equal to 10." });
     fireEvent.blur(number);
-    expect(getByText("Value must be less than or equal to 10.")).toBeTruthy();
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "12", error: "Value must be less than or equal to 10." });
   });
 
   test("Cannot increment the value if it is greater than the max value", () => {
@@ -160,13 +191,16 @@ describe("Number component tests", () => {
   });
 
   test("Increment and decrement the value with min, max and step", () => {
-    const { getByLabelText, getByText, getAllByRole } = render(
-      <DxcNumber label="Number label" min={5} max={20} step={8} />
+    const onBlur = jest.fn(({ value, error }) => {
+      expect(value).toBe("1");
+      expect(error).toBe("Value must be greater than or equal to 5.");
+    });
+    const { getByLabelText, getAllByRole } = render(
+      <DxcNumber label="Number label" min={5} max={20} step={8} onBlur={onBlur} />
     );
     const number = getByLabelText("Number label");
     userEvent.type(number, "1");
     fireEvent.blur(number);
-    expect(getByText("Value must be greater than or equal to 5.")).toBeTruthy();
     const increment = getAllByRole("button")[1];
     userEvent.click(increment);
     expect(number.value).toBe("5");
@@ -209,24 +243,6 @@ describe("Number component tests", () => {
     expect(number.value).toBe("5");
   });
 
-  test("Custom error messages", () => {
-    const onChange = jest.fn((value) => {
-      expect(value).toBe("2");
-    });
-
-    const onBlur = jest.fn(({ value, error }) => {
-      expect(value).toBe("2");
-      expect(error).toBe("Value must be greater than or equal to 5.");
-    });
-
-    const { getByLabelText } = render(
-      <DxcNumber label="Number label" min={5} step={8} onChange={onChange} onBlur={onBlur} />
-    );
-    const number = getByLabelText("Number label");
-    userEvent.type(number, "2");
-    fireEvent.blur(number);
-  });
-
   test("Number has correct accesibility attributes", () => {
     const { getByLabelText, getAllByRole } = render(<DxcNumber label="Number label" />);
     const number = getByLabelText("Number label");

package/test/Paginator.test.js

@@ -62,7 +62,7 @@ describe("Paginator component tests", () => {
     act(() => {
       userEvent.click(goToPageSelect);
     });
-    const goToPageOption = getByRole("option", { name: /2/i });
+    const goToPageOption = getByText("2");
     act(() => {
       userEvent.click(goToPageOption);
     });

package/test/Password.test.js

@@ -1,5 +1,5 @@
 import React from "react";
-import { render } from "@testing-library/react";
+import { render, fireEvent } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 import DxcPassword from "../src/password/Password";
 
@@ -24,13 +24,22 @@ describe("Password component tests", () => {
     const { getByRole } = render(<DxcPassword label="Password" onChange={onChange} />);
     const password = getByRole("textbox");
     userEvent.type(password, "Pa$$w0rd");
-    expect(onChange).toHaveBeenCalledWith("P");
+    expect(onChange).toHaveBeenCalledWith({ value: "P", error: null });
+    expect(password.value).toBe("Pa$$w0rd");
+  });
+
+  test("onBlur function is called correctly", () => {
+    const onBlur = jest.fn();
+    const { getByRole } = render(<DxcPassword label="Password" onBlur={onBlur} />);
+    const password = getByRole("textbox");
+    userEvent.type(password, "Pa$$w0rd");    
+    fireEvent.blur(password);
+    expect(onBlur).toHaveBeenCalledWith({ value: "Pa$$w0rd", error: null });
     expect(password.value).toBe("Pa$$w0rd");
   });
 
   test("Clear password value", () => {
-    const onChange = jest.fn();
-    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" onChange={onChange} clearable />);
+    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" clearable />);
     const password = getByRole("textbox");
     userEvent.type(password, "Pa$$w0rd");
     expect(password.value).toBe("Pa$$w0rd");
@@ -40,8 +49,7 @@ describe("Password component tests", () => {
   });
 
   test("Non clearable password has no clear icon", () => {
-    const onChange = jest.fn();
-    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" onChange={onChange} />);
+    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" />);
     const password = getByRole("textbox");
     userEvent.type(password, "Pa$$w0rd");
     expect(password.value).toBe("Pa$$w0rd");
@@ -50,8 +58,7 @@ describe("Password component tests", () => {
   });
 
   test("Show/hide password button works correctly", () => {
-    const onChange = jest.fn();
-    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" clearable onChange={onChange} />);
+    const { getAllByRole, getByRole } = render(<DxcPassword label="Password" clearable />);
     const password = getByRole("textbox");
     userEvent.type(password, "Pa$$w0rd");
     expect(password.value).toBe("Pa$$w0rd");

package/test/ResultsetTable.test.js

@@ -266,13 +266,12 @@ describe("ResultsetTable component tests", () => {
     act(() => {
       userEvent.click(goToPageSelect);
     });
-    const goToPageOption = getByRole("option", { name: /2/i });
+    const goToPageOption = getByText("2");
     act(() => {
       userEvent.click(goToPageOption);
     });
 
     expect(getByText("4 to 6 of 10")).toBeTruthy();
-    // expect(getByText("Page: 2 of 4")).toBeTruthy();
     expect(getByText("Rick")).toBeTruthy();
     expect(getByText("Mark")).toBeTruthy();
     expect(getByText("Cris")).toBeTruthy();

package/test/Select.test.js

@@ -21,18 +21,43 @@ const optionsWithoutIcon = [
 const optionsWithIcons = [
   {
     value: 1,
-    label: "Facebook",
-    iconSrc: "/testIconFacebook",
+    label: "Image 1",
+    icon: (
+      <svg
+        data-testid="image-1"
+        xmlns="http://www.w3.org/2000/svg"
+        height="24px"
+        viewBox="0 0 24 24"
+        width="24px"
+        fill="currentColor"
+      >
+        <path d="M0 0h24v24H0V0z" fill="none" />
+        <path d="M16 1H4c-1.1 0-2 .9-2 2v14h2V3h12V1zm3 4H8c-1.1 0-2 .9-2 2v14c0 1.1.9 2 2 2h11c1.1 0 2-.9 2-2V7c0-1.1-.9-2-2-2zm0 16H8V7h11v14z" />
+      </svg>
+    ),
   },
   {
     value: 2,
-    label: "Twitter",
-    iconSrc: "/testIconTwitter",
-  },
-  {
-    value: 3,
-    label: "Linkedin",
-    iconSrc: "/testIconLinkedin",
+    label: "Image 2",
+    icon: (
+      <svg
+        data-testid="image-2"
+        version="1.1"
+        x="0px"
+        y="0px"
+        width="24px"
+        height="24px"
+        viewBox="0 0 24 24"
+        enable-background="new 0 0 24 24"
+      >
+        <g id="Bounding_Box">
+          <rect fill="none" width="24" height="24" />
+        </g>
+        <g id="Master">
+          <path d="M19,9.3V4h-3v2.6L12,3L2,12h3v8h5v-6h4v6h5v-8h3L19,9.3z M10,10c0-1.1,0.9-2,2-2s2,0.9,2,2H10z" />
+        </g>
+      </svg>
+    ),
   },
 ];
 
@@ -60,29 +85,27 @@ describe("Select component tests", () => {
   });
 
   test("Select renders options with icons before", () => {
-    const { getByRole, getAllByRole } = render(
+    const { getByRole, getByTestId } = render(
       <DxcSelect label="test-select-name" options={optionsWithIcons}></DxcSelect>
     );
     const select = getByRole("button");
     act(() => {
       userEvent.click(select);
     });
-    expect(getAllByRole("img")[0].getAttribute("src")).toEqual("/testIconFacebook");
-    expect(getAllByRole("img")[1].getAttribute("src")).toEqual("/testIconTwitter");
-    expect(getAllByRole("img")[2].getAttribute("src")).toEqual("/testIconLinkedin");
+    expect(getByTestId("image-1")).toBeTruthy();
+    expect(getByTestId("image-2")).toBeTruthy();
   });
 
   test("Select renders options with icons after", () => {
-    const { getByRole, getAllByRole } = render(
+    const { getByRole, getByTestId } = render(
       <DxcSelect label="test-select-name" options={optionsWithIcons} iconPosition="after"></DxcSelect>
     );
     const select = getByRole("button");
     act(() => {
       userEvent.click(select);
     });
-    expect(getAllByRole("img")[0].getAttribute("src")).toEqual("/testIconFacebook");
-    expect(getAllByRole("img")[1].getAttribute("src")).toEqual("/testIconTwitter");
-    expect(getAllByRole("img")[2].getAttribute("src")).toEqual("/testIconLinkedin");
+    expect(getByTestId("image-1")).toBeTruthy();
+    expect(getByTestId("image-2")).toBeTruthy();
   });
 
   test("Select renders with a default value", () => {