nodejs/node

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

Offen

#53.867 geöffnet am 16.07.2024

Auf GitHub ansehenContributor Guide
 (14 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)JavaScript (35.535 Forks)batch import
good first issuetest_runner

Repository-Metriken

Stars
 (117.218 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem erfordert, die Erfassung von process.argv und process.cwd() von runner.js nach test runner.js zu verschieben und sie als Optionen an run() zu übergeben. Untersuche den Code in lib/internal/test runner/runner.js um Zeile 472 und in lib/internal/main/test runner.js um Zeile 68, um die aktuelle Verwendung zu verstehen. Stelle sicher, dass alle Abhängigkeiten von process Eigenschaften umgesetzt und als Parameter übergeben werden. Keine verknüpften PRs oder zusätzlicher Kontext in den Kommentaren.
Tech Stack
javascriptnodejs
Domain
backendtesting
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
JavaScriptNode.js internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65