test: simplify timers interval rejection assertion by HoonDongKang · Pull Request #65322 · nodejs/node · GitHub
Skip to content

test: simplify timers interval rejection assertion - #65322

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
HoonDongKang:simplify-test-timers-interval-promisified
Aug 23, 2026
Merged

test: simplify timers interval rejection assertion#65322
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
HoonDongKang:simplify-test-timers-interval-promisified

Conversation

@HoonDongKang

Copy link
Copy Markdown
Contributor

Replace a rejection-only try/catch block in
test-timers-interval-promisified.js with assert.rejects().

This simplifies the abort rejection test by verifying both the rejection and
the expected error shape in a single assertion, improving readability without
changing the tested behavior.

Refs:

Tests:

  • ./node test/parallel/test-timers-interval-promisified.js
  • tools/test.py test/parallel/test-timers-interval-promisified.js

Signed-off-by: HoonDongKang <d159123@naver.com>
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure. labels Aug 16, 2026
@codecov

codecov Bot commented Aug 16, 2026

Copy link
Copy Markdown

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 16, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 16, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@lpinca lpinca added request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. and removed request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. labels Aug 17, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@trivikr trivikr added commit-queue PRs queued for automated landing through the Commit Queue. author ready PRs with CI started, the required approvals, and no outstanding review comments. labels Aug 21, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 82e7ff4 into nodejs:main Aug 23, 2026
87 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 23, 2026
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Signed-off-by: HoonDongKang <d159123@naver.com>
PR-URL: #65322
Refs: #57338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Signed-off-by: HoonDongKang <d159123@naver.com>
PR-URL: #65322
Refs: #57338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 27, 2026
Signed-off-by: HoonDongKang <d159123@naver.com>
PR-URL: #65322
Refs: #57338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Sep 7, 2026
Signed-off-by: HoonDongKang <d159123@naver.com>
PR-URL: #65322
Refs: #57338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Sep 7, 2026
Signed-off-by: HoonDongKang <d159123@naver.com>
PR-URL: #65322
Refs: #57338
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants