Canonicalize empty result objects, too.

Small addition to PR #7439.
apollographql · Jan 15, 2021 · 419e90a · 419e90a
1 parent 2cb0944
commit 419e90a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/cache/inmemory/object-canon.ts b/src/cache/inmemory/object-canon.ts
@@ -172,6 +172,9 @@ export class ObjectCanon {
   // Arrays that contain the same elements in a different order can share
   // the same SortedKeysInfo object, to save memory.
   private keysByJSON = new Map<string, SortedKeysInfo>();
+
+  // This has to come last because it depends on keysByJSON.
+  public readonly empty = this.admit({});
 }
 
 type SortedKeysInfo = {

diff --git a/src/cache/inmemory/readFromStore.ts b/src/cache/inmemory/readFromStore.ts
@@ -189,7 +189,7 @@ export class StoreReader {
         !context.policies.rootTypenamesById[objectOrReference.__ref] &&
         !context.store.has(objectOrReference.__ref)) {
       return {
-        result: {},
+        result: this.canon.empty,
         missing: [missingFromInvariant(
           new InvariantError(
             `Dangling reference to missing ${objectOrReference.__ref} object`