Convert error to warning for non-compliant use of __

This unbreaks various uses of graphql-js which were defining fields with __ while retaining some form of reporting warning for non-compliant use.

Fixes #690

Author: leebyron

src/type/__tests__/validation-test.js

@@ -341,15 +341,27 @@ describe('Type System: Objects must have fields', () => {
     );
   });
 
-  it('rejects an Object type with reserved named fields', () => {
-    expect(
-      () => schemaWithFieldType(new GraphQLObjectType({
+  it('warns when an Object type with reserved named fields', () => {
+    /* eslint-disable no-console */
+    const prevConsoleError = console.error;
+    const calls = [];
+    console.error = function () {
+      calls.push(Array.prototype.slice.call(arguments));
+    };
+    try {
+      schemaWithFieldType(new GraphQLObjectType({
         name: 'SomeObject',
         fields: { __notPartOfIntrospection: { type: GraphQLString } }
-      }))
-    ).to.throw(
-      'Name "__notPartOfIntrospection" must not begin with "__", which is reserved by GraphQL introspection.'
-    );
+      }));
+
+      expect(calls[0][0]).contains([
+        [ 'GraphQL: Name "__notPartOfIntrospection" must not begin with ' +
+          '"__", which is reserved by GraphQL introspection.' ]
+      ]);
+    } finally {
+      console.error = prevConsoleError;
+    }
+    /* eslint-enable no-console */
   });
 
   it('rejects an Object type with incorrectly typed fields', () => {

src/utilities/assertValidName.js

@@ -10,6 +10,9 @@
 
 const NAME_RX = /^[_a-zA-Z][_a-zA-Z0-9]*$/;
 
+// Ensures console warnings are only issued once.
+let hasWarnedAboutDunder = false;
+
 /**
  * Upholds the spec rules about naming.
  */
@@ -22,11 +25,20 @@ export function assertValidName(
       `Must be named. Unexpected name: ${name}.`
     );
   }
-  if (!isIntrospection && name.slice(0, 2) === '__') {
-    throw new Error(
-      `Name "${name}" must not begin with "__", which is reserved by ` +
-      'GraphQL introspection.'
-    );
+  if (!isIntrospection && name.slice(0, 2) === '__' && !hasWarnedAboutDunder) {
+    hasWarnedAboutDunder = true;
+    /* eslint-disable no-console */
+    if (console && console.error) {
+      const stack = new Error().stack;
+      const stackOnly = stack ?
+        '\n' + stack.split('\n').slice(1).join('\n') :
+        '';
+      console.error(
+        `GraphQL: Name "${name}" must not begin with "__", which is ` +
+        'reserved by GraphQL introspection.' + stackOnly
+      );
+    }
+    /* eslint-enable no-console */
   }
   if (!NAME_RX.test(name)) {
     throw new Error(