Enum about schema-to-yup

[kristianmandrup]

I will add a test for it now and see if I can fix it ;)

[kristianmandrup]

If you look at types/mixed.js you can see it references this.constraints.enum
There are string tests for oneOf and notOneOf. The use of enum should just be an alias.

  oneOf() {
    let value =
      this.constraints.enum || this.constraints.oneOf || this.constraints.anyOf;
    if (this.isNothing(value)) return this;
    value = Array.isArray(value) ? value : [value];
    return this.addConstraint("oneOf", { value, errName: "enum" });
  }

  notOneOf() {
    const { not, notOneOf } = this.constraints;
    let value = notOneOf || (not && (not.enum || not.oneOf));
    value = Array.isArray(value) ? value : [value];
    return this.addConstraint("notOneOf", { value });
  }

As you can see, String just extends Mixed

class YupString extends YupMixed {
  constructor(obj) {
    super(obj);
    this.type = "string";
    this.base = this.yup.string();
  }

  static create(obj) {
    return new YupString(obj);
  }

  convert() {
    // generic constraint building, including oneOf (enum) etc
    super.convert();
    // ... string specific constraint building
  }

  // ...
}

[kristianmandrup]

Pls check my latest commit to master (better string testing) and debug from there. I see some tests are currently broken. Sorry.

[kristianmandrup]

Fixed this issue and all string related issues in latest release :)