Home Original page

test: update var to let const in tests by GreenPioneer · Pull Request #9917 · nodejs/node

@nodejs-github-bot added the test

Issues and PRs related to the tests.

label

Dec 1, 2016

@mscdex mscdex added the tls

Issues and PRs related to the tls subsystem.

label

Dec 1, 2016

@imyller imyller added the code-and-learn

Issues related to the Code-and-Learn events and PRs submitted during the events.

label

Dec 1, 2016

@GreenPioneer GreenPioneer changed the title test: update var to let const in test-tis-on-empty-socket test: update var to let const in test-tis-on-empty-socket and test-child-process-stdout-flush-exit

Dec 1, 2016

@GreenPioneer GreenPioneer changed the title test: update var to let const in test-tis-on-empty-socket and test-child-process-stdout-flush-exit test: update var to let const in tests

Dec 1, 2016

lpinca

lpinca

lpinca

lpinca

@GreenPioneer 

Also updating assertStrict and moving the assert required.

jasnell pushed a commit that referenced this pull request

Dec 6, 2016 
Also updating assertStrict and moving the assert required.

PR-URL: #9917
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>

addaleax pushed a commit that referenced this pull request

Dec 8, 2016 
Also updating assertStrict and moving the assert required.

PR-URL: #9917
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>

addaleax pushed a commit to addaleax/node that referenced this pull request

Dec 8, 2016 
Also updating assertStrict and moving the assert required.

PR-URL: nodejs#9917
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Dec 21, 2016 
Also updating assertStrict and moving the assert required.

PR-URL: #9917
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>