Skip to content

languages: Escape test names for bun runner#45749

Merged
Veykril merged 1 commit intozed-industries:mainfrom
pckilgore:pckilgore/45746-fix-bun-regexp
Jan 4, 2026
Merged

languages: Escape test names for bun runner#45749
Veykril merged 1 commit intozed-industries:mainfrom
pckilgore:pckilgore/45746-fix-bun-regexp

Conversation

@pckilgore
Copy link
Contributor

Closes #45746

Release Notes:

  • Fixed un-escaped regexp characters in test names passed to bun test

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 27, 2025
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same implementation I found for, e.g., TYPESCRIPT_VITEST_TEST_NAME_VARIABLE

@maxdeviant maxdeviant changed the title fix(bun): escape test names for runner languages: Escape test names for bun runner Dec 27, 2025
Copy link
Contributor

@versecafe versecafe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe the issue also exists for the node:test setup this slipped through the cracks when I added support for the two if you could patch that in this PR as well would be great.

@Veykril Veykril merged commit 64b0502 into zed-industries:main Jan 4, 2026
25 checks passed
@Veykril
Copy link
Member

Veykril commented Jan 4, 2026

Thanks!

rtfeldman pushed a commit that referenced this pull request Jan 5, 2026
Closes #45746

Release Notes:

- Fixed un-escaped regexp characters in test names passed to `bun test`
@pckilgore pckilgore deleted the pckilgore/45746-fix-bun-regexp branch January 5, 2026 19:45
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
Closes zed-industries#45746

Release Notes:

- Fixed un-escaped regexp characters in test names passed to `bun test`
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
Closes zed-industries#45746

Release Notes:

- Fixed un-escaped regexp characters in test names passed to `bun test`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bun test runner fails when test names contain regex control characters

3 participants