Home Original page

test: cleanup IIFE tests by cjihrig · Pull Request #7694 · nodejs/node

@nodejs-github-bot added the test

Issues and PRs related to the tests.

label

Jul 13, 2016

@cjihrig 

A number of test files use IIFEs to separate distinct tests from
each other in the same file. The project has been moving toward
using block scopes and let/const in favor of IIFEs. This commit
moves IIFE tests to block scopes. Some additional cleanup such
as use of strictEqual() and common.mustCall() is also included.

PR-URL: nodejs#7694
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>

evanlucas pushed a commit that referenced this pull request

Jul 18, 2016 
A number of test files use IIFEs to separate distinct tests from
each other in the same file. The project has been moving toward
using block scopes and let/const in favor of IIFEs. This commit
moves IIFE tests to block scopes. Some additional cleanup such
as use of strictEqual() and common.mustCall() is also included.

PR-URL: #7694
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>

evanlucas pushed a commit that referenced this pull request

Jul 20, 2016 
A number of test files use IIFEs to separate distinct tests from
each other in the same file. The project has been moving toward
using block scopes and let/const in favor of IIFEs. This commit
moves IIFE tests to block scopes. Some additional cleanup such
as use of strictEqual() and common.mustCall() is also included.

PR-URL: #7694
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>