vektah / gqlparser / 930

87%

master: 87%

Build Type

Pull #436

circleci

Committed by alex-dvoretsky_catonian

Commit Message

fix: reject duplicate enum values and repeated non-repeatable directives

LoadSchema silently merged duplicate enum values (no uniqueness check),
unlike object fields which already error with "Field X.Y can only be
defined once." Add the equivalent check for enum values, reported as
"Enum value X.Y can only be defined once."

Also enforce that a non-repeatable directive is applied at most once per
schema location during schema validation (the executable-document rule
in validator/rules already covers queries, but the type system was
unchecked). The check is scoped to single authored locations via a
singleLocation flag: a type's own directives are merged across its base
definition and every extension, which the spec treats as distinct
locations, so the merged list is exempt to avoid rejecting a directive
used once per location.

Adds YAML cases covering duplicate enum values (same definition and
across an extension), a repeated non-repeatable directive on a field and
on an enum value, and positive cases (repeatable directive, the same
directive on distinct locations, and a directive on a type plus its
extension).

Fixes #435

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #436: fix: reject duplicate enum values and repeated non-repeatable directives

Coverage Stats

35 of 39 new or added lines in 1 file covered. (89.74%)

7 existing lines in 1 file now uncovered.

5080 of 5816 relevant lines covered (87.35%)

1.01 hits per line