jcarr
/
noVNC
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove binary protocol tests 

We require standard adherence now, so remove tests that assume
the old system.
This commit is contained in:
Pierre Ossman 2017-02-03 17:16:02 +01:00
parent c4482d2de5
commit ec7ba3eeae
1 changed files with 0 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
tests/test.websock.js
View File

@ -249,10 +249,6 @@ describe('Websock', function() {
expect(WebSocket).to.have.been.calledWith('ws://localhost:8675', 'binary');
});
it('should fail if we specify a protocol besides binary', function () {
expect(function () { sock.open('ws:///', 'base64'); }).to.throw(Error);
});
// it('should initialize the event handlers')?
});
@ -312,17 +308,6 @@ describe('Websock', function() {
expect(sock._recv_message).to.have.been.calledOnce;
});
it('should fail if a protocol besides binary is requested', function () {
sock._websocket.protocol = 'base64';
expect(sock._websocket.onopen).to.throw(Error);
});
it('should assume binary if no protocol was available on opening', function () {
sock._websocket.protocol = null;
sock._websocket.onopen();
expect(sock._mode).to.equal('binary');
});
it('should call the open event handler on opening', function () {
sock._websocket.onopen();
expect(sock._eventHandlers.open).to.have.been.calledOnce;