Skip to content

test: add "mustCall" to test-fs-readfile-unlink - #27453

Closed
ckthewu wants to merge 1 commit into
nodejs:masterfrom
ckthewu:master
Closed

test: add "mustCall" to test-fs-readfile-unlink#27453
ckthewu wants to merge 1 commit into
nodejs:masterfrom
ckthewu:master

Conversation

@ckthewu

Copy link
Copy Markdown

Add common.mustCall to test-fs-readfile-unlink

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

Add common.mustCall to test-fs-readfile-unlink
@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Apr 28, 2019
@joyeecheungjoyeecheung added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Apr 28, 2019
@gireeshpunathil

Copy link
Copy Markdown
Member

please leave 👍 to fast track

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@starkwang

Copy link
Copy Markdown
Contributor

Landed in 413256d

starkwang pushed a commit that referenced this pull request Apr 29, 2019
Add common.mustCall to test-fs-readfile-unlink
PR-URL: #27453
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
targos pushed a commit that referenced this pull request Apr 29, 2019
Add common.mustCall to test-fs-readfile-unlink
PR-URL: #27453
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@targostargos mentioned this pull request May 6, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@ckthewu@gireeshpunathil@nodejs-github-bot@starkwang@cjihrig@joyeecheung@ZYSzys