Skip to main content
GoodFirstPicks
DashboardIssuesReposLeaderboard

GoodFirstPicks by Leaveitblank © 2026

CreatorRequest a RepoPrivacy PolicyTerms of Service
test_runner: do not read from `process.argv` and `process.cwd()` in run() | GoodFirstPicks

test_runner: do not read from `process.argv` and `process.cwd()` in run()

nodejs/node 13 comments 1mo ago
View on GitHub
lowopenScope: somewhat clearSkill match: maybeTest focusedNode.jsJavaScript

Why this is a good first issue

Requires modifying test runner to avoid direct process access.

AI Summary

The issue involves modifying the test runner to avoid directly accessing `process.argv` and `process.cwd()` in the `run()` function, instead passing these as options from a higher level. The task is straightforward but depends on another PR landing first.

Issue Description

Currently the codebase for https://github.com/nodejs/node/blob/362afa52ebe462a39874915e5e70d261db153c58/lib/internal/test_runner/runner.js#L472 accesses some poperties of process, capturing some options there. However, we also expose run() to the end users, therefore we should capture all this information in https://github.com/nodejs/node/blob/362afa52ebe462a39874915e5e70d261db153c58/lib/internal/main/test_runner.js#L68 and pass it down to run() as options.

GitHub Labels

good first issuetest_runner

Want to work on this?

Claim this issue to let others know you're working on it. You'll earn 10 points when you complete it!

Risk Flags

  • dependency on another PR
Loading labels...

Details

Points10 pts
Difficultylow
Scopesomewhat clear
Skill Matchmaybe
Test Focusedyes