Merge branch 'SirCmpwn-fix-send-on-disconnect'

Updated patch to use the readyState constant even in the tests.
LucaBongiorni · Apr 30, 2016 · 3120c9b · 3120c9b
2 parents 32df3fd + cf0623f
commit 3120c9b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/include/websock.js b/include/websock.js
@@ -197,7 +197,7 @@ function Websock() {
             }
 
             if (this._websocket.bufferedAmount < this.maxBufferedAmount) {
-                if (this._sQlen > 0) {
+                if (this._sQlen > 0 && this._websocket.readyState === WebSocket.OPEN) {
                     this._websocket.send(this._encode_message());
                     this._sQlen = 0;
                 }

diff --git a/tests/test.websock.js b/tests/test.websock.js
@@ -173,6 +173,7 @@ describe('Websock', function() {
             it('should actually send on the websocket if the websocket does not have too much buffered', function () {
                 sock.maxBufferedAmount = 10;
                 sock._websocket.bufferedAmount = 8;
+                sock._websocket.readyState = WebSocket.OPEN
                 sock._sQ = new Uint8Array([1, 2, 3]);
                 sock._sQlen = 3;
                 var encoded = sock._encode_message();