Skip to content

Commit 8e2db7b

Browse files
committed
Add error for passing null for non-nullable argument
1 parent d089e70 commit 8e2db7b

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

src/execution/execute.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -551,6 +551,14 @@ function selectionSetWithFragmentArgumentsApplied(
551551
const variableName = argDef.variable.name.value;
552552
const providedArg = providedArguments.get(variableName);
553553
if (providedArg != null) {
554+
if (providedArg.value.kind === Kind.NULL && isNonNullType(argDef.type)) {
555+
exeContext.errors.push(
556+
new GraphQLError(
557+
`Fragment argument "$${variableName}" on fragment "${fragment.name.value}" is non-null, but null was provided.`,
558+
[providedArg, argDef],
559+
),
560+
);
561+
}
554562
fragmentArgumentValues.set(variableName, providedArg.value);
555563
} else if (argDef.defaultValue != null) {
556564
fragmentArgumentValues.set(variableName, argDef.defaultValue);

0 commit comments

Comments
 (0)