node-csvtojson icon indicating copy to clipboard operation
node-csvtojson copied to clipboard

Added rowIndex to colParser function

Open abramsimon opened this issue 5 years ago • 1 comments

Adds another parameter to the custom colParser function callback that exposes the row index being processed. Zero-based. Adds an assert to an existing test to provide coverage.

The unit test for this was not executing correctly, so no assertions would fail under any conditions. I refactored the test to use async/await instead of done() and the test started working as expected.

Issue #373

abramsimon avatar Oct 03 '20 01:10 abramsimon

Pull Request Test Coverage Report for Build 243

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.421%

Totals Coverage Status
Change from base Build 238: 0.0%
Covered Lines: 664
Relevant Lines: 680

💛 - Coveralls

coveralls avatar Oct 03 '20 01:10 coveralls