Skip to content

fix(ext/node): pass test-child-process-bad-stdio node compat test#31851

Merged
fraidev merged 9 commits intodenoland:mainfrom
fraidev:feat/node-test-mock-api
Jan 28, 2026
Merged

fix(ext/node): pass test-child-process-bad-stdio node compat test#31851
fraidev merged 9 commits intodenoland:mainfrom
fraidev:feat/node-test-mock-api

Conversation

@fraidev
Copy link
Copy Markdown
Contributor

@fraidev fraidev commented Jan 15, 2026

  • Added shell command transformation logic, which handles executable path detection (including shell variable references like "${VAR}"), argument splitting, and translation
  • Enable test-child-process-bad-stdio.js node compat test

Needs:

@fraidev fraidev added the ci-draft Run the CI on draft PRs. label Jan 15, 2026
@fraidev fraidev changed the title Feat/node test mock api feat(ext/node): implement mock API for node:test module Jan 15, 2026
@fraidev fraidev force-pushed the feat/node-test-mock-api branch from b043bf7 to 200bf7d Compare January 23, 2026 18:57
@fraidev fraidev changed the title feat(ext/node): implement mock API for node:test module fix(ext/node): pass test-child-process-bad-stdio node compat test Jan 23, 2026
@fraidev fraidev force-pushed the feat/node-test-mock-api branch from 5bfb1ae to e61a28d Compare January 23, 2026 19:47
@fraidev fraidev force-pushed the feat/node-test-mock-api branch from e61a28d to 4b8890d Compare January 23, 2026 19:54
@fraidev fraidev marked this pull request as ready for review January 23, 2026 20:15
@fraidev fraidev removed the ci-draft Run the CI on draft PRs. label Jan 23, 2026
@fraidev fraidev requested a review from bartlomieju January 24, 2026 03:26
@fraidev fraidev mentioned this pull request Jan 24, 2026
68 tasks
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it would be better to land this change in a separate PR with its own tests.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Make sense! Here a PR adding it with tests: #31954

@fraidev fraidev requested a review from bartlomieju January 27, 2026 20:21
Copy link
Copy Markdown
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM, please update PR description before landing

@fraidev fraidev enabled auto-merge (squash) January 28, 2026 02:34
@fraidev fraidev merged commit a1fb6b6 into denoland:main Jan 28, 2026
20 checks passed
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.

2 participants