Fix issue where an empty array returned from fetchMore would rerender with an empty list

[jerelmiller]

Fixes #11642

When using fetchMore with any suspense hook, an empty array was returned from the query whenever the server returned an empty array. This was because we were attaching the raw server value on the promise and emitting that value. This PR updates the resolved value to call getCurrentResult which contains the full cached result.

[changeset-bot]

🦋 Changeset detected

Latest commit: e43dbd9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@apollo/client	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

size-limit report 📦

Path	Size
dist/apollo-client.min.cjs	38.35 KB (+0.02% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs"	46.17 KB (+0.01% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs" (production)	43.73 KB (+0.01% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js"	33.95 KB (0%)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js" (production)	31.87 KB (0%)
import { ApolloProvider } from "dist/react/index.js"	1.23 KB (0%)
import { ApolloProvider } from "dist/react/index.js" (production)	1.22 KB (0%)
import { useQuery } from "dist/react/index.js"	5.26 KB (0%)
import { useQuery } from "dist/react/index.js" (production)	4.35 KB (0%)
import { useLazyQuery } from "dist/react/index.js"	5.5 KB (0%)
import { useLazyQuery } from "dist/react/index.js" (production)	4.58 KB (0%)
import { useMutation } from "dist/react/index.js"	3.51 KB (0%)
import { useMutation } from "dist/react/index.js" (production)	2.73 KB (0%)
import { useSubscription } from "dist/react/index.js"	3.19 KB (0%)
import { useSubscription } from "dist/react/index.js" (production)	2.38 KB (0%)
import { useSuspenseQuery } from "dist/react/index.js"	5.37 KB (+0.08% 🔺)
import { useSuspenseQuery } from "dist/react/index.js" (production)	4.03 KB (+0.15% 🔺)
import { useBackgroundQuery } from "dist/react/index.js"	4.83 KB (+0.09% 🔺)
import { useBackgroundQuery } from "dist/react/index.js" (production)	3.49 KB (+0.09% 🔺)
import { useLoadableQuery } from "dist/react/index.js"	4.98 KB (+0.06% 🔺)
import { useLoadableQuery } from "dist/react/index.js" (production)	3.63 KB (+0.11% 🔺)
import { useReadQuery } from "dist/react/index.js"	3.12 KB (+0.16% 🔺)
import { useReadQuery } from "dist/react/index.js" (production)	3.06 KB (+0.13% 🔺)
import { useFragment } from "dist/react/index.js"	2.27 KB (0%)
import { useFragment } from "dist/react/index.js" (production)	2.22 KB (0%)

[jerelmiller]

Turns out these type policies were needed all along in some of these tests. The bug that this PR fixes masked this issue which is why these wasn't originally part of these tests.

[jerelmiller]

/release:pr

[github-actions]

A new release has been made for this PR. You can install it with:

npm i @apollo/[email protected]

[netlify]

✅ Deploy Preview for apollo-client-docs ready!

Name	Link
🔨 Latest commit	00a0508
🔍 Latest deploy log	https://app.netlify.com/sites/apollo-client-docs/deploys/65fa01ff01ca410008b9fb49
😎 Deploy Preview	https://deploy-preview-11706--apollo-client-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[alessbell]

The type policies in uBQ tests! Makes a lot of sense now :) LGTM 🚀