Revert "Remove all 'instanceof GraphQLSchema' checks" (#377)

graphql · May 5, 2016 · 2ac41f6 · 2ac41f6
1 parent 71df461
commit 2ac41f6
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 0 deletions.
diff --git a/src/execution/execute.js b/src/execution/execute.js
@@ -117,6 +117,11 @@ export function execute(
   operationName?: ?string
 ): Promise<ExecutionResult> {
   invariant(schema, 'Must provide schema');
+  invariant(
+    schema instanceof GraphQLSchema,
+    'Schema must be an instance of GraphQLSchema. Also ensure that there are ' +
+    'not multiple versions of GraphQL installed in your node_modules directory.'
+  );
 
   // If a valid context cannot be created due to incorrect arguments,
   // this will throw an error.

diff --git a/src/utilities/extendSchema.js b/src/utilities/extendSchema.js
@@ -94,6 +94,10 @@ export function extendSchema(
   schema: GraphQLSchema,
   documentAST: Document
 ): GraphQLSchema {
+  invariant(
+    schema instanceof GraphQLSchema,
+    'Must provide valid GraphQLSchema'
+  );
 
   invariant(
     documentAST && documentAST.kind === DOCUMENT,

diff --git a/src/validation/validate.js b/src/validation/validate.js
@@ -53,6 +53,11 @@ export function validate(
 ): Array<GraphQLError> {
   invariant(schema, 'Must provide schema');
   invariant(ast, 'Must provide document');
+  invariant(
+    schema instanceof GraphQLSchema,
+    'Schema must be an instance of GraphQLSchema. Also ensure that there are ' +
+    'not multiple versions of GraphQL installed in your node_modules directory.'
+  );
   const typeInfo = new TypeInfo(schema);
   return visitUsingRules(schema, typeInfo, ast, rules || specifiedRules);
 }