nodejs/node

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

Open

#53,867 创建于 2024年7月16日

在 GitHub 查看
 (13 评论) (5 反应) (0 负责人)JavaScript (35,535 fork)batch import
good first issuetest_runner

仓库指标

Star
 (117,218 star)
PR 合并指标
 (平均合并 18天 17小时) (30 天内合并 219 个 PR)

描述

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.

贡献者指南

研究方向
该问题要求将 process.argv 和 process.cwd() 的捕获从 runner.js 移动到 test runner.js，并将它们作为选项传递给 run()。请检查 lib/internal/test runner/runner.js 第 472 行附近和 lib/internal/main/test runner.js 第 68 行附近的代码，以了解当前用法。确保所有对 process 属性的依赖都已重新定位并作为参数传递。评论中没有链接的 PR 或额外上下文。
技术栈
javascriptnodejs
领域
backendtesting
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
JavaScriptNode.js internals
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65