From f1350946c3d1d9765f776c465df33685d959ce07 Mon Sep 17 00:00:00 2001 From: DavisCacici <71649801+DavisCacici@users.noreply.github.com> Date: Tue, 11 Feb 2025 14:07:29 +0100 Subject: [PATCH] Update resolvers.ts, handle error when "resolver" variable in addObject is undefined I suggest this update to the addObject method: if the resolver is undefined, provide a clear message to the user. --- packages/graphql-modules/src/module/resolvers.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/packages/graphql-modules/src/module/resolvers.ts b/packages/graphql-modules/src/module/resolvers.ts index f74bebcc99..29f76f14a7 100644 --- a/packages/graphql-modules/src/module/resolvers.ts +++ b/packages/graphql-modules/src/module/resolvers.ts @@ -258,6 +258,13 @@ function addObject({ if (fields.hasOwnProperty(fieldName)) { const resolver = fields[fieldName]; + if(typeof resolver === 'undefined'){ + throw new ResolverDuplicatedError( + `Resolver not implement of "${typeName}.${fieldName}"`, + useLocation({ dirname: config.dirname, id: config.id }) + ); + } + if (isResolveFn(resolver)) { if (container[typeName][fieldName]) { throw new ResolverDuplicatedError(