fix(archives): adjust queries and response handling for updated data …

…schema (#1221) * fix(archives): adjust queries and response handling for updated data schema
cryostatio · Apr 13, 2024 · 9c1a626 · 9c1a626
1 parent 4acf4a3
commit 9c1a626
Show file tree

Hide file tree

Showing 4 changed files with 55 additions and 45 deletions.
diff --git a/src/app/Archives/AllTargetsArchivedRecordingsTable.tsx b/src/app/Archives/AllTargetsArchivedRecordingsTable.tsx
@@ -112,7 +112,7 @@ export const AllTargetsArchivedRecordingsTable: React.FC<AllTargetsArchivedRecor
       setArchivesForTargets(
         targetNodes.map((node) => {
           const target: Target = {
-            connectUrl: node.target.serviceUri,
+            connectUrl: node.target.connectUrl,
             alias: node.target.alias,
             labels: [],
             annotations: {
@@ -123,7 +123,7 @@ export const AllTargetsArchivedRecordingsTable: React.FC<AllTargetsArchivedRecor
           return {
             target,
             targetAsObs: of(target),
-            archiveCount: node.recordings.archived.aggregate.count,
+            archiveCount: node.target.recordings.archived.aggregate.count,
           };
         }),
       );
@@ -148,13 +148,13 @@ export const AllTargetsArchivedRecordingsTable: React.FC<AllTargetsArchivedRecor
           `query AllTargetsArchives {
                targetNodes {
                  target {
-                   serviceUri
+                   connectUrl
                    alias
-                 }
-                 recordings {
-                   archived {
-                     aggregate {
-                       count
+                   recordings {
+                     archived {
+                       aggregate {
+                         count
+                       }
                      }
                    }
                  }
@@ -178,10 +178,12 @@ export const AllTargetsArchivedRecordingsTable: React.FC<AllTargetsArchivedRecor
             `
         query ArchiveCountForTarget($connectUrl: String) {
           targetNodes(filter: { name: $connectUrl }) {
-            recordings {
-              archived {
-                aggregate {
-                  count
+            target {
+              recordings {
+                archived {
+                  aggregate {
+                    count
+                  }
                 }
               }
             }
@@ -196,7 +198,7 @@ export const AllTargetsArchivedRecordingsTable: React.FC<AllTargetsArchivedRecor
                 {
                   target: target,
                   targetAsObs: of(target),
-                  archiveCount: v.data.targetNodes[0].recordings.archived.aggregate.count,
+                  archiveCount: v.data.targetNodes[0].target.recordings.archived.aggregate.count,
                 },
               ];
             });

diff --git a/src/app/RecordingMetadata/BulkEditLabels.tsx b/src/app/RecordingMetadata/BulkEditLabels.tsx
@@ -164,7 +164,7 @@ export const BulkEditLabels: React.FC<BulkEditLabelsProps> = ({
       observable = isUploadsTable
         ? context.api
             .graphql<any>(
-              `query GetUploadedRecordings($filter: ArchivedRecordingFilterInput) {
+              `query GetUploadedRecordings($filter: ArchivedRecordingsFilterInput) {
                 archivedRecordings(filter: $filter) {
                   data {
                     name

diff --git a/src/app/Recordings/ArchivedRecordingsTable.tsx b/src/app/Recordings/ArchivedRecordingsTable.tsx
@@ -198,7 +198,10 @@ export const ArchivedRecordingsTable: React.FC<ArchivedRecordingsTableProps> = (
             downloadUrl
             reportUrl
             metadata {
-              labels
+              labels {
+                key
+                value
+              }
             }
             size
           }
@@ -213,14 +216,17 @@ export const ArchivedRecordingsTable: React.FC<ArchivedRecordingsTableProps> = (
   const queryUploadedRecordings = React.useCallback(() => {
     /* eslint-disable-next-line @typescript-eslint/no-explicit-any */
     return context.api.graphql<any>(
-      `query UploadedRecordings($filter: ArchivedRecordingFilterInput){
+      `query UploadedRecordings($filter: ArchivedRecordingsFilterInput) {
         archivedRecordings(filter: $filter) {
           data {
             name
             downloadUrl
             reportUrl
             metadata {
-              labels
+              labels {
+                key
+                value
+              }
             }
             size
           }

diff --git a/src/test/Archives/AllTargetsArchivedRecordingsTable.test.tsx b/src/test/Archives/AllTargetsArchivedRecordingsTable.test.tsx
@@ -76,42 +76,42 @@ const mockTargetsAndCountsResponse = {
   data: {
     targetNodes: [
       {
-        recordings: {
-          archived: {
-            aggregate: {
-              count: mockCount1,
-            },
-          },
-        },
         target: {
           alias: mockAlias1,
-          serviceUri: mockConnectUrl1,
-        },
-      },
-      {
-        recordings: {
-          archived: {
-            aggregate: {
-              count: mockCount2,
+          connectUrl: mockConnectUrl1,
+          recordings: {
+            archived: {
+              aggregate: {
+                count: mockCount1,
+              },
             },
           },
         },
-        target: {
-          alias: mockAlias2,
-          serviceUri: mockConnectUrl2,
-        },
       },
       {
-        recordings: {
-          archived: {
-            aggregate: {
-              count: mockCount3,
+        target: {
+          alias: mockAlias2,
+          connectUrl: mockConnectUrl2,
+          recordings: {
+            archived: {
+              aggregate: {
+                count: mockCount2,
+              },
             },
           },
         },
+      },
+      {
         target: {
           alias: mockAlias3,
-          serviceUri: mockConnectUrl3,
+          connectUrl: mockConnectUrl3,
+          recordings: {
+            archived: {
+              aggregate: {
+                count: mockCount3,
+              },
+            },
+          },
         },
       },
     ],
@@ -122,10 +122,12 @@ const mockNewTargetCountResponse = {
   data: {
     targetNodes: [
       {
-        recordings: {
-          archived: {
-            aggregate: {
-              count: mockNewCount,
+        target: {
+          recordings: {
+            archived: {
+              aggregate: {
+                count: mockNewCount,
+              },
             },
           },
         },