test_runner: add `proto` null by rluvaton · Pull Request #48663 · nodejs/node · GitHub

extracted from:

tools: add eslint rule for __proto__: null in object #48646

TODO:

Run CITGM
mark as a major change

reference: #48646 (comment)

Review requested:

@nodejs/test_runner

anonrig added the needs-benchmark-ci PR that need a benchmark CI run.

@anonrig we don't have benchmarks for the test runner – and I don't think we need it for this PR.

TODO:

Run CITGM

mark as a major change

IMHO that would be overkill for this PR:

I don't think we need it for node:test code: it's been designed to minimize the risk of user code influencing its results, so the changes in this PR should not be observable for users – and if they are, we would consider it a bug, so this would still be a patch rather than a breaking change.
I don't think we need it for code related to node --test, because no user code would be run in the process.
Also, because this code is relatively new, I think it's fair to assume no one is relying on mutating %Object.prototype% anyway.

That being said, if others think differently, I'm OK with taking the more prudent path.

Any chance you could add the lint rule along with this PR, but make it apply only to lib/internal/test_runner?

CI: https://ci.nodejs.org/job/node-test-pull-request/52666/

Any chance you could add the lint rule along with this PR, but make it apply only to lib/internal/test_runner?

Sure

Any chance you could add the lint rule along with this PR, but make it apply only to lib/internal/test_runner?

see #48646 (comment)

@aduh95 I've added the eslint rule...

There are a few tests that need fixing. Also can you please rebase on top of main? Merge commits tend to break our tooling.

@aduh95 done

@rluvaton thanks, but it seems the merge commit is still there, and there are a few unrelated commits that were added. Did you maybe rebase main on top of your branch, instead of rebasing your branch on top of main? Let me know if you'd prefer I take care of it.

thanks, but it seems the merge commit is still there, and there are a few unrelated commits that were added. Did you maybe rebase main on top of your branch, instead of rebasing your branch on top of main? Let me know if you'd prefer I take care of it.

it would be great, I'm having problem for some reason

CI: https://ci.nodejs.org/job/node-test-pull-request/52915/

@aduh95 done and fixed the original issue as well

CI: https://ci.nodejs.org/job/node-test-pull-request/52965/

failed again on a different unreleated thing...

CI: https://ci.nodejs.org/job/node-test-pull-request/52970/

Landed in ee391f3

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

PR-URL: #48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

PR-URL: #48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

PR-URL: nodejs/node#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Jul 5, 2023

anonrig added the needs-benchmark-ci PR that need a benchmark CI run. label Jul 6, 2023

aduh95 approved these changes Jul 9, 2023

View reviewed changes

benjamingr added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 9, 2023

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 9, 2023

atlowChemi approved these changes Jul 19, 2023

View reviewed changes

aduh95 reviewed Jul 21, 2023

View reviewed changes

Comment thread .eslintrc.js Outdated

rluvaton force-pushed the add-proto-null-to-test-runner branch from 1862cb6 to 32b3373 Compare July 24, 2023 08:40

aduh95 approved these changes Jul 24, 2023

View reviewed changes

aduh95 added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Jul 24, 2023

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 24, 2023

This was referenced Jul 25, 2023

CI Reliability 2023-07-25 nodejs/reliability#623

Open

CI Reliability 2023-07-26 nodejs/reliability#624

Open

CI Reliability 2023-07-27 nodejs/reliability#625

Open

CI Reliability 2023-07-28 nodejs/reliability#626

Open

aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 28, 2023

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 28, 2023

atlowChemi approved these changes Jul 28, 2023

View reviewed changes

atlowChemi added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 28, 2023

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 28, 2023

github-actions Bot mentioned this pull request Jul 29, 2023

CI Reliability 2023-07-29 nodejs/reliability#627

Open

23 tasks

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 29, 2023

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 29, 2023

nodejs-github-bot merged commit ee391f3 into nodejs:main Jul 29, 2023

Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023

test_runner: add __proto__ null

c8c5ef9

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023

test_runner: add __proto__ null

fc67767

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

UlisesGascon pushed a commit to UlisesGascon/node that referenced this pull request Aug 14, 2023

test_runner: add __proto__ null

3f9b679

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

RafaelGSS pushed a commit that referenced this pull request Aug 15, 2023

test_runner: add __proto__ null

c9ad263

PR-URL: #48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

UlisesGascon mentioned this pull request Aug 15, 2023

v20.6.0 proposal #49185

Merged

RafaelGSS pushed a commit to RafaelGSS/node that referenced this pull request Aug 15, 2023

test_runner: add __proto__ null

8a93829

PR-URL: nodejs#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

RafaelGSS pushed a commit that referenced this pull request Aug 17, 2023

test_runner: add __proto__ null

fa19b0e

PR-URL: #48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

targos pushed a commit that referenced this pull request Nov 27, 2023

test_runner: add __proto__ null

3a74316

PR-URL: #48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

targos mentioned this pull request Nov 28, 2023

v18.19.0 release proposal #50953

Merged

sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024

test_runner: add __proto__ null

8425cab

PR-URL: nodejs/node#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024

test_runner: add __proto__ null

a5b13c7

PR-URL: nodejs/node#48663 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

test_runner: add `__proto__` null by rluvaton · Pull Request #48663 · nodejs/node · GitHub

Uh oh!

Uh oh!

test_runner: add __proto__ null - #48663

test_runner: add __proto__ null#48663nodejs-github-bot merged 1 commit intonodejs:mainnodejs/node:mainfrom rluvaton:add-proto-null-to-test-runnerrluvaton/node:add-proto-null-to-test-runnerCopy head branch name to clipboard

Conversation

rluvaton commented Jul 5, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nodejs-github-bot commented Jul 5, 2023

Uh oh!

aduh95 commented Jul 9, 2023

Uh oh!

aduh95 left a comment

Choose a reason for hiding this comment

Uh oh!

nodejs-github-bot commented Jul 9, 2023

Uh oh!

rluvaton commented Jul 10, 2023

Uh oh!

rluvaton commented Jul 12, 2023

Uh oh!

rluvaton commented Jul 20, 2023

Uh oh!

Uh oh!

aduh95 commented Jul 21, 2023

Uh oh!

rluvaton commented Jul 24, 2023

Uh oh!

aduh95 commented Jul 24, 2023

Uh oh!

rluvaton commented Jul 24, 2023

Uh oh!

nodejs-github-bot commented Jul 24, 2023

Uh oh!

rluvaton commented Jul 28, 2023

Uh oh!

nodejs-github-bot commented Jul 28, 2023

Uh oh!

rluvaton commented Jul 28, 2023

Uh oh!

nodejs-github-bot commented Jul 28, 2023

Uh oh!

nodejs-github-bot commented Jul 29, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

test_runner: add `proto` null by rluvaton · Pull Request #48663 · nodejs/node · GitHub

test_runner: add proto null - #48663

test_runner: add proto null#48663
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
rluvaton:add-proto-null-to-test-runner

rluvaton commented Jul 5, 2023 •
edited
Loading