move guard to appropriate location (#103)

guard only necessary for those using graphql-execute as drop-in replacement for graphql@15
yaacovCR · Dec 5, 2021 · 719ef94 · 719ef94
1 parent d5bb5e4
commit 719ef94
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/execution/execute.ts b/src/execution/execute.ts
@@ -1,6 +1,7 @@
 import type { PromiseOrValue } from '../jsutils/PromiseOrValue';
 import { isPromise } from '../jsutils/isPromise';
 import { isAsyncIterable } from '../jsutils/isAsyncIterable';
+import { devAssert } from '../jsutils/devAssert';
 
 import type {
   ExecutionArgs,
@@ -24,6 +25,12 @@ export function execute(
 ): PromiseOrValue<
   ExecutionResult | AsyncGenerator<AsyncExecutionResult, void, void>
 > {
+  // Temporary for v15 to v16 migration. Remove in v17
+  devAssert(
+    arguments.length < 2,
+    'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+  );
+
   const executor = new Executor();
   return executor.execute(args);
 }

diff --git a/src/execution/executor.ts b/src/execution/executor.ts
@@ -257,12 +257,6 @@ export class Executor {
   ): PromiseOrValue<
     ExecutionResult | AsyncGenerator<AsyncExecutionResult, void, void>
   > {
-    // Temporary for v15 to v16 migration. Remove in v17
-    devAssert(
-      arguments.length < 2,
-      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
-    );
-
     const exeContext = this.buildExecutionContext(args);
 
     // If a valid execution context cannot be created due to incorrect arguments,