Avoid registering QueryPromise when skip is true during SSR (#7310)

CHANGELOG.md

@@ -64,6 +64,9 @@
 - Allow optional arguments in `keyArgs: [...]` arrays for `InMemoryCache` field policies. <br/>
   [@benjamn](https://github.com/benjamn) in [#7109](https://github.com/apollographql/apollo-client/pull/7109)
 
+- Avoid registering `QueryPromise` when `skip` is `true` during server-side rendering. <br/>
+  [@izumin5210](https://github.com/izumin5210) in [#7310](https://github.com/apollographql/apollo-client/pull/7310)
+
 ## Apollo Client 3.2.7
 
 ## Bug Fixes

src/react/data/QueryData.ts

@@ -152,7 +152,8 @@ export class QueryData<TData, TVariables> extends OperationData {
   };
 
   private getExecuteSsrResult() {
-    const ssrDisabled = this.getOptions().ssr === false;
+    const { ssr, skip } = this.getOptions();
+    const ssrDisabled = ssr === false || skip;
     const fetchDisabled = this.refreshClient().client.disableNetworkFetches;
 
     const ssrLoading = {

src/react/ssr/__tests__/useQuery.test.tsx

@@ -166,4 +166,29 @@ describe('useQuery Hook SSR', () => {
       });
     }
   );
+
+  it('should skip SSR tree rendering if `skip` option is `true`', async () => {
+    let renderCount = 0;
+    const Component = () => {
+      const { data, loading } = useQuery(CAR_QUERY, { skip: true });
+      renderCount += 1;
+
+      if (!loading) {
+        const { make } = data.cars[0];
+        return <div>{make}</div>;
+      }
+      return null;
+    };
+
+    const app = (
+      <MockedProvider mocks={CAR_MOCKS}>
+        <Component />
+      </MockedProvider>
+    );
+
+    return renderToStringWithData(app).then((result) => {
+      expect(renderCount).toBe(1);
+      expect(result).toEqual('');
+    });
+  });
 });