@dxc-technology/halstack-react 0.0.0-669d8de → 0.0.0-69ca263

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxc-technology/halstack-react",
-  "version": "0.0.0-669d8de",
+  "version": "0.0.0-69ca263",
   "description": "DXC Halstack React components library",
   "repository": "dxc-technology/halstack-react",
   "homepage": "http://developer.dxc.com/tools/react",
@@ -11,10 +11,11 @@
     "url": "https://dxc.com"
   },
   "main": "./dist/main.js",
+  "types": "./dist/main.d.ts",
   "peerDependencies": {
-    "react": "^16.8.6",
-    "react-dom": "^16.8.6",
-    "styled-components": ">= 4.3.1"
+    "react": "^17.0.1",
+    "react-dom": "^17.0.1",
+    "styled-components": "^5.0.1"
   },
   "dependencies": {
     "@date-io/date-fns": "^1.3.9",
@@ -28,7 +29,6 @@
     "date-fns": "^2.0.0-beta.4",
     "moment": "2.24.0",
     "prop-types": "^15.7.2",
-    "react-use-scrollspy": "^2.0.0",
     "rgb-hex": "^3.0.0",
     "uuid": "^8.3.2"
   },
@@ -36,7 +36,9 @@
     "test": "jest",
     "test:watch": "npm test -- --watch --coverage",
     "build": "babel src --out-dir dist --copy-files --verbose && node ../scripts/build/copy-readme.js",
-    "build:watch": "babel src --watch --out-dir dist --copy-files --verbose"
+    "build:watch": "babel src --watch --out-dir dist --copy-files --verbose",
+    "storybook": "start-storybook -p 6006",
+    "build-storybook": "build-storybook"
   },
   "devDependencies": {
     "@babel/cli": "^7.6.2",
@@ -47,9 +49,14 @@
     "@babel/plugin-transform-runtime": "^7.10.1",
     "@babel/preset-env": "^7.6.2",
     "@babel/preset-react": "^7.0.0",
+    "@storybook/addon-actions": "^6.4.9",
+    "@storybook/addon-essentials": "^6.4.9",
+    "@storybook/addon-links": "^6.4.9",
+    "@storybook/react": "^6.4.9",
     "@testing-library/react": "^11.2.5",
     "babel-jest": "^24.8.0",
     "babel-loader": "^8.0.6",
+    "chromatic": "^6.3.3",
     "eslint": "^5.16.0",
     "eslint-config-airbnb": "^17.1.0",
     "eslint-config-prettier": "^6.7.0",
@@ -58,6 +65,7 @@
     "eslint-plugin-jsx-a11y": "^6.2.1",
     "eslint-plugin-react": "^7.13.0",
     "eslint-plugin-react-hooks": "^4.2.0",
+    "eslint-plugin-storybook": "^0.5.5",
     "identity-obj-proxy": "^3.0.0",
     "jest": "^25.5.4",
     "react": "file:../node_modules/react",
@@ -70,9 +78,6 @@
       "\\.(css|less|scss|sass)$": "identity-obj-proxy",
       "\\.(svg)$": "<rootDir>/test/mocks/svgMock.js",
       "\\.(png)$": "<rootDir>/test/mocks/pngMock.js"
-    },
-    "transformIgnorePatterns": [
-      "/node_modules/(?!react-use-scrollspy).+\\.js$"
-    ]
+    }
   }
 }

package/test/Date.test.js

@@ -51,7 +51,7 @@ describe("Controlled Date Component", () => {
 
   test("Calendar´s value is the same as the input´s date if it´s right (Depending on the format)", () => {
     const { getByRole, getByText, getByTestId } = render(<DxcDate label="Birthdate" value="16-10-2020" />);
-    fireEvent.click(getByTestId('calendarIcon'));
+    fireEvent.click(getByTestId("calendarIcon"));
     expect(getByText("Fri, Oct 16")).toBeTruthy();
   });
 
@@ -64,7 +64,7 @@ describe("Controlled Date Component", () => {
     fireEvent.change(input, { target: { value: "10-02-2020" } });
 
     rerender(<DxcDate label="Birthdate" value="10-02-2020" />);
-    const calendarButton = getByTestId('calendarIcon');
+    const calendarButton = getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     expect(getByText("Mon, Feb 10")).toBeTruthy();
   });
@@ -152,7 +152,7 @@ describe("Controlled Date Component", () => {
     const onChange = jest.fn();
 
     const component = render(<DxcDate label="Birthdate" value="01-02-2020" onChange={onChange} />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "10" });
     fireEvent.click(dayButton);
@@ -166,7 +166,7 @@ describe("Controlled Date Component", () => {
     });
 
     const component = render(<DxcDate label="Birthdate" value="30-10-2020" onChange={onChange} />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "16" });
     fireEvent.click(dayButton);
@@ -179,7 +179,7 @@ describe("Controlled Date Component", () => {
     });
 
     const component = render(<DxcDate label="Birthdate" value="01-10-2020" onChange={onChange} />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "16" });
     fireEvent.click(dayButton);
@@ -192,7 +192,7 @@ describe("Controlled Date Component", () => {
     });
 
     const component = render(<DxcDate label="Birthdate" format="yyyy/MM/dd" value="2020/10/01" onChange={onChange} />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "16" });
     fireEvent.click(dayButton);
@@ -201,7 +201,7 @@ describe("Controlled Date Component", () => {
 
   test("Check selected date on calendar is the same date as the one on the input", () => {
     const component = render(<DxcDate label="Birthdate" value="01-10-2020" />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "1" });
     expect(dayButton.classList.contains("MuiPickersDay-daySelected")).toBe(true);
@@ -249,14 +249,16 @@ describe("Uncontrolled Date Component", () => {
   const date = new Date("2020-10-16 00:00:00");
 
   test("Calendar´s value is the same as the input´s date if it´s right after changing the input value", () => {
-    const { getByRole, getByText, rerender, getByPlaceholderText, getByTestId } = render(<DxcDate label="Birthdate" placeholder />);
+    const { getByRole, getByText, rerender, getByPlaceholderText, getByTestId } = render(
+      <DxcDate label="Birthdate" placeholder />
+    );
     // const input = getByRole("textbox");
     const input = getByPlaceholderText(defaultFormat);
 
     fireEvent.change(input, { target: { value: "10-02-2020" } });
 
     rerender(<DxcDate label="Birthdate" value="10-02-2020" />);
-    const calendarButton = getByTestId('calendarIcon');
+    const calendarButton = getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     expect(getByText("Mon, Feb 10")).toBeTruthy();
   });
@@ -336,7 +338,7 @@ describe("Uncontrolled Date Component", () => {
     const onChange = jest.fn();
 
     const component = render(<DxcDate label="Birthdate" onChange={onChange} />);
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "10" });
     fireEvent.click(dayButton);
@@ -350,7 +352,7 @@ describe("Uncontrolled Date Component", () => {
     const input = component.getByPlaceholderText(defaultFormat);
 
     fireEvent.change(input, { target: { value: "10-02-2020" } });
-    const calendarButton = component.getByTestId('calendarIcon');
+    const calendarButton = component.getByTestId("calendarIcon");
     fireEvent.click(calendarButton);
     const dayButton = component.getByRole("button", { name: "10" });
     expect(dayButton.classList.contains("MuiPickersDay-daySelected")).toBe(true);

package/test/{NewDate.test.js → DateInput.test.js}

@@ -2,41 +2,55 @@ import React from "react";
 import { render, fireEvent } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 
-import DxcNewDate from "../src/new-date/NewDate";
+import DxcDateInput from "../src/date-input/DateInput";
 
-describe("NewDate component tests", () => {
+describe("DateInput component tests", () => {
   test("Renders with correct label, helper text, optional, placeholder and clearable action", () => {
     const { getByText, getByRole, getAllByRole } = render(
-      <DxcNewDate label="Example label" helperText="Example of helper text" placeholder optional clearable />
+      <DxcDateInput label="Example label" helperText="Example of helper text" placeholder optional clearable />
     );
     const input = getByRole("textbox");
     expect(getByText("Example label")).toBeTruthy();
     expect(getByText("Example of helper text")).toBeTruthy();
     expect(getByText("(Optional)")).toBeTruthy();
     expect(input.getAttribute("placeholder")).toBe("DD-MM-YYYY");
-    userEvent.type(input, "10/90/2010");
+    userEvent.type(input, "10/10/2010");
     const closeAction = getAllByRole("button")[0];
     userEvent.click(closeAction);
     expect(input.value).toBe("");
   });
 
   test("Renders with personalized error", () => {
-    const { getByText } = render(<DxcNewDate error="Personalized error." />);
-
+    const { getByText } = render(<DxcDateInput error="Personalized error." />);
     expect(getByText("Personalized error.")).toBeTruthy();
   });
 
-  test("Renders with correct format: user typed date but it's invalid", () => {
-    const { getByText, getByRole } = render(<DxcNewDate label="With format MM/dd/yyyy" format="MM/dd/yyyy" />);
+  test("Renders with correct format: user typed date but it's invalid, onBlur error", () => {
+    const onBlur = jest.fn(({ value, error }) => {
+      expect(value).toBe("10/90/2010");
+      expect(error).toBe("Invalid date.");
+    });
+    const { getByRole } = render(<DxcDateInput label="With format MM/dd/yyyy" format="MM/dd/yyyy" onBlur={onBlur} />);
     const input = getByRole("textbox");
 
     userEvent.type(input, "10/90/2010");
     fireEvent.blur(input);
-    expect(getByText("Invalid date.")).toBeTruthy();
+  });
+
+  test("Renders with correct format: user typed date but it's invalid, onChange error", () => {
+    const onChange = jest.fn();
+    const { getByRole } = render(
+      <DxcDateInput label="With format MM/dd/yyyy" format="MM/dd/yyyy" onChange={onChange} />
+    );
+    const input = getByRole("textbox");
+
+    userEvent.type(input, "10/90/2010");
+    expect(onChange).toHaveBeenCalledTimes(10);
+    expect(onChange).toHaveBeenCalledWith({ value: "10/90/2010", error: "Invalid date.", date: null });
   });
 
   test("Calendar renders with correct date: today's date", () => {
-    const { getByText, getByRole } = render(<DxcNewDate />);
+    const { getByText, getByRole } = render(<DxcDateInput />);
     const calendarAction = getByRole("button");
     const d = new Date();
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -46,7 +60,7 @@ describe("NewDate component tests", () => {
   });
 
   test("Calendar renders with correct date: value prop", () => {
-    const { getByText, getByRole } = render(<DxcNewDate value="20-10-2019" />);
+    const { getByText, getByRole } = render(<DxcDateInput value="20-10-2019" />);
     const calendarAction = getByRole("button");
     const d = new Date(2019, 9, 20);
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -56,7 +70,7 @@ describe("NewDate component tests", () => {
   });
 
   test("Calendar renders with correct date: user typed value", () => {
-    const { getByText, getByRole } = render(<DxcNewDate />);
+    const { getByText, getByRole } = render(<DxcDateInput />);
     const calendarAction = getByRole("button");
     const d = new Date(2010, 0, 1);
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -68,7 +82,8 @@ describe("NewDate component tests", () => {
   });
 
   test("Calendar renders with correct date: invalid date, renders with today's date", () => {
-    const { getByText, getByRole } = render(<DxcNewDate />);
+    const onBlur = jest.fn();
+    const { getByText, getByRole } = render(<DxcDateInput onBlur={onBlur} />);
     const calendarAction = getByRole("button");
     const d = new Date();
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -76,17 +91,18 @@ describe("NewDate component tests", () => {
 
     userEvent.type(input, "01-01-xxxx");
     fireEvent.blur(input);
-    expect(getByText("Invalid date.")).toBeTruthy();
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({ value: "01-01-xxxx", error: "Invalid date.", date: null });
     userEvent.click(calendarAction);
     expect(getByText(d.toLocaleString("en-US", options))).toBeTruthy();
   });
 
   test("Selecting a date from the calendar with an specific format", () => {
-    const { getByText, getByRole } = render(<DxcNewDate label="With format M-dd-yyyy" format="M-dd-yyyy" />);
+    const { getByText, getByRole } = render(<DxcDateInput label="With format M-dd-yyyy" format="M-dd-yyyy" />);
     const input = getByRole("textbox");
     const calendarAction = getByRole("button");
     userEvent.click(calendarAction);
-    const dayButton = getByRole("button", { name: "10" });
+    const dayButton = getByText("10").closest("button");
     fireEvent.click(dayButton);
     const d = new Date();
     d.setDate(10);
@@ -97,7 +113,7 @@ describe("NewDate component tests", () => {
   });
 
   test("Selecting a date from the calendar (using keyboard presses)", () => {
-    const { getByRole } = render(<DxcNewDate />);
+    const { getByRole } = render(<DxcDateInput />);
     const calendarAction = getByRole("button");
     const input = getByRole("textbox");
 
@@ -114,14 +130,14 @@ describe("NewDate component tests", () => {
   test("onChange & onBlur functions are called correctly", () => {
     const onBlur = jest.fn();
     const onChange = jest.fn();
-    const { getByRole } = render(<DxcNewDate onChange={onChange} onBlur={onBlur} />);
+    const { getByRole } = render(<DxcDateInput onChange={onChange} onBlur={onBlur} />);
     const input = getByRole("textbox");
     const d = new Date(2011, 9, 10);
 
     userEvent.type(input, "10-10-2011");
     expect(input.value).toBe("10-10-2011");
     expect(onChange).toHaveBeenCalledTimes(10);
-    expect(onChange).toHaveBeenCalledWith({ value: "10-10-2011", date: d });
+    expect(onChange).toHaveBeenCalledWith({ value: "10-10-2011", error: null, date: d });
     fireEvent.blur(input);
     expect(onBlur).toHaveBeenCalled();
     expect(onBlur).toHaveBeenCalledWith({ value: "10-10-2011", error: null, date: d });
@@ -130,14 +146,13 @@ describe("NewDate component tests", () => {
   test("onChange & onBlur functions are called correctly, also with errors", () => {
     const onBlur = jest.fn();
     const onChange = jest.fn();
-    const { getByRole } = render(<DxcNewDate onChange={onChange} onBlur={onBlur} />);
+    const { getByRole } = render(<DxcDateInput onChange={onChange} onBlur={onBlur} />);
     const input = getByRole("textbox");
-    const d = new Date(2011, 9, 10);
 
     userEvent.type(input, "10-10-");
     expect(input.value).toBe("10-10-");
     expect(onChange).toHaveBeenCalledTimes(6);
-    expect(onChange).toHaveBeenCalledWith({ value: "10-10-", date: null });
+    expect(onChange).toHaveBeenCalledWith({ value: "10-10-", error: "Invalid date.", date: null });
     fireEvent.blur(input);
     expect(onBlur).toHaveBeenCalled();
     expect(onBlur).toHaveBeenCalledWith({ value: "10-10-", error: "Invalid date.", date: null });
@@ -145,7 +160,7 @@ describe("NewDate component tests", () => {
 
   test("onBlur function removes the error when it is fixed", () => {
     const onBlur = jest.fn();
-    const { getByRole } = render(<DxcNewDate onBlur={onBlur} />);
+    const { getByRole } = render(<DxcDateInput onBlur={onBlur} />);
     const input = getByRole("textbox");
     const d = new Date(2002, 1, 20);
 
@@ -164,7 +179,7 @@ describe("NewDate component tests", () => {
 
   test("onBlur function removes the error when the input is empty", () => {
     const onBlur = jest.fn();
-    const { getByRole } = render(<DxcNewDate onBlur={onBlur} />);
+    const { getByRole } = render(<DxcDateInput onBlur={onBlur} optional />);
     const input = getByRole("textbox");
 
     userEvent.type(input, "test");
@@ -178,8 +193,32 @@ describe("NewDate component tests", () => {
     expect(onBlur).toHaveBeenCalledWith({ value: "", error: null, date: null });
   });
 
-  test("Disabled date (calendar action must be shown but not clickable)", () => {
-    const { getByRole, queryByText } = render(<DxcNewDate disabled />);
+  test("onBlur & onChange functions error: required field (not optional)", () => {
+    const onBlur = jest.fn();
+    const onChange = jest.fn();
+    const { getByRole } = render(<DxcDateInput onBlur={onBlur} onChange={onChange} />);
+    const date = getByRole("textbox");
+
+    userEvent.type(date, "t");
+    expect(date.value).toBe("t");
+    userEvent.clear(date);
+    fireEvent.blur(date);
+    expect(onBlur).toHaveBeenCalled();
+    expect(onBlur).toHaveBeenCalledWith({
+      value: "",
+      error: "This field is required. Please, enter a value.",
+      date: null,
+    });
+    expect(onChange).toHaveBeenCalled();
+    expect(onChange).toHaveBeenCalledWith({
+      value: "",
+      error: "This field is required. Please, enter a value.",
+      date: null,
+    });
+  });
+
+  test("Disabled date input (calendar action must be shown but not clickable)", () => {
+    const { getByRole, queryByText } = render(<DxcDateInput disabled />);
     const calendarAction = getByRole("button");
     const d = new Date();
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -190,7 +229,7 @@ describe("NewDate component tests", () => {
   });
 
   test("Input has correct accesibility attributes", () => {
-    const { getByRole } = render(<DxcNewDate label="Date label" />);
+    const { getByRole } = render(<DxcDateInput label="Date input label" />);
     const input = getByRole("textbox");
     expect(input.getAttribute("aria-autocomplete")).toBeNull();
     expect(input.getAttribute("aria-controls")).toBeNull();

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>);