processing/p5.js

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

Open

#7.854 geöffnet am 31. Mai 2025

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (20.784 Stars) (3.178 Forks)batch import
Help Wantedp5.js 2.0+

Beschreibung

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
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptNode.jsp5.js testingCSV format
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
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.