Skip to content

test: cover Unicode CSV escaping#997

Merged
Baskarayelu merged 1 commit into
Remitwise-Org:mainfrom
maixuancanh:test/csv-escape-coverage
Jul 24, 2026
Merged

test: cover Unicode CSV escaping#997
Baskarayelu merged 1 commit into
Remitwise-Org:mainfrom
maixuancanh:test/csv-escape-coverage

Conversation

@maixuancanh

Copy link
Copy Markdown

Summary

  • add coverage for Unicode CSV fields that do not require quoting
  • verify Unicode content is preserved when commas and quotes trigger RFC 4180 escaping

Verification

  • node node_modules/vitest/vitest.mjs run --config vitest.config.mjs --configLoader runner tests/unit/utils/export-serializer.test.ts (13 tests passed)
  • npx eslint tests/unit/utils/export-serializer.test.ts

Repository-wide lint and build were also attempted. They are currently blocked by unrelated errors already present on main, including JSX parsing errors, duplicate declarations, and missing modules.

Closes #940

@Baskarayelu
Baskarayelu merged commit 7ffa092 into Remitwise-Org:main Jul 24, 2026
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.

Add unit tests for the CSV escape function

2 participants