Additional tests

Author: mrodrig

test/config/testCsvFilesList.js

@@ -41,7 +41,9 @@ const fs = require('fs'),
         {key: 'emptyLastFieldValueNoEol', file: '../data/csv/emptyLastFieldValueNoEol.csv'},
         {key: 'lastCharFieldDelimiter', file: '../data/csv/lastCharFieldDelimiter.csv'},
         {key: 'nativeMapMethod', file: '../data/csv/nativeMapMethod.csv'},
-        {key: 'nestedDotKeys', file: '../data/csv/nestedDotKeys.csv'}
+        {key: 'nestedDotKeys', file: '../data/csv/nestedDotKeys.csv'},
+        {key: 'nestedDotKeysWithArray', file: '../data/csv/nestedDotKeysWithArray.csv'},
+        {key: 'nestedDotKeysWithArrayExpandedUnwound', file: '../data/csv/nestedDotKeysWithArrayExpandedUnwound.csv'}
     ];
 
 function readCsvFile(filePath) {

test/config/testJsonFilesList.js

@@ -31,5 +31,7 @@ module.exports = {
     emptyLastFieldValue: require('../data/json/emptyLastFieldValue.json'),
     emptyLastFieldValueNoEol: require('../data/json/emptyLastFieldValueNoEol.json'),
     nativeMapMethod: require('../data/json/nativeMapMethod.json'),
-    nestedDotKeys: require('../data/json/nestedDotKeys.json')
+    nestedDotKeys: require('../data/json/nestedDotKeys.json'),
+    nestedDotKeysWithArray: require('../data/json/nestedDotKeysWithArray.json'),
+    nestedDotKeysWithArrayExpandedUnwound: require('../data/json/nestedDotKeysWithArrayExpandedUnwound.json')
 };

test/csv2json.js

@@ -207,6 +207,24 @@ function runTests(jsonTestData, csvTestData) {
                     done();
                 });
             });
+
+            // Test case for #184
+            it('should properly convert keys with escaped/nested dots in them even when they appear in an array', (done) => {
+                converter.csv2json(csvTestData.nestedDotKeysWithArray, (err, json) => {
+                    if (err) done(err);
+                    json.should.deepEqual(jsonTestData.nestedDotKeysWithArray);
+                    done();
+                });
+            });
+
+            // Test case for #184
+            it('should properly convert keys with escaped/nested dots in them even when they appear in an expanded, unwound array', (done) => {
+                converter.csv2json(csvTestData.nestedDotKeysWithArrayExpandedUnwound, (err, json) => {
+                    if (err) done(err);
+                    json.should.deepEqual(jsonTestData.nestedDotKeysWithArrayExpandedUnwound);
+                    done();
+                });
+            });
         });
 
         describe('Error Handling', () => {

test/data/csv/nestedDotKeysWithArray.csv

@@ -0,0 +1,2 @@
+a\.a,a\.b.c\.d,a\.b.c,a\.b.c\.f,a.a,a.b,a.b\.c\.d,a.f.g\.h,a.f.g\.h\.k,a.f.h.l\.m\.n\.o\.p,c
+a,4,5,6,1,2,32,3,5,qrstuv,"[{""d.e"":103,""f"":2},{""d.e"":105,""f"":5}]"

test/data/csv/nestedDotKeysWithArrayExpandedUnwound.csv

@@ -0,0 +1,3 @@
+a\.a,a\.b.c\.d,a\.b.c,a\.b.c\.f,a.a,a.b,a.b\.c\.d,a.f.g\.h,a.f.g\.h\.k,a.f.h.l\.m\.n\.o\.p,c.d\.e,c.f
+a,4,5,6,1,2,32,3,5,qrstuv,103,2
+a,4,5,6,1,2,32,3,5,qrstuv,105,5

test/data/json/nestedDotKeysWithArray.json

@@ -0,0 +1,26 @@
+[
+    {
+        "a.a": "a",
+        "a.b": {
+            "c.d": 4,
+            "c": 5,
+            "c.f": 6
+        },
+        "a": {
+            "a": 1,
+            "b": 2,
+            "b.c.d": 32,
+            "f": {
+                "g.h": 3,
+                "g.h.k": 5,
+                "h": {
+                    "l.m.n.o.p": "qrstuv"
+                }
+            }
+        },
+        "c": [
+            { "d.e" : 103, "f": 2 },
+            { "d.e" : 105, "f": 5 }
+        ]
+    }
+]

test/data/json/nestedDotKeysWithArrayExpandedUnwound.json

@@ -0,0 +1,44 @@
+[
+    {
+        "a.a": "a",
+        "a.b": {
+            "c.d": 4,
+            "c": 5,
+            "c.f": 6
+        },
+        "a": {
+            "a": 1,
+            "b": 2,
+            "b.c.d": 32,
+            "f": {
+                "g.h": 3,
+                "g.h.k": 5,
+                "h": {
+                    "l.m.n.o.p": "qrstuv"
+                }
+            }
+        },
+        "c": { "d.e" : 103, "f": 2 }
+    },
+    {
+        "a.a": "a",
+        "a.b": {
+            "c.d": 4,
+            "c": 5,
+            "c.f": 6
+        },
+        "a": {
+            "a": 1,
+            "b": 2,
+            "b.c.d": 32,
+            "f": {
+                "g.h": 3,
+                "g.h.k": 5,
+                "h": {
+                    "l.m.n.o.p": "qrstuv"
+                }
+            }
+        },
+        "c": { "d.e" : 105, "f": 5 }
+    }
+]

test/json2csv.js

@@ -154,6 +154,15 @@ function runTests(jsonTestData, csvTestData) {
                     done();
                 });
             });
+
+            // Test case for #184
+            it('should properly handle keys with nested dots in them even when they appear in array fields', (done) => {
+                converter.json2csv(jsonTestData.nestedDotKeysWithArray, (err, csv) => {
+                    if (err) done(err);
+                    csv.should.equal(csvTestData.nestedDotKeysWithArray);
+                    done();
+                });
+            });
         });
 
         describe('Error Handling', () => {
@@ -646,6 +655,22 @@ function runTests(jsonTestData, csvTestData) {
                     wrapBooleans: true
                 });
             });
+
+            // Test case for #184
+            it('should handle keys with nested dots when expanding and unwinding arrays', (done) => {
+                converter.json2csv(jsonTestData.nestedDotKeysWithArrayExpandedUnwound, (err, csv) => {
+                    if (err) done(err);
+
+                    // Replace raw boolean values with quoted versions
+                    let expectedCsv = csvTestData.nestedDotKeysWithArrayExpandedUnwound;
+
+                    csv.should.equal(expectedCsv);
+                    done();
+                }, {
+                    expandArrayObjects: true,
+                    unwindArrays: true
+                });
+            });
         });
     });