Fix a bug where the new operationType property wasn't passed into operation. (#12748)

Co-authored-by: Jerel Miller <[email protected]>
Co-authored-by: jerelmiller <[email protected]>

Author: phryneas

.changeset/fresh-swans-remain.md

@@ -0,0 +1,5 @@
+---
+"@apollo/client": patch
+---
+
+Fix a bug where the new `operationType` property wasn't passed into `operation`.

.size-limits.json

@@ -1,6 +1,6 @@
 {
-  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (CJS)": 43643,
-  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (production) (CJS)": 38635,
-  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\"": 33333,
-  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (production)": 27618
+  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (CJS)": 43723,
+  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (production) (CJS)": 38671,
+  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\"": 33349,
+  "import { ApolloClient, InMemoryCache, HttpLink } from \"@apollo/client\" (production)": 27607
 }

src/core/__tests__/ApolloClient/links.test.ts

@@ -1,4 +1,4 @@
-import type { FormattedExecutionResult } from "graphql";
+import { type FormattedExecutionResult, OperationTypeNode } from "graphql";
 import { gql } from "graphql-tag";
 import type { Observable, Subscription } from "rxjs";
 import { map, of } from "rxjs";
@@ -395,4 +395,50 @@ describe("Link interactions", () => {
 
     expect(result.current!.query).toMatchDocument(expectedQuery);
   });
+
+  it("passes all expected properties into links", async () => {
+    let operation!: Operation;
+    const link = new ApolloLink((op) => {
+      operation = op;
+      return of({ data: { hello: "world" } });
+    });
+    const client = new ApolloClient({
+      link,
+      cache: new InMemoryCache(),
+    });
+    const query = gql`
+      query HelloWorld {
+        hello
+      }
+    `;
+    await client
+      .query({
+        query,
+      })
+      .catch(() => {});
+    expect(operation).toStrictEqualTyped({
+      variables: {},
+      extensions: {},
+      operationName: "HelloWorld",
+      operationType: OperationTypeNode.QUERY,
+      query,
+    });
+    expect(Object.keys(operation)).toEqual([
+      "variables",
+      "extensions",
+      "operationName",
+      "operationType",
+      "query",
+    ]);
+    expect(Object.getOwnPropertyNames(operation)).toEqual([
+      "variables",
+      "extensions",
+      "operationName",
+      "operationType",
+      "query",
+      "setContext",
+      "getContext",
+      "client",
+    ]);
+  });
 });

src/link/utils/transformOperation.ts

@@ -6,6 +6,7 @@ export function transformOperation(operation: GraphQLRequest): GraphQLRequest {
     variables: operation.variables || {},
     extensions: operation.extensions || {},
     operationName: operation.operationName,
+    operationType: operation.operationType,
     query: operation.query,
   };