fix(archives): add ArchivedRecordingCreated notification hook to refr…

…esh All Archives view (#576) (#580)

* add ArchivedRecordingCreated notification hook to refresh allArchives

* fix tests

(cherry picked from commit e220ce9)

Co-authored-by: Max Cao <macao@redhat.com>

src/app/Archives/AllArchivedRecordingsTable.tsx

@@ -157,6 +157,14 @@ export const AllArchivedRecordingsTable: React.FunctionComponent<AllArchivedReco
     );
   }, [addSubscription, context.notificationChannel, refreshDirectoriesAndCounts]);
 
+  React.useEffect(() => {
+    addSubscription(
+      context.notificationChannel.messages(NotificationCategory.ArchivedRecordingCreated).subscribe((v) => {
+        refreshDirectoriesAndCounts();
+      })
+    );
+  }, [addSubscription, context.notificationChannel, refreshDirectoriesAndCounts]);
+
   React.useEffect(() => {
     addSubscription(
       context.notificationChannel.messages(NotificationCategory.ArchivedRecordingDeleted).subscribe((v) => {

src/test/Archives/AllArchivedRecordingsTable.test.tsx

@@ -141,25 +141,30 @@ jest
 
 jest
   .spyOn(defaultServices.notificationChannel, 'messages')
-  .mockReturnValueOnce(of()) // renders correctly
-  .mockReturnValueOnce(of())
-  .mockReturnValueOnce(of())
+  .mockReturnValueOnce(of()) // renders correctly  // NotificationCategory.RecordingMetadataUpdated
+  .mockReturnValueOnce(of()) // NotificationCategory.ActiveRecordingSaved
+  .mockReturnValueOnce(of()) // NotificationCategory.ArchivedRecordingCreated
+  .mockReturnValueOnce(of()) // NotificationCategory.ArchivedRecordingDeleted
 
   .mockReturnValueOnce(of()) // shows no recordings when empty
   .mockReturnValueOnce(of())
   .mockReturnValueOnce(of())
+  .mockReturnValueOnce(of())
 
   .mockReturnValueOnce(of()) // has the correct table elements
   .mockReturnValueOnce(of())
   .mockReturnValueOnce(of())
+  .mockReturnValueOnce(of())
 
   .mockReturnValueOnce(of()) // correctly handles the search function
   .mockReturnValueOnce(of())
   .mockReturnValueOnce(of())
+  .mockReturnValueOnce(of())
 
   .mockReturnValueOnce(of()) // expands targets to show their <ArchivedRecordingsTable />
   .mockReturnValueOnce(of())
   .mockReturnValueOnce(of())
+  .mockReturnValueOnce(of())
 
   .mockReturnValueOnce(of(mockRecordingSavedNotification)) // increments the count when an archived recording is saved
   .mockReturnValueOnce(of())