Remove superficial usages of 'Array.from' (#3042)

.flowconfig

@@ -33,7 +33,6 @@ export-renamed-default=error
 [options]
 all=true
 module.use_strict=true
-babel_loose_array_spread=true
 experimental.const_params=true
 include_warnings=true
 no_flowlib=true

src/__testUtils__/__tests__/genFuzzStrings-test.js

@@ -7,7 +7,7 @@ function expectFuzzStrings(options: {|
   allowedChars: Array<string>,
   maxLength: number,
 |}) {
-  return expect(Array.from(genFuzzStrings(options)));
+  return expect([...genFuzzStrings(options)]);
 }
 
 describe('genFuzzStrings', () => {

src/type/schema.js

@@ -203,7 +203,7 @@ export class GraphQLSchema {
     // Keep track of all implementations by interface name.
     this._implementationsMap = Object.create(null);
 
-    for (const namedType of Array.from(allReferencedTypes)) {
+    for (const namedType of allReferencedTypes) {
       if (namedType == null) {
         continue;
       }

src/utilities/astFromValue.js

@@ -62,9 +62,7 @@ export function astFromValue(value: mixed, type: GraphQLInputType): ?ValueNode {
     const itemType = type.ofType;
     if (isIterableObject(value)) {
       const valuesNodes = [];
-      // Since we transpile for-of in loose mode it doesn't support iterators
-      // and it's required to first convert iterable into array
-      for (const item of Array.from(value)) {
+      for (const item of value) {
         const itemNode = astFromValue(item, itemType);
         if (itemNode != null) {
           valuesNodes.push(itemNode);

src/validation/rules/FieldsOnCorrectTypeRule.js

@@ -105,7 +105,7 @@ function getSuggestedTypeNames(
     }
   }
 
-  return Array.from(suggestedTypes)
+  return [...suggestedTypes]
     .sort((typeA, typeB) => {
       // Suggest both interface and object types based on how common they are.
       const usageCountDiff = usageCount[typeB.name] - usageCount[typeA.name];