Home Original page

vm: add vm proto property lookup test by legendecas · Pull Request #54606 · nodejs/node

@nodejs-github-bot added needs-ci

PRs that need a full CI run.

test

Issues and PRs related to the tests.

labels

Aug 28, 2024

@legendecas 

Add more test coverage on vm prototype properties lookup with
`in` operator and property access.

@avivkeller avivkeller added the vm

Issues and PRs related to the vm subsystem.

label

Aug 28, 2024

joyeecheung

aduh95 pushed a commit that referenced this pull request

Sep 12, 2024 
Add more test coverage on vm prototype properties lookup with
`in` operator and property access.

PR-URL: #54606
Refs: #54436
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

@targos targos added the dont-land-on-v20.x

PRs that should not land on the v20.x-staging branch and should not be released in v20.x.

label

Sep 22, 2024

louwers pushed a commit to louwers/node that referenced this pull request

Nov 2, 2024 
Add more test coverage on vm prototype properties lookup with
`in` operator and property access.

PR-URL: nodejs#54606
Refs: nodejs#54436
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

tpoisseau pushed a commit to tpoisseau/node that referenced this pull request

Nov 21, 2024 
Add more test coverage on vm prototype properties lookup with
`in` operator and property access.

PR-URL: nodejs#54606
Refs: nodejs#54436
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>