[p5.js 2.0 Bug Report]: IO loadTable test failed
Open
Nobody has claimed this yet.
Help Wanted
p5.js 2.0+
- Dominant language
- JavaScript
- Stars
- 24k
- Forks
- 3.8k
- Avg merge
- 3d 16h
- Merged PRs (30d)
- 25
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:
- 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with test/unit/io/loadTable.js, especially the failing assertion at line 87, and run npm test on Windows to reproduce the line-ending mismatch. Check the CSV case for escaped quotes and returns within quoted fields; done means the test's expected and received newline behavior agree.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100