Skip to content

Test UTF-8 in all inputs and outputs to close #735 - #775

Merged
mathiasrw merged 1 commit into
mainfrom
fix/735
Oct 23, 2026
Merged

mathiasrw merged 1 commit into
mainfrom
fix/735

Conversation

@mathiasrw

Copy link
Copy Markdown
Owner

Closes #735

Stacked on #774.

Change

  • test/js/utf8.test.js puts æ, 中, 🦖, 👩‍💻 and \r\n in every input and output: the pattern (as text, . with and without --unicode, --literal), the replacement (plain, € and §, --replacement-js), the content of a file (in place, --output, --output-match), piped data (as content and with --replacement-pipe), file names and globs (as given and with --full-path), and the delimiters (with and without a tail).
  • Each case checks what real stdout or a file gets, so half an emoji shows up as �.
  • test/cli/run.sh has a few of the same through the CLI, as examples.

Good to know

  • One test shows the known problem: without --unicode, . takes half of 🦖, and the result has �.

@mathiasrw
mathiasrw added this pull request to stack #777 October 23, 2026 01:54
@mathiasrw
mathiasrw force-pushed the fix/735 branch 3 times, most recently from 0d755af to d665cb8 Compare October 23, 2026 02:01
Base automatically changed from fix/711 to main October 23, 2026 02:36
@mathiasrw
mathiasrw merged commit 2848894 into main Oct 23, 2026
6 of 7 checks passed
@mathiasrw
mathiasrw deleted the fix/735 branch October 23, 2026 02:42
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.

One test file for emoji and UTF-8 in all inputs and outputs

1 participant