ACM-36696 [Backport release-2.14] Preserve newlines in CSV export to match console rendering

frontend/src/resources/utils/utils.ts

@@ -53,9 +53,14 @@ export function exportObjectString(object: Record<string, string>) {
   return keyValueMap.toString()
 }
 
-export function returnCSVSafeString(exportValue: string | ReactNode) {
-  // extract newlines
-  return `"${typeof exportValue === 'string' ? exportValue.split('\n').join(' ').replace(/"/g, '""') : exportValue}"`
+export function returnCSVSafeString(exportValue: string | number | ReactNode) {
+  if (typeof exportValue === 'number') {
+    return `"${exportValue}"`
+  }
+  if (typeof exportValue !== 'string') {
+    return '"-"'
+  }
+  return `"${exportValue.replaceAll('"', '""')}"`
 }
 
 export const getISOStringTimestamp = (timestamp: string) => {

frontend/src/routes/Governance/common/util.test.tsx

@@ -1,10 +1,11 @@
 /* Copyright Contributors to the Open Cluster Management project */
 'use strict'
 
-import { hasInformOnlyPolicies, getPolicyRemediation, resolveExternalStatus, parseStringMap } from './util'
+import { hasInformOnlyPolicies, getPolicyRemediation, resolveExternalStatus, parseStringMap, preserveWhitespace, returnCSVSafeString } from './util'
 import { PolicyTableItem } from '../policies/Policies'
 import { Policy, PolicyTemplate, REMEDIATION_ACTION } from '../../../resources'
 import { cloneDeep } from 'lodash'
+import { render } from '@testing-library/react'
 
 describe('Test resolveExternalStatus', () => {
   const mockPolicyWithManagers = (managers: string[]): Policy => {
@@ -901,3 +902,24 @@ describe('Test parseStringValue', () => {
     )
   })
 })
+
+describe('ACM-32500: UI description text should match CSV export for multiline descriptions', () => {
+  const descriptionFromApi =
+    'Policy is placed on hub or managed clusters with label acm-virt-config=acm-dr-virt-config-file-name.\nCreates a velero Schedule for all virtualmachines.kubevirt.io resources with a cluster.open-cluster-management.io/backup-vm label.'
+
+  test('preserveWhitespace textContent should equal CSV export value', () => {
+    const formatted = preserveWhitespace(descriptionFromApi)
+    const { container } = render(<>{formatted}</>)
+    const uiText = container.textContent
+
+    const csvRaw = returnCSVSafeString(descriptionFromApi)
+    const csvText = csvRaw.slice(1, -1).replace(/""/g, '"')
+
+    expect(uiText).toEqual(csvText)
+  })
+
+  test('preserveWhitespace returns undefined for empty input', () => {
+    expect(preserveWhitespace(undefined)).toBeUndefined()
+    expect(preserveWhitespace('')).toBeUndefined()
+  })
+})

frontend/src/ui-components/AcmTable/AcmTable.test.tsx

@@ -1150,10 +1150,10 @@ describe('AcmTable', () => {
 })
 
 describe('returnCSVSafeString', () => {
-  test('returns a csv safe string replacing new line with space', () => {
+  test('returns a csv safe string preserving newlines', () => {
     const content = 'testing for multiline\ndescription'
     const exportContent = returnCSVSafeString(content)
-    expect(exportContent).toEqual('"testing for multiline description"')
+    expect(exportContent).toEqual('"testing for multiline\ndescription"')
   })
 
   test('returns a csv safe string escaping double quotes', () => {