feat: Fetch isCached Field in PullPageDataQueryOpts (#3622)

Author: nsdeschenes

src/pages/PullRequestPage/PullBundleAnalysis/PullBundleAnalysis.test.tsx

@@ -48,6 +48,8 @@ const mockPullPageData = (
           bundleAnalysis: {
             bundleAnalysisReport: {
               __typename: headBundleType,
+              isCached:
+                headBundleType === 'BundleAnalysisReport' ? false : undefined,
             },
           },
         },

src/pages/PullRequestPage/PullCoverage/PullCoverageTabs/PullCoverageTabs.test.jsx

@@ -96,6 +96,7 @@ const mockPullData = {
           bundleAnalysis: {
             bundleAnalysisReport: {
               __typename: 'BundleAnalysisReport',
+              isCached: false,
             },
           },
         },

src/pages/PullRequestPage/PullCoverage/PullCoverageTabs/useTabsCounts.test.tsx

@@ -26,6 +26,7 @@ const mockPullData = {
           bundleAnalysis: {
             bundleAnalysisReport: {
               __typename: 'BundleAnalysisReport',
+              isCached: false,
             },
           },
         },

src/pages/PullRequestPage/PullCoverage/routes/FileViewer/FileViewer.test.jsx

@@ -102,6 +102,7 @@ const mockPullData = {
         bundleAnalysis: {
           bundleAnalysisReport: {
             __typename: 'BundleAnalysisReport',
+            isCached: false,
           },
         },
       },

src/pages/PullRequestPage/PullRequestPage.test.tsx

@@ -49,6 +49,7 @@ const mockPullPageData = {
     bundleAnalysis: {
       bundleAnalysisReport: {
         __typename: 'BundleAnalysisReport',
+        isCached: false,
       },
     },
   },
@@ -75,6 +76,7 @@ const mockPullPageDataTeam = {
     bundleAnalysis: {
       bundleAnalysisReport: {
         __typename: 'BundleAnalysisReport',
+        isCached: false,
       },
     },
   },

src/pages/PullRequestPage/queries/PullPageDataQueryOpts.test.tsx

@@ -25,6 +25,7 @@ const mockPullData = {
           bundleAnalysis: {
             bundleAnalysisReport: {
               __typename: 'BundleAnalysisReport',
+              isCached: false,
             },
           },
         },
@@ -60,6 +61,7 @@ const mockPullDataTeam = {
           bundleAnalysis: {
             bundleAnalysisReport: {
               __typename: 'BundleAnalysisReport',
+              isCached: false,
             },
           },
         },
@@ -195,6 +197,7 @@ describe('PullPageDataQueryOpts', () => {
                   bundleAnalysis: {
                     bundleAnalysisReport: {
                       __typename: 'BundleAnalysisReport',
+                      isCached: false,
                     },
                   },
                 },
@@ -402,6 +405,7 @@ describe('PullPageDataQueryOpts', () => {
                 bundleAnalysis: {
                   bundleAnalysisReport: {
                     __typename: 'BundleAnalysisReport',
+                    isCached: false,
                   },
                 },
               },

src/pages/PullRequestPage/queries/PullPageDataQueryOpts.tsx

@@ -17,6 +17,16 @@ import Api from 'shared/api'
 import { rejectNetworkError } from 'shared/api/helpers'
 import A from 'ui/A'
 
+const BundleAnalysisReportSchema = z.object({
+  __typename: z.literal('BundleAnalysisReport'),
+  isCached: z.boolean(),
+})
+
+const BundleAnalysisReportUnion = z.discriminatedUnion('__typename', [
+  BundleAnalysisReportSchema,
+  z.object({ __typename: MissingHeadReportSchema.shape.__typename }),
+])
+
 const BundleAnalysisComparisonResult = z.union([
   z.literal('BundleAnalysisComparison'),
   FirstPullRequestSchema.shape.__typename,
@@ -48,12 +58,7 @@ const RepositorySchema = z.object({
           commitid: z.string(),
           bundleAnalysis: z
             .object({
-              bundleAnalysisReport: z
-                .discriminatedUnion('__typename', [
-                  z.object({ __typename: z.literal('BundleAnalysisReport') }),
-                  z.object({ __typename: z.literal('MissingHeadReport') }),
-                ])
-                .nullable(),
+              bundleAnalysisReport: BundleAnalysisReportUnion.nullable(),
             })
             .nullable(),
         })
@@ -120,6 +125,12 @@ query PullPageData(
           head {
             commitid
             bundleAnalysis {
+              bundleAnalysisReport {
+                __typename
+                ... on BundleAnalysisReport {
+                  isCached
+                }
+              }
               bundleAnalysisReport {
                 __typename
               }
@@ -155,6 +166,7 @@ query PullPageData(
           }
           bundleAnalysisCompareWithBase {
             __typename
+            
           }
         }
       }