APPEALS-53603: Modify logic to always show recipient information if p…

…resent (#22414)

* initial commit

* making rspec tests happy

* linting

* adding tests back and refactoring

* small char error

---------

Co-authored-by: nhansen3 <noah.hansen1323@gmail.com>

client/app/queue/components/NotificationTableColumns.jsx

@@ -81,7 +81,7 @@ export const recipientInformationColumn = (notifications) => {
     tableData: notifications,
     valueName: 'Recipient Information',
     // eslint-disable-next-line no-negated-condition
-    valueFunction: (notification) => notification.status !== 'delivered' ? '—' : notification.recipient_information
+    valueFunction: (notification) => notification.recipient_information ?? '—'
   };
 };
 

client/test/data/notifications.js

@@ -118,9 +118,9 @@ export const notifications = [
       notification_type: 'Email and SMS',
       event_date: '2022-10-27',
       event_type: 'Appeal decision mailed (Non-contested claims)',
-      recipient_email: 'test@caseflow.com',
+      recipient_email: null,
       recipient_phone_number: '2468012345',
-      email_notification_status: 'sent',
+      email_notification_status: 'temporary-failure',
       sms_notification_status: 'delivered',
       notification_content: 'string'
     }