Skip to content

test(csv): parseCsvBatch unit tests (FlowwStar/FlowStar#340)#390

Open
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:test/issue-340-parse-csv-tests
Open

test(csv): parseCsvBatch unit tests (FlowwStar/FlowStar#340)#390
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:test/issue-340-parse-csv-tests

Conversation

@jdjioe5-cpu

Copy link
Copy Markdown

Closes #340. Adds 5 vitest cases (plain row, quoted commas, escaped quotes, empty trailing cell, optional duration shape) via the public parseCsvBatch entry point. lib/tests/csv-parser.test.ts.

…issue FlowwStar#340)

Adds lib/__tests__/csv-parser.test.ts with 5 vitest tests covering the
parseCsvLine branches via the public parseCsvBatch entry point:

- plain comma-separated row
- commas inside quoted fields
- embedded escaped double-quote pairs ("" -> ")
- empty trailing cells
- optional duration row shape with quoted commas

Closes FlowwStar#340.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

good first issue: add unit tests for parseCsvLine() in lib/csv-parser.ts

1 participant