Modernize apollo-utilities graphql transformation functionality

package.json

@@ -91,7 +91,7 @@
   "devDependencies": {
     "@octokit/rest": "15.18.1",
     "@types/benchmark": "1.0.31",
-    "@types/graphql": "0.12.7",
+    "@types/graphql": "14.0.3",
     "@types/isomorphic-fetch": "0.0.34",
     "@types/jest": "23.3.10",
     "@types/lodash": "4.14.119",
@@ -105,7 +105,7 @@
     "danger": "4.4.10",
     "fetch-mock": "7.2.5",
     "flow-bin": "0.88.0",
-    "graphql": "0.13.2",
+    "graphql": "14.0.2",
     "graphql-tag": "2.10.0",
     "isomorphic-fetch": "2.2.1",
     "jest": "23.6.0",

packages/apollo-cache/src/utils.ts

@@ -71,21 +71,17 @@ function selectionSetFromObj(obj: any): SelectionSetNode {
   const selections: FieldNode[] = [];
 
   Object.keys(obj).forEach(key => {
+    const nestedSelSet: SelectionSetNode = selectionSetFromObj(obj[key]);
+
     const field: FieldNode = {
       kind: 'Field',
       name: {
         kind: 'Name',
         value: key,
       },
+      selectionSet: nestedSelSet || undefined,
     };
 
-    // Recurse
-    const nestedSelSet: SelectionSetNode = selectionSetFromObj(obj[key]);
-
-    if (nestedSelSet) {
-      field.selectionSet = nestedSelSet;
-    }
-
     selections.push(field);
   });
 

packages/apollo-client/src/core/ObservableQuery.ts

@@ -22,7 +22,7 @@ import { QueryStoreValue } from '../data/queries';
 
 export type ApolloCurrentResult<T> = {
   data: T | {};
-  errors?: GraphQLError[];
+  errors?: ReadonlyArray<GraphQLError>;
   loading: boolean;
   networkStatus: NetworkStatus;
   error?: ApolloError;
@@ -228,7 +228,8 @@ export class ObservableQuery<
   public isDifferentFromLastResult(newResult: ApolloQueryResult<TData>) {
     const { lastResultSnapshot: snapshot } = this;
     return !(
-      snapshot && newResult &&
+      snapshot &&
+      newResult &&
       snapshot.networkStatus === newResult.networkStatus &&
       snapshot.stale === newResult.stale &&
       isEqual(snapshot.data, newResult.data)
@@ -356,7 +357,9 @@ export class ObservableQuery<
   // XXX the subscription variables are separate from the query variables.
   // if you want to update subscription variables, right now you have to do that separately,
   // and you can only do it by stopping the subscription and then subscribing again with new variables.
-  public subscribeToMore<TSubscriptionData = TData>(options: SubscribeToMoreOptions<TData, TVariables, TSubscriptionData>) {
+  public subscribeToMore<TSubscriptionData = TData>(
+    options: SubscribeToMoreOptions<TData, TVariables, TSubscriptionData>,
+  ) {
     const subscription = this.queryManager
       .startGraphQLSubscription({
         query: options.document,
@@ -366,13 +369,14 @@ export class ObservableQuery<
         next: (subscriptionData: { data: TSubscriptionData }) => {
           if (options.updateQuery) {
             this.updateQuery((previous, { variables }) =>
-              (options.updateQuery as UpdateQueryFn<TData, TVariables, TSubscriptionData>)(
-                previous,
-                {
-                  subscriptionData,
-                  variables,
-                },
-              ),
+              (options.updateQuery as UpdateQueryFn<
+                TData,
+                TVariables,
+                TSubscriptionData
+              >)(previous, {
+                subscriptionData,
+                variables,
+              }),
             );
           }
         },

packages/apollo-client/src/core/types.ts

@@ -18,7 +18,7 @@ export type PureQueryOptions = {
 
 export type ApolloQueryResult<T> = {
   data: T;
-  errors?: GraphQLError[];
+  errors?: ReadonlyArray<GraphQLError>;
   loading: boolean;
   networkStatus: NetworkStatus;
   stale: boolean;

packages/apollo-client/src/data/queries.ts

@@ -10,7 +10,7 @@ export type QueryStoreValue = {
   previousVariables?: Object | null;
   networkStatus: NetworkStatus;
   networkError?: Error | null;
-  graphQLErrors?: GraphQLError[];
+  graphQLErrors?: ReadonlyArray<GraphQLError>;
   metadata: any;
 };
 
@@ -78,7 +78,7 @@ export class QueryStore {
       networkStatus = NetworkStatus.loading;
     }
 
-    let graphQLErrors: GraphQLError[] = [];
+    let graphQLErrors: ReadonlyArray<GraphQLError> = [];
     if (previousQuery && previousQuery.graphQLErrors) {
       graphQLErrors = previousQuery.graphQLErrors;
     }

packages/apollo-client/src/errors/ApolloError.ts

@@ -31,7 +31,7 @@ const generateErrorMessage = (err: ApolloError) => {
 
 export class ApolloError extends Error {
   public message: string;
-  public graphQLErrors: GraphQLError[];
+  public graphQLErrors: ReadonlyArray<GraphQLError>;
   public networkError: Error | null;
 
   // An object that can be used to provide some additional information
@@ -48,7 +48,7 @@ export class ApolloError extends Error {
     errorMessage,
     extraInfo,
   }: {
-    graphQLErrors?: GraphQLError[];
+    graphQLErrors?: ReadonlyArray<GraphQLError>;
     networkError?: Error | null;
     errorMessage?: string;
     extraInfo?: any;

packages/apollo-utilities/src/__tests__/transform.ts

@@ -534,7 +534,7 @@ describe('query transforms', () => {
     `;
     const expectedQueryStr = print(expectedQuery);
 
-    expect(expectedQueryStr).toBe(print(newQueryDoc));
+    expect(print(newQueryDoc)).toBe(expectedQueryStr);
   });
 
   it('should not add duplicates', () => {
@@ -565,7 +565,7 @@ describe('query transforms', () => {
     `;
     const expectedQueryStr = print(expectedQuery);
 
-    expect(expectedQueryStr).toBe(print(newQueryDoc));
+    expect(print(newQueryDoc)).toBe(expectedQueryStr);
   });
 
   it('should not screw up on a FragmentSpread within the query AST', () => {
@@ -1167,8 +1167,6 @@ describe('getDirectivesFromDocument', () => {
       const expected = gql`
         fragment client on ClientData {
           hi @client
-          bye @storage
-          bar
         }
 
         query Mixed {