Change sources type to standard array

Nayni · Nayni · commit 2f1a6ae52936 · 2019-02-01T20:08:15.000+01:00
diff --git a/types/graphql-resolve-batch/graphql-resolve-batch-tests.ts b/types/graphql-resolve-batch/graphql-resolve-batch-tests.ts
@@ -57,7 +57,7 @@ const withSourceAndResultTyped = createBatchResolver<
     SomeTestSource,
     SomeTestResult
 >((sources, _, __) => {
-    // $ExpectType ReadonlyArray<SomeTestSource>
+    // $ExpectType SomeTestSource[]
     const verifySources = sources;
 
     return batchFunction(sources);
@@ -68,7 +68,7 @@ const withSourceAndResultTypedAsPromise = createBatchResolver<
     SomeTestSource,
     SomeTestResult
 >(async (sources, _, __) => {
-    // $ExpectType ReadonlyArray<SomeTestSource>
+    // $ExpectType SomeTestSource[]
     const verifySources = sources;
     const result = await asyncBatchFunction(sources);
     return result;
@@ -80,7 +80,7 @@ const withSourceAndArgsAndResultTyped = createBatchResolver<
     SomeTestResult,
     SomeTestArgs
 >(async (sources, args, _) => {
-    // $ExpectType ReadonlyArray<SomeTestSource>
+    // $ExpectType SomeTestSource[]
     const verifySources = sources;
     // $ExpectType string
     const verifyArgs = args.someArg;
@@ -96,7 +96,7 @@ const withSourceAndArgsAndContextTyped = createBatchResolver<
     SomeTestArgs,
     SomeTestContext
 >(async (sources, args, context, info) => {
-    // $ExpectType ReadonlyArray<SomeTestSource>
+    // $ExpectType SomeTestSource[]
     const verifySources = sources;
     // $ExpectType string
     const verifyArgs = args.someArg;
@@ -112,7 +112,7 @@ const withSourceAndArgsAndContextTyped = createBatchResolver<
 // $ExpectType ResolverFunction<SomeTestSource, any, any, SomeTestResult[]>
 const withResultIsArray = createBatchResolver<SomeTestSource, SomeTestResult[]>(
     (sources, _, __) => {
-        // $ExpectType ReadonlyArray<SomeTestSource>
+        // $ExpectType SomeTestSource[]
         const verifySources = sources;
 
         return asyncBatchFunctionWhenTReturnIsArray(sources);
diff --git a/types/graphql-resolve-batch/index.d.ts b/types/graphql-resolve-batch/index.d.ts
@@ -4,7 +4,7 @@
 // Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
 // TypeScript Version: 2.6
 
-import { GraphQLResolveInfo } from 'graphql';
+import { GraphQLResolveInfo } from "graphql";
 
 /**
  * Creates a GraphQL.js field resolver that batches together multiple resolves
@@ -45,7 +45,7 @@ export type ResolverFunction<TSource, TArgs, TContext, TReturn> = (
  * A batch function to resolve all fields for the given sources in a single batch.
  */
 export type BatchResolveFunction<TSource, TArgs, TContext, TReturn> = (
-    sources: ReadonlyArray<TSource>,
+    sources: TSource[],
     args: TArgs,
     context: TContext,
     info: GraphQLResolveInfo
