Skip to content

Commit

Permalink
fix missing invariant fieldName param
Browse files Browse the repository at this point in the history
Reviewed By: captbaritone

Differential Revision: D48702412

fbshipit-source-id: 694a3711cd4008bd404ca83d679e504adb87a1c2
  • Loading branch information
marchaase-meta authored and facebook-github-bot committed Aug 28, 2023
1 parent 5d0887b commit 5044a2e
Showing 1 changed file with 11 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,9 @@ function resolverDataInjector<TFragmentType: FragmentType, TData: ?{...}>(
if (isRequiredField === true) {
invariant(
false,
'Expected required resolver field `%` to be present. But resolvers fragment data is null/undefined.',
'Expected required resolver field `%s` in fragment `%s` to be present. But resolvers fragment data is null/undefined.',
fieldName,
fragment.name,
);
} else {
return resolverFn(null, args);
Expand All @@ -60,15 +61,22 @@ function resolverDataInjector<TFragmentType: FragmentType, TData: ?{...}>(
if (isRequiredField === true) {
invariant(
data[fieldName] != null,
'Expected required resolver field `%` to be non-null.',
'Expected required resolver field `%s` in fragment `%s` to be non-null.',
fieldName,
fragment.name,
);
}

return resolverFn(data[fieldName], args);
} else {
// If both `data` and `fieldName` is available, we expect the
// `fieldName` field in the `data` object.
invariant(false, 'Missing field `%` in resolver response.', fieldName);
invariant(
false,
'Missing field `%s` in fragment `%s` in resolver response.',
fieldName,
fragment.name,
);
}
} else {
// By default we will pass the full set of the fragment data to the resolver
Expand Down

0 comments on commit 5044a2e

Please sign in to comment.