4
96%
master: 96%

Ran 21 May 2026 12:38PM UTC

Files 14

Run time 2s

Badge

Embed ▾

Committed 21 May 2026 12:36PM UTC coverage: 95.604% (-0.1%) from 95.701%

Job # 26226312343.4

Build Type

push

github

Committed by

web-flow

Commit Message

fix(validators): handle pointer fields in RequiredIf (#334) (#335)

RequiredIf silently skipped its check whenever either side of the rule
was a pointer:

  - When the destination field is a pointer (e.g. *bool), reflect.ValueOf
    returns a value of Kind Pointer. convTypeByBaseKind has no
    string-to-Pointer conversion, so dstVal == wantVal was never
    evaluated and the function fell through to default-True / skip.

  - When the source field is a pointer to a zero value (e.g.
    *string("")), val != nil && !IsEmpty(val) was true, so the
    rule's 'must be present and not empty' contract was not enforced.

Unwrap pointers on both sides: treat a nil pointer as the field being
absent (rule does not trigger, dependent field remains optional), and
treat a pointer to a zero value as the zero value itself for the
not-empty check. Adds five subtests covering each pointer scenario;
two of them fail on the old code and pass on the new code.

Closes #334

Coverage Stats

3001 of 3139 relevant lines covered (95.6%)

1.11 hits per line

gookit / validate / 26226312343 / 4
96%
master: 96%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job Go-1.22 - 26226312343.4

gookit / validate / 26226312343 / 4 96% master: 96%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job Go-1.22 - 26226312343.4

gookit / validate / 26226312343 / 4
96%
master: 96%

README BADGES
x