Remove unnecessary async code, part 2 (#271)

Continuation of #269
graphql · Jun 25, 2020 · bae5ead · bae5ead
1 parent 5b42850
commit bae5ead
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/src/node/__tests__/plural.js b/src/node/__tests__/plural.js
@@ -7,7 +7,7 @@ import {
   GraphQLObjectType,
   GraphQLSchema,
   GraphQLString,
-  graphql,
+  graphqlSync,
 } from 'graphql';
 
 import { pluralIdentifyingRootField } from '../plural';
@@ -47,7 +47,7 @@ const schema = new GraphQLSchema({
 const context = { lang: 'en' };
 
 describe('pluralIdentifyingRootField()', () => {
-  it('allows fetching', async () => {
+  it('allows fetching', () => {
     const query = `
       {
         usernames(usernames:[ "dschafer", "leebyron", "schrockn" ]) {
@@ -57,7 +57,7 @@ describe('pluralIdentifyingRootField()', () => {
       }
     `;
 
-    expect(await graphql(schema, query, null, context)).to.deep.equal({
+    expect(graphqlSync(schema, query, null, context)).to.deep.equal({
       data: {
         usernames: [
           {
@@ -77,7 +77,7 @@ describe('pluralIdentifyingRootField()', () => {
     });
   });
 
-  it('correctly introspects', async () => {
+  it('correctly introspects', () => {
     const query = `
       {
         __schema {
@@ -113,7 +113,7 @@ describe('pluralIdentifyingRootField()', () => {
       }
     `;
 
-    expect(await graphql(schema, query)).to.deep.equal({
+    expect(graphqlSync(schema, query)).to.deep.equal({
       data: {
         __schema: {
           queryType: {

diff --git a/src/node/plural.js b/src/node/plural.js
@@ -24,24 +24,24 @@ type PluralIdentifyingRootFieldConfig = {|
 export function pluralIdentifyingRootField(
   config: PluralIdentifyingRootFieldConfig,
 ): GraphQLFieldConfig<mixed, mixed> {
-  const inputArgs = {};
   let inputType = config.inputType;
   if (isNonNullType(inputType)) {
     inputType = inputType.ofType;
   }
-  inputArgs[config.argName] = {
-    type: new GraphQLNonNull(new GraphQLList(new GraphQLNonNull(inputType))),
-  };
   return {
     description: config.description,
     type: new GraphQLList(config.outputType),
-    args: inputArgs,
+    args: {
+      [config.argName]: {
+        type: new GraphQLNonNull(
+          new GraphQLList(new GraphQLNonNull(inputType)),
+        ),
+      },
+    },
     resolve(_obj, args, context, info) {
       const inputs = args[config.argName];
-      return Promise.all(
-        inputs.map((input) =>
-          Promise.resolve(config.resolveSingleInput(input, context, info)),
-        ),
+      return inputs.map((input) =>
+        config.resolveSingleInput(input, context, info),
       );
     },
   };