nodejs/node

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

Ouverte

#53 867 ouverte le 16 juil. 2024

Voir sur GitHubGuide contributeur
 (14 commentaires) (5 réactions) (0 personne assignée)JavaScript (35 535 forks)batch import
good first issuetest_runner

Métriques du dépôt

Stars
 (117 218 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Le problème nécessite de déplacer la capture de process.argv et process.cwd() de runner.js vers test runner.js et de les passer comme options à run(). Examinez le code dans lib/internal/test runner/runner.js autour de la ligne 472 et dans lib/internal/main/test runner.js autour de la ligne 68 pour comprendre l'utilisation actuelle. Assurez vous que toutes les dépendances aux propriétés de process soient relocalisées et passées en tant que paramètres. Aucune PR liée ou contexte supplémentaire dans les commentaires.
Stack technique
javascriptnodejs
Domaine
backendtesting
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaScriptNode.js internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65