Don't warn for caches that don't enable data masking

src/cache/core/__tests__/cache.ts

@@ -198,6 +198,33 @@ describe("abstract cache", () => {
       );
     });
 
+    it("returns original data and does not warn on caches that don't implement the required interface with the default data masking config", () => {
+      using consoleSpy = spyOnConsole("warn");
+
+      const query = gql`
+        query {
+          user {
+            __typename
+            id
+            ...UserFields
+          }
+        }
+
+        fragment UserFields on User {
+          name
+        }
+      `;
+      const data = {
+        user: { __typename: "User", id: 1, name: "Mister Masked" },
+      };
+      const cache = new TestCache();
+
+      const result = cache.maskOperation(query, data);
+
+      expect(result).toBe(data);
+      expect(consoleSpy.warn).not.toHaveBeenCalled();
+    });
+
     it("returns masked query for caches that implement required interface", () => {
       class MaskingCache extends TestCache {
         public readonly dataMasking = true;

src/cache/core/cache.ts

@@ -382,6 +382,10 @@ export abstract class ApolloCache<TSerialized> implements DataProxy {
   }
 
   public maskOperation<TData = unknown>(document: DocumentNode, data: TData) {
+    if (!this.dataMasking) {
+      return data;
+    }
+
     if (!this.fragmentMatches) {
       if (__DEV__) {
         invariant.warn(
@@ -392,10 +396,6 @@ export abstract class ApolloCache<TSerialized> implements DataProxy {
       return data;
     }
 
-    if (!this.dataMasking) {
-      return data;
-    }
-
     return maskOperation(data, document, this.fragmentMatches.bind(this));
   }