chore(*): add tests

src/utils/convertToCsv.spec.ts

@@ -40,4 +40,36 @@ describe('convertToCsv', () => {
 
     expect(convertToCSV(data)).toEqual(expectedOutput)
   })
+
+  it('should convert values with mixed quotes', () => {
+    const data = [{ foo: `',''`, bar: `","` }]
+
+    const expectedOutput = `foo:$4. bar:$3.\r\n"',''",""","""`
+
+    expect(convertToCSV(data)).toEqual(expectedOutput)
+  })
+
+  it('should convert values with mixed quotes', () => {
+    const data = [{ foo: `','`, bar: `,"` }]
+
+    const expectedOutput = `foo:$3. bar:$2.\r\n"','",","""`
+
+    expect(convertToCSV(data)).toEqual(expectedOutput)
+  })
+
+  it('should convert values with mixed quotes', () => {
+    const data = [{ foo: `"`, bar: `'` }]
+
+    const expectedOutput = `foo:$1. bar:$1.\r\n"""","'"`
+
+    expect(convertToCSV(data)).toEqual(expectedOutput)
+  })
+
+  it('should convert values with mixed quotes', () => {
+    const data = [{ foo: `,`, bar: `',` }]
+
+    const expectedOutput = `foo:$1. bar:$2.\r\n",","',"`
+
+    expect(convertToCSV(data)).toEqual(expectedOutput)
+  })
 })

src/utils/convertToCsv.ts

@@ -69,35 +69,20 @@ export const convertToCSV = (data: any) => {
   csvTest = data.map((row: any) => {
     const fields = Object.keys(row).map((fieldName, index) => {
       let value
-      let containsSpecialChar = false
       const currentCell = row[fieldName]
 
-      if (JSON.stringify(currentCell).search(/(\\t|\\n|\\r)/gm) > -1) {
+      // stringify with replacer converts null values to empty strings
-        containsSpecialChar = true
+      // also wraps the value in double quotes
-      }
+      value = JSON.stringify(currentCell, replacer).replace(/\\\"/g, `""`)
-
-      value = JSON.stringify(currentCell, replacer)
 
       value = value.replace(/\\\\/gm, '\\')
 
-      if (containsSpecialChar) {
+      if (
-        if (value.includes(',') || value.includes('"')) {
+        value.substring(1, value.length - 1).search(/(\\t|\\n|\\r|,|\'|\")/gm) <
-          value = '"' + value + '"'
+        0
-        }
+      ) {
-      } else {
+        // Remove wrapping quotes for values that don't contain special characters
-        if (
+        value = value.substring(1, value.length - 1)
-          !value.includes(',') &&
-          value.includes('"') &&
-          !value.includes('\\"')
-        ) {
-          value = value.substring(1, value.length - 1)
-        }
-
-        if (value.includes("'")) {
-          value = '"' + value + '"'
-        }
-
-        value = value.replace(/\\"/gm, '""')
       }
 
       value = value.replace(/\r\n/gm, '\n')