CSVs aren't being parsed correctly as per the documentation

• Operating System: OS X El Capitan 10.11.6
• Node Version: v11.0.0
• NPM Version: 6.4.1
• csv-parser Version: 2.1.0

Expected Behavior

Following the example in the README.md:

Suppose you have a CSV file data.csv which contains the data:

NAME, AGE
Daffy Duck, 24
Bugs Bunny, 22

It could then be parsed, and results shown like so:

const csv = require('csv-parser')
const fs = require('fs')
const results = [];

fs.createReadStream('data.csv')
  .pipe(csv())
  .on('data', results.push)
  .on('end', () => {
    console.log(results);
    // [
    //   { NAME: 'Daffy Duck', AGE: 24 },
    //   { NAME: 'Bugs Bunny', AGE: 22 }
    // ]
  });

Actual Behavior

I created a file data.csv in the same directory as the node file I’m running.

My code is copied and pasted from above:

const csv = require('csv-parser')
const fs = require('fs')
const results = [];

fs.createReadStream('data.csv')
  .pipe(csv())
  .on('data', results.push)
  .on('end', () => {
    console.log(results);
    // [
    //   { NAME: 'Daffy Duck', AGE: 24 },
    //   { NAME: 'Bugs Bunny', AGE: 22 }
    // ]
  });

However the result to console.log is: []

I know I’m accessing the correct thing because I then modified the code to this:

const csv = require('csv-parser')
const fs = require('fs')
const results = [];

fs.createReadStream('data.csv')
  .pipe(csv())
  .on('data', (results) => {
    console.log(results);
  })
  .on('end', () => {
    console.log(results);
    // [
    //   { NAME: 'Daffy Duck', AGE: 24 },
    //   { NAME: 'Bugs Bunny', AGE: 22 }
    // ]
  });

Output to console.log:

Row { NAME: 'Daffy Duck', ' AGE': ' 24' }
Row { NAME: 'Bugs Bunny', ' AGE': ' 22' }
[]

The other header is getting turned into a string value and it’s not being picked up. The empty array still persists

How Do We Reproduce?

Run the example

[shellscape]

@bjdooi thanks for stopping in. Change your data handler to:

  .on('data', (d) => {
    results.push(d);
  })

Probably a context issue passing results.push as the handler fn. Feel free to submit a PR to the README to update this if you’re up for it.

[DanielAsher]

This really should be sorted out as I wasted a good amount of time using the example code on npmjs.