fix: Remove extra space character in mode withBOM: true #190 (#194)

zemirco · Aug 15, 2017 · e8b6f6b · e8b6f6b
1 parent d7ad29f
commit e8b6f6b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/json2csv.js b/lib/json2csv.js
@@ -303,7 +303,7 @@ function createColumnContent(params, str) {
   });
   // Add BOM character if required
   if (params.withBOM) {
-    str = '\ufeff ' + str;
+    str = '\ufeff' + str;
   }
 
   return str;

diff --git a/test/fixtures/csv/withBOM.csv b/test/fixtures/csv/withBOM.csv
@@ -2,4 +2,4 @@
 "Audi",0,"blue",
 "BMW",15000,"red","manual"
 "Mercedes",20000,"yellow",
-"Citroën",30000,"green",
+"Citroën",30000,"green",
diff --git a/test/index.js b/test/index.js
@@ -722,8 +722,8 @@ async.parallel(loadFixtures(csvFixtures), function (err) {
       data: jsonSpecialCharacters,
       withBOM: true
     });
-    // Compare csv length to check if the BOM character and the space after are present
-    t.equal(csv.length, csvFixtures.default.length + 2);
+    // Compare csv length to check if the BOM character is present
+    t.equal(csv.length, csvFixtures.default.length + 1);
     t.equal(csv.length, csvFixtures.withBOM.length);
     t.end();
   });