react: MutationResult.client is always set

As far as I can tell, whenever a MutationResult is provided to the user of
useMutation (or the component or HOC), it always contains a `client`; eg,
OperationData.refreshClient checks this invariant.

MutationResult is also used to type an internal state variable used by
MutationData and useMutation; this one doesn't contain a `client`. This PR
changes that variable to different type without `client`, and the actual exposed
MutationResult now has a non-optional `client`. This makes it match
`QueryResult`.

Now TypeScript consumers of `useMutation` don't need to check to see if `client`
exists before using it.

src/react/data/MutationData.ts

@@ -6,20 +6,22 @@ import {
   MutationDataOptions,
   MutationTuple,
   MutationFunctionOptions,
-  MutationResult
+  MutationResult,
 } from '../types/types';
 import { OperationData } from './OperationData';
 import { OperationVariables } from '../../core/types';
 import { FetchResult } from '../../link/core/types';
 
+type MutationResultWithoutClient<TData = any> = Omit<MutationResult<TData>, 'client'>;
+
 export class MutationData<
   TData = any,
   TVariables = OperationVariables
 > extends OperationData {
   private mostRecentMutationId: number;
-  private result: MutationResult<TData>;
-  private previousResult?: MutationResult<TData>;
-  private setResult: (result: MutationResult<TData>) => any;
+  private result: MutationResultWithoutClient<TData>;
+  private previousResult?: MutationResultWithoutClient<TData>;
+  private setResult: (result: MutationResultWithoutClient<TData>) => any;
 
   constructor({
     options,
@@ -29,8 +31,8 @@ export class MutationData<
   }: {
     options: MutationDataOptions<TData, TVariables>;
     context: any;
-    result: MutationResult<TData>;
-    setResult: (result: MutationResult<TData>) => any;
+    result: MutationResultWithoutClient<TData>;
+    setResult: (result: MutationResultWithoutClient<TData>) => any;
   }) {
     super(options, context);
     this.verifyDocumentType(options.mutation, DocumentType.Mutation);
@@ -39,7 +41,7 @@ export class MutationData<
     this.mostRecentMutationId = 0;
   }
 
-  public execute(result: MutationResult<TData>) {
+  public execute(result: MutationResultWithoutClient<TData>): MutationTuple<TData, TVariables> {
     this.isMounted = true;
     this.verifyDocumentType(this.getOptions().mutation, DocumentType.Mutation);
     return [
@@ -148,7 +150,7 @@ export class MutationData<
     return this.mostRecentMutationId === mutationId;
   }
 
-  private updateResult(result: MutationResult<TData>) {
+  private updateResult(result: MutationResultWithoutClient<TData>) {
     if (
       this.isMounted &&
       (!this.previousResult || !equal(this.previousResult, result))

src/react/types/types.ts

@@ -189,8 +189,8 @@ export interface MutationResult<TData = any> {
   error?: ApolloError;
   loading: boolean;
   called: boolean;
-  client?: ApolloClient<object>;
-}
+  client: ApolloClient<object>;
+};
 
 export declare type MutationFunction<
   TData = any,