processing/p5.js

[p5.js 2.0 Bug Report]: IO loadTable test failed

Open

#7854 opened on May 31, 2025

View on GitHub
 (8 comments) (0 reactions) (0 assignees)JavaScript (20,784 stars) (3,178 forks)batch import
Help Wantedp5.js 2.0+

Description

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

No response

Web browser and version

No response

Operating system

Windows

Steps to reproduce this

Steps:

  1. run command npm test

Snippet:


CSV files should handle escaped quotes and returns within quoted fields
AssertionError: expected 'David,\r\nSr. "the boss"' to equal 'David,\nSr. "the boss"'
 - /test/unit/io/loadTable.js:87:12
- Expected
+ Received

- David,
+ David,

  Sr. "the boss"

Contributor guide

Tech stack
javascriptnodejs
Domain
testingdata
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JavaScriptNode.jsp5.js testingCSV format
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75
Research direction
This bug involves a failing unit test in '/test/unit/io/loadTable.js' (line 87) where expected CSV parsing output differs on carriage returns. Investigate whether the error stems from the test expectation (which includes \r\n) or the CSV parser's handling of line endings. Check if the test runner or operating system introduces additional carriage returns. Review the CSV parsing logic in p5.js source, particularly in the loadTable function or related utilities. The fix may involve updating the test expectation to match cross platform behavior or adjusting the parser.