test_runner: do not read from `process.argv` and `process.cwd()` in run() · nodejs/node#53867

(13 comments) (5 reactions) (0 assignees)JavaScript (117,218 stars) (35,535 forks)batch import

good first issuetest_runner

説明

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.

技術スタック: javascriptnodejs
領域: backendtesting
Issue 種別: refactor
難度: 3
推定時間: half day
活動状況: active
明確さ: mostly clear
前提条件: understanding of Node.js internalsknowledge of process.argv and process.cwd()familiarity with test runner module
初心者向け度: 65
調査方針: The issue requires moving the capture of process.argv and process.cwd() from runner.js to test runner.js and passing them as options to run(). Examine the code in lib/internal/test runner/runner.js around line 472 and lib/internal/main/test runner.js around line 68 to understand current usage. Ensure that all dependencies on process properties are relocated and passed as parameters. No linked PRs or additional context in comments.