diff options
| author | cjihrig <cjihrig@gmail.com> | 2016-07-12 19:47:32 -0400 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2016-07-14 22:07:14 -0400 |
| commit | 4a408321d9c4a6964c9d89a0dd09067f36b4dbfc (patch) | |
| tree | 5014fee0b2ef8772f4ebf60fd4bf27a72b0ef5b9 /test/sequential/test-child-process-execsync.js | |
| parent | 4220c24b17955f7f627bcedc89b6c2a0e655e618 (diff) | |
| download | android-node-v8-4a408321d9c4a6964c9d89a0dd09067f36b4dbfc.tar.gz android-node-v8-4a408321d9c4a6964c9d89a0dd09067f36b4dbfc.tar.bz2 android-node-v8-4a408321d9c4a6964c9d89a0dd09067f36b4dbfc.zip | |
test: cleanup IIFE tests
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: https://github.com/nodejs/node/pull/7694
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Diffstat (limited to 'test/sequential/test-child-process-execsync.js')
| -rw-r--r-- | test/sequential/test-child-process-execsync.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/sequential/test-child-process-execsync.js b/test/sequential/test-child-process-execsync.js index 2fad4ba8c2..fc3c0494dc 100644 --- a/test/sequential/test-child-process-execsync.js +++ b/test/sequential/test-child-process-execsync.js @@ -70,8 +70,8 @@ assert.strictEqual(ret, msg + '\n', } // Verify that stderr is not accessed when stdio = 'ignore' - GH #7966 -(function() { +{ assert.throws(function() { execSync('exit -1', {stdio: 'ignore'}); }, /Command failed: exit -1/); -})(); +} |