add enforce.loose()

[ealush]

enforce.shape() supports defining an object schema, but it requires an exact match of an object. If the object contains extra keys, it will throw an error.

enforce.loose should implement the same behaviour of enforce.shape, but also allow extra keys to be present in the data object.

See Shape for reference.

[kapalex]

What do you think of the following implementation ?

export function shape(obj, shapeObj, loose) {
  for (const key in shapeObj) {
    const current = shapeObj[key];
    const value = obj[key];

    if (
      !asArray(current).every(fn =>
        fn(fn.name === optional.name ? [obj, key] : value)
      )
    ) {
      return false;
    }
  }

  if (!loose) {
    for (const key in obj) {
      if (!shapeObj[key]) {
        return false;
      }
    }
  }

  return true;
}

export function loose(obj, shapeObj) {
  return shape(obj, shapeObj, true);
}

Would also allow using shape with an additional param to get the same result (might be more clear than loose)

[ealush]

Yes, that's pretty much what I had in mind. I would argue though that instead of a boolean argument, it might be better an options object with a loose property. This will prevent breaking changes if we want to add more capabilities in the future.

[ealush]

@kapalex If you still want to take this issue, you can go ahead. I am done implementing my breaking changes :)

[kapalex]

Should get to it today or tomorrow 😄

[ealush]

closed by #505