@dxc-technology/halstack-react 0.0.0-92f4a00 → 0.0.0-9427b76

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxc-technology/halstack-react",
-  "version": "0.0.0-92f4a00",
+  "version": "0.0.0-9427b76",
   "description": "DXC Halstack React components library",
   "repository": "dxc-technology/halstack-react",
   "homepage": "http://developer.dxc.com/tools/react",
@@ -11,6 +11,7 @@
     "url": "https://dxc.com"
   },
   "main": "./dist/main.js",
+  "types": "./dist/main.d.ts",
   "peerDependencies": {
     "react": "^16.8.6",
     "react-dom": "^16.8.6",

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/NewDate.test.js

@@ -14,7 +14,7 @@ describe("NewDate component tests", () => {
     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("");
@@ -22,17 +22,29 @@ describe("NewDate component tests", () => {
 
   test("Renders with personalized error", () => {
     const { getByText } = render(<DxcNewDate 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(<DxcNewDate 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(<DxcNewDate 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", () => {
@@ -68,7 +80,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(<DxcNewDate onBlur={onBlur} />);
     const calendarAction = getByRole("button");
     const d = new Date();
     const options = { weekday: "short", month: "short", day: "numeric" };
@@ -76,7 +89,8 @@ 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();
   });
@@ -86,7 +100,7 @@ describe("NewDate component tests", () => {
     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);
@@ -121,7 +135,7 @@ describe("NewDate component tests", () => {
     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 });
@@ -132,12 +146,11 @@ describe("NewDate component tests", () => {
     const onChange = jest.fn();
     const { getByRole } = render(<DxcNewDate 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 });
@@ -164,7 +177,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(<DxcNewDate onBlur={onBlur} optional />);
     const input = getByRole("textbox");
 
     userEvent.type(input, "test");
@@ -178,6 +191,22 @@ describe("NewDate component tests", () => {
     expect(onBlur).toHaveBeenCalledWith({ value: "", error: null, date: null });
   });
 
+  test("onBlur & onChange functions error: required field (not optional)", () => {
+    const onBlur = jest.fn();
+    const onChange = jest.fn();
+    const { getByRole } = render(<DxcNewDate 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 (calendar action must be shown but not clickable)", () => {
     const { getByRole, queryByText } = render(<DxcNewDate disabled />);
     const calendarAction = getByRole("button");