made unwrapErrorHandler more generic & updated test to use a custom validate function

Author: lpillonel

packages/core/src/validate.js

@@ -131,10 +131,11 @@ function unwrapErrorHandler(errorHandler) {
  return Object.keys(errorHandler).reduce((acc, key) => {
  if (key === "addError") {
  return acc;
- } else if (key === "__errors") {
+ } else if (isPlainObject(errorHandler[key])) {
+ return { ...acc, [key]: unwrapErrorHandler(errorHandler[key]) };
+ } else {
  return { ...acc, [key]: errorHandler[key] };
  }
- return { ...acc, [key]: unwrapErrorHandler(errorHandler[key]) };
  }, {});
 }
 

packages/core/test/Form_test.js

@@ -3422,20 +3422,31 @@ describe("Form omitExtraData and liveOmit", () => {
  it("should ignore extra array properties passed within the errorSchema", () => {
  const schema = {
  type: "object",
+ required: ["foo", "bar"],
  properties: {
- foo: { type: "string" },
+ foo: { type: "string", minLength: 10 },
+ bar: { type: "string" },
  },
  };
 
- const extraErrors = {
- __warnings: ["a warning that should be ignored"],
- foo: {
- __errors: ["foo", "bar"],
- __warnings: ["another warning"],
- },
+ const formData = {
+ foo: "dummy",
  };
 
- const { node } = createFormComponent({ schema, extraErrors });
+ const validate = (formData, errors) => {
+ if (formData.foo === "dummy") {
+ errors.foo.__warnings = ["value might be too common"];
+ }
+
+ return errors;
+ };
+
+ const { node } = createFormComponent({
+ schema,
+ formData,
+ validate,
+ liveValidate: true,
+ });
 
  expect(node.querySelectorAll(".error-detail li")).to.have.length.of(2);
  });