nodejs/node

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

Open

#53.867 aberto em 16 de jul. de 2024

Ver no GitHub
 (13 comments) (5 reactions) (0 assignees)JavaScript (35.535 forks)batch import
good first issuetest_runner

Métricas do repositório

Stars
 (117.218 stars)
Métricas de merge de PR
 (Mesclagem média 18d 17h) (219 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema requer mover a captura de process.argv e process.cwd() de runner.js para test runner.js e passá las como opções para run(). Examine o código em lib/internal/test runner/runner.js por volta da linha 472 e em lib/internal/main/test runner.js por volta da linha 68 para entender o uso atual. Certifique se de que todas as dependências nas propriedades de process sejam realocadas e passadas como parâmetros. Nenhum PR vinculado ou contexto adicional nos comentários.
Pilha de tecnologia
javascriptnodejs
Domain
backendtesting
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
JavaScriptNode.js internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65