X Tutup
The Wayback Machine - https://web.archive.org/web/20201101025119/https://github.com/nodejs/node/pull/35852
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add test to fs/promises setImmediate #35852

Merged

Conversation

@tyankatsu0105
Copy link
Contributor

@tyankatsu0105 tyankatsu0105 commented Oct 28, 2020

Added test pattern to setImmediate when ref's type is not boolean

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
@yosuke-furukawa yosuke-furukawa self-requested a review Oct 28, 2020
@shisama shisama added the request-ci label Oct 28, 2020
@github-actions github-actions bot removed the request-ci label Oct 28, 2020
@Leko
Leko approved these changes Oct 28, 2020
Copy link
Member

@rickyes rickyes left a comment

LGTM

@rickyes rickyes added the request-ci label Oct 29, 2020
@github-actions github-actions bot removed the request-ci label Oct 29, 2020
@nodejs-github-bot

This comment has been hidden.

@rickyes rickyes added the request-ci label Oct 29, 2020
@github-actions github-actions bot removed the request-ci label Oct 29, 2020
@Trott
Trott approved these changes Oct 29, 2020
@Trott Trott mentioned this pull request Oct 29, 2020
3 of 3 tasks complete
@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Oct 29, 2020

Add test pattern to setImmediate when ref's type is not boolean

PR-URL: #35852
Reviewed-By: Yosuke Furukawa <yosuke.furukawa@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@Trott Trott force-pushed the tyankatsu0105:coverage/test-timers-promisifiled branch from b08259a to 5dd344a Oct 31, 2020
@Trott Trott merged commit 5dd344a into nodejs:master Oct 31, 2020
23 of 24 checks passed
23 of 24 checks passed
test-asan test-asan
Details
build-tarball
Details
build-tarball
Details
build-windows-with-quic
Details
build-windows-with-quic
Details
coverage-linux
Details
coverage-linux
Details
coverage-windows
Details
coverage-windows
Details
lint-addon-docs
Details
lint-addon-docs
Details
build-docs
Details
build-docs
Details
test-asan
Details
test-linux
Details
test-macOS-with-quic
Details
lint-cpp
Details
lint-cpp
Details
test-linux-with-quic
Details
test-linux-with-quic
Details
lint-md
Details
lint-js
Details
lint-py
Details
lint-codeowners
Details
@Trott
Copy link
Member

@Trott Trott commented Oct 31, 2020

Landed in 5dd344a

@Trott
Copy link
Member

@Trott Trott commented Oct 31, 2020

Thanks for the contribution! 🎉

@tyankatsu0105 tyankatsu0105 deleted the tyankatsu0105:coverage/test-timers-promisifiled branch Oct 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

9 participants
You can’t perform that action at this time.
X Tutup