Skip to content

test: refactor test-cluster-basic - #13905

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:refactor-test-cluster-basic
Closed

test: refactor test-cluster-basic#13905
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:refactor-test-cluster-basic

Conversation

@Trott

Copy link
Copy Markdown
Member
  • Use common.mustNotCall() to check that callback is not invoked.
  • Add space per test writing guide.
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test cluster

* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
@TrottTrott added cluster Issues and PRs related to the cluster subsystem. test Issues and PRs related to the tests. labels Jun 25, 2017
@Trott

Copy link
Copy Markdown
MemberAuthor

@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 3b727aa

@TrottTrott closed this Jun 29, 2017
Trott added a commit to Trott/io.js that referenced this pull request Jun 29, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: nodejs#13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit that referenced this pull request Jun 29, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@addaleaxaddaleax mentioned this pull request Jun 29, 2017
addaleax pushed a commit that referenced this pull request Jul 11, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit that referenced this pull request Jul 18, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Aug 14, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Aug 15, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Aug 16, 2017
* Use common.mustNotCall() to check that callback is not invoked.
* Add space per test writing guide.
PR-URL: #13905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Aug 16, 2017
@Trott
Trott deleted the refactor-test-cluster-basic branch January 13, 2022 22:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clusterIssues and PRs related to the cluster subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@jasnell@lpinca@cjihrig@hiroppy@MylesBorins