csvtojson - npm Package Compare versions

Comparing version 0.3.12 to 0.3.13

libs/core/csvConverter.js

@@ -31,4 +31,5 @@ module.exports = csvAdv;
   this.resultObject = new Result();
-  if (this.param.constructResult) {
-    this.pipe(this.resultObject);
+  this.pipe(this.resultObject);
+  if (!this.param.constructResult) {
+    this.resultObject.disableConstruct();
   }
@@ -35,0 +36,0 @@ this.headRow = [];

libs/core/Result.js

@@ -21,1 +21,7 @@ module.exports=Result;
 }
+
+Result.prototype.disableConstruct=function(){
+  this._write=function(d,e,cb){
+    cb();//do nothing just dropit
+  }
+}

package.json

@@ -15,3 +15,3 @@ {
   }],
-  "version": "0.3.12",
+  "version": "0.3.13",
   "keywords": [
@@ -18,0 +18,0 @@ "csv",

test/testCSVConverter.js

@@ -169,2 +169,18 @@ var CSVAdv = require("../libs/core/csvConverter.js");
   });
+  it ("shoudl parse large csv file",function(done){
+    var testData=__dirname+"/data/large-csv-sample.csv";
+    var rs=fs.createReadStream(testData);
+    var csvConverter=new CSVAdv({
+      constructResult:false
+    });
+    var count=0;
+    csvConverter.on("record_parsed",function(d){
+      count++;
+    });
+    csvConverter.on("end_parsed",function(){
+      assert(count===5290);
+      done();
+    });
+    rs.pipe(csvConverter);
+  });
 });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

131673

increased by151.24%

Number of package files

32

increased by3.23%

Lines of code

887

increased by2.54%

No dependency changes