Provide reasonable default version of 'parseLiteral'

Author: IvanGoncharov

src/type/definition.js

@@ -12,6 +12,7 @@ import invariant from '../jsutils/invariant';
 import isNullish from '../jsutils/isNullish';
 import * as Kind from '../language/kinds';
 import { assertValidName } from '../utilities/assertValidName';
+import { scalarValueFromAST } from '../utilities/scalarValueFromAST';
 import type {
   ScalarTypeDefinitionNode,
   ObjectTypeDefinitionNode,
@@ -345,7 +346,10 @@ export class GraphQLScalarType {
   // Parses an externally provided value to use as an input.
   parseValue(value: mixed): mixed {
     const parser = this._scalarConfig.parseValue;
-    return parser && !isNullish(value) ? parser(value) : undefined;
+    if (isNullish(value)) {
+      return undefined;
+    }
+    return parser ? parser(value) : value;
   }
 
   // Determines if an internal value is valid for this type.
@@ -357,7 +361,7 @@ export class GraphQLScalarType {
   // Parses an externally provided literal value to use as an input.
   parseLiteral(valueNode: ValueNode): mixed {
     const parser = this._scalarConfig.parseLiteral;
-    return parser ? parser(valueNode) : undefined;
+    return (parser || scalarValueFromAST)(valueNode);
   }
 
   toString(): string {

src/utilities/__tests__/scalarValueFromAST-test.js

@@ -0,0 +1,77 @@
+/**
+ *  Copyright (c) 2017, Facebook, Inc.
+ *  All rights reserved.
+ *
+ *  This source code is licensed under the BSD-style license found in the
+ *  LICENSE file in the root directory of this source tree. An additional grant
+ *  of patent rights can be found in the PATENTS file in the same directory.
+ */
+
+import { describe, it } from 'mocha';
+import { expect } from 'chai';
+import { scalarValueFromAST } from '../scalarValueFromAST';
+import { parseValue } from '../../language';
+
+describe('scalarValueFromAST', () => {
+
+  function testCase(valueText, expected) {
+    expect(
+      scalarValueFromAST(parseValue(valueText))
+    ).to.deep.equal(expected);
+  }
+
+  function testNegativeCase(valueText, errorMsg) {
+    expect(
+      () => scalarValueFromAST(parseValue(valueText))
+    ).to.throw(errorMsg);
+  }
+
+  it('parses simple values', () => {
+    testCase('null', null);
+    testCase('true', true);
+    testCase('false', false);
+    testCase('123', 123);
+    testCase('123.456', 123.456);
+    testCase('"abc123"', 'abc123');
+  });
+
+  it('parses lists of values', () => {
+    testCase('[true, false]', [ true, false ]);
+    testCase('[true, 123.45]', [ true, 123.45 ]);
+    testCase('[true, null]', [ true, null ]);
+    testCase('[true, ["foo", 1.2]]', [ true, [ 'foo', 1.2 ] ]);
+  });
+
+  it('parses input objects', () => {
+    testCase(
+      '{ int: 123, requiredBool: false }',
+      { int: 123, requiredBool: false }
+    );
+    testCase(
+      '{ foo: [{ bar: "baz"}]}',
+      { foo: [ { bar: 'baz'} ] }
+    );
+  });
+
+  it('rejects enum values and query variables', () => {
+    testNegativeCase('TEST_ENUM_VALUE', 'Scalar value can not contain Enum.');
+    testNegativeCase(
+      '$test_variable',
+      'Scalar value can not contain Query variable.'
+    );
+    testNegativeCase('[TEST_ENUM_VALUE]', 'Scalar value can not contain Enum.');
+    testNegativeCase(
+      '[$test_variable]',
+      'Scalar value can not contain Query variable.'
+    );
+    testNegativeCase(
+      '{foo: TEST_ENUM_VALUE}',
+      'Scalar value can not contain Enum.'
+    );
+    testNegativeCase(
+      '{bar: $test_variable}',
+      'Scalar value can not contain Query variable.'
+    );
+  });
+
+});

src/utilities/buildASTSchema.js

@@ -448,12 +448,6 @@ export function buildASTSchema(ast: DocumentNode): GraphQLSchema {
       description: getDescription(def),
       astNode: def,
       serialize: () => null,
-      // Note: validation calls the parse functions to determine if a
-      // literal value is correct. Returning null would cause use of custom
-      // scalars to always fail validation. Returning false causes them to
-      // always pass validation.
-      parseValue: () => false,
-      parseLiteral: () => false,
     });
   }
 

src/utilities/buildClientSchema.js

@@ -227,12 +227,6 @@ export function buildClientSchema(
       name: scalarIntrospection.name,
       description: scalarIntrospection.description,
       serialize: id => id,
-      // Note: validation calls the parse functions to determine if a
-      // literal value is correct. Returning null would cause use of custom
-      // scalars to always fail validation. Returning false causes them to
-      // always pass validation.
-      parseValue: () => false,
-      parseLiteral: () => false,
     });
   }
 

src/utilities/extendSchema.js

@@ -499,12 +499,6 @@ export function extendSchema(
       description: getDescription(typeNode),
       astNode: typeNode,
       serialize: id => id,
-      // Note: validation calls the parse functions to determine if a
-      // literal value is correct. Returning null would cause use of custom
-      // scalars to always fail validation. Returning false causes them to
-      // always pass validation.
-      parseValue: () => false,
-      parseLiteral: () => false,
     });
   }
 

src/utilities/index.js

@@ -58,6 +58,10 @@ export { valueFromAST } from './valueFromAST';
 // Create a GraphQL language AST from a JavaScript value.
 export { astFromValue } from './astFromValue';
 
+// Create a JavaScript value from a GraphQL language AST representation
+// of Scalar.
+export { scalarValueFromAST } from './scalarValueFromAST';
+
 // A helper to use within recursive-descent visitors which need to be aware of
 // the GraphQL type system.
 export { TypeInfo } from './TypeInfo';

src/utilities/scalarValueFromAST.js

@@ -0,0 +1,43 @@
+/* @flow */
+/**
+ *  Copyright (c) 2017, Facebook, Inc.
+ *  All rights reserved.
+ *
+ *  This source code is licensed under the BSD-style license found in the
+ *  LICENSE file in the root directory of this source tree. An additional grant
+ *  of patent rights can be found in the PATENTS file in the same directory.
+ */
+
+import keyValMap from '../jsutils/keyValMap';
+import * as Kind from '../language/kinds';
+import type { ValueNode } from '../language/ast';
+
+/**
+ * Create a JavaScript value from a GraphQL language AST representation
+ * of Scalar.
+ */
+export function scalarValueFromAST(astValue: ValueNode): mixed {
+  switch (astValue.kind) {
+    case Kind.NULL:
+      return null;
+    case Kind.INT:
+      return parseInt(astValue.value, 10);
+    case Kind.FLOAT:
+      return parseFloat(astValue.value);
+    case Kind.STRING:
+    case Kind.BOOLEAN:
+      return astValue.value;
+    case Kind.LIST:
+      return astValue.values.map(scalarValueFromAST);
+    case Kind.OBJECT:
+      return keyValMap(
+        astValue.fields,
+        field => field.name.value,
+        field => scalarValueFromAST(field.value),
+      );
+    case Kind.ENUM:
+      throw new Error('Scalar value can not contain Enum.');
+    case Kind.VARIABLE:
+      throw new Error('Scalar value can not contain Query variable.');
+  }
+}