Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

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

New issue
Copy link
New issue
Copy link
Open
#53937
Open
test_runner: do not read from process.argv and process.cwd() in run()#53867
#53937
Copy link
Labels
good first issueIssues that are suitable for first-time contributors.Issues that are suitable for first-time contributors.test_runnerIssues and PRs related to the test runner subsystem.Issues and PRs related to the test runner subsystem.
@mcollina

Description

@mcollina
mcollina
opened on Jul 16, 2024
Issue body actions

Currently the codebase for

node/lib/internal/test_runner/runner.js

Line 472 in 362afa5

function run(options = kEmptyObject) {
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

node/lib/internal/main/test_runner.js

Line 68 in 362afa5

const options = {
and pass it down to run() as options.

avivkeller, MoLow, GMgoes, eliphazbouye and ShubhamChavan1

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueIssues that are suitable for first-time contributors.Issues that are suitable for first-time contributors.test_runnerIssues and PRs related to the test runner subsystem.Issues and PRs related to the test runner subsystem.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions
      Morty Proxy This is a proxified and sanitized view of the page, visit original site.