Skip to content

test: print arguments passed to mustNotCall function - #33951

Closed
lundibundi wants to merge 2 commits into
nodejs:masterfrom
lundibundi:test-must-not-call-print-args
Closed

test: print arguments passed to mustNotCall function#33951
lundibundi wants to merge 2 commits into
nodejs:masterfrom
lundibundi:test-must-not-call-print-args

Conversation

@lundibundi

Copy link
Copy Markdown
Member

Refs: #33949 (comment)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

/cc @nodejs/testing

Refs: nodejs#33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jun 18, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@lundibundilundibundi added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 19, 2020
jasnell pushed a commit that referenced this pull request Jun 24, 2020
Refs: #33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
PR-URL: #33951
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell

Copy link
Copy Markdown
Member

Landed in 225a03c

@jasnelljasnell closed this Jun 24, 2020
codebytere pushed a commit that referenced this pull request Jun 27, 2020
Refs: #33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
PR-URL: #33951
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@codebyterecodebytere mentioned this pull request Jun 28, 2020
codebytere pushed a commit that referenced this pull request Jun 30, 2020
Refs: #33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
PR-URL: #33951
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
codebytere pushed a commit that referenced this pull request Jul 10, 2020
Refs: #33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
PR-URL: #33951
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@codebyterecodebytere mentioned this pull request Jul 13, 2020
codebytere pushed a commit that referenced this pull request Jul 14, 2020
Refs: #33949 (comment)
Signed-off-by: Denys Otrishko <shishugi@gmail.com>
PR-URL: #33951
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@lundibundi@nodejs-github-bot@jasnell@Trott@devsnek@trivikr