gookit / validate / 40864170708

91%

push

github

fix(validate): resolve an acronym-style field name ignoring case

A cross-field rule (required_without_all, required_if, eq_field ...) and the getters
resolved a name with UpperFirst only, so "md5" became "Md5" and missed the field MD5
(likewise "url" for URL): the rule then treated the field as an empty value, which
produced a confusing required error (#347).

- StructData.tryGetRV falls back to a case-insensitive match over the struct fields
  when the exact and UpperFirst names both miss; the first declared field wins and
  only the first segment of a path is matched.
- the enum rules report an unsupported map value instead of "value must be in the
  enum [...]", which reads like the value is simply not in the list.

The first attempt of this change reported every rule field that HasField did not know,
which broke legitimate uses (a lower-case name, a dotted path into a nil nested
pointer, the paths of an empty slice), so this resolution fix replaces it.

Tests: the #347 struct with lower-case rule names, the getters with all casings, an
unknown field still missing, and the map message of the enum rules.

36 of 50 new or added lines in 3 files covered. (72.0%)

4286 of 4713 relevant lines covered (90.94%)

5.97 hits per line