diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-03-10 15:37:51 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2019-03-13 18:50:35 +0100 |
commit | 32853c0a136b0616844cdf5c276d2f1eb9a4fc50 (patch) | |
tree | d16b0fd161a3f8af6ec0407a74a71d42606a83c3 /lib/internal/js_stream_socket.js | |
parent | 5f38797ea51886708fbde22527fc54e64382925c (diff) | |
download | android-node-v8-32853c0a136b0616844cdf5c276d2f1eb9a4fc50.tar.gz android-node-v8-32853c0a136b0616844cdf5c276d2f1eb9a4fc50.tar.bz2 android-node-v8-32853c0a136b0616844cdf5c276d2f1eb9a4fc50.zip |
lib,test: improve faulty assert usage detection
This improves our custom eslint rules to detect assertions to detect
assertions with only a single argument and fixes false negatives in
case unary expressions are used.
Some rules were extended to also lint our docs and tools and the lib
rule was simplified to prohibit most assertion calls.
PR-URL: https://github.com/nodejs/node/pull/26569
Refs: https://github.com/nodejs/node/pull/26565
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Diffstat (limited to 'lib/internal/js_stream_socket.js')
-rw-r--r-- | lib/internal/js_stream_socket.js | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/internal/js_stream_socket.js b/lib/internal/js_stream_socket.js index 8343b6c264..038aa56c42 100644 --- a/lib/internal/js_stream_socket.js +++ b/lib/internal/js_stream_socket.js @@ -1,6 +1,6 @@ 'use strict'; -const assert = require('assert'); +const assert = require('internal/assert'); const util = require('util'); const { Socket } = require('net'); const { JSStream } = internalBinding('js_stream'); @@ -122,8 +122,8 @@ class JSStreamSocket extends Socket { this[kPendingShutdownRequest] = req; return 0; } - assert.strictEqual(this[kCurrentWriteRequest], null); - assert.strictEqual(this[kCurrentShutdownRequest], null); + assert(this[kCurrentWriteRequest] === null); + assert(this[kCurrentShutdownRequest] === null); this[kCurrentShutdownRequest] = req; const handle = this._handle; @@ -148,8 +148,8 @@ class JSStreamSocket extends Socket { } doWrite(req, bufs) { - assert.strictEqual(this[kCurrentWriteRequest], null); - assert.strictEqual(this[kCurrentShutdownRequest], null); + assert(this[kCurrentWriteRequest] === null); + assert(this[kCurrentShutdownRequest] === null); const handle = this._handle; const self = this; @@ -214,7 +214,7 @@ class JSStreamSocket extends Socket { setImmediate(() => { // Should be already set by net.js - assert.strictEqual(this._handle, null); + assert(this._handle === null); this.finishWrite(handle, uv.UV_ECANCELED); this.finishShutdown(handle, uv.UV_ECANCELED); |