From 24cf1f0f9aeddf7f7dc402928db5f7d1c83caa9c Mon Sep 17 00:00:00 2001 From: Niko Lehto Date: Tue, 28 Jan 2020 10:01:54 +0100 Subject: [PATCH] Make clipBoardPasteFrom() test more specific Don't rely on clientCutText() to test clipboardPasteFrom(). --- kasmweb/tests/test.rfb.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/kasmweb/tests/test.rfb.js b/kasmweb/tests/test.rfb.js index 3eee247..624644e 100644 --- a/kasmweb/tests/test.rfb.js +++ b/kasmweb/tests/test.rfb.js @@ -291,12 +291,18 @@ describe('Remote Frame Buffer Protocol Client', function () { }); describe('#clipboardPasteFrom', function () { + beforeEach(function () { + sinon.spy(RFB.messages, 'clientCutText'); + }); + + afterEach(function () { + RFB.messages.clientCutText.restore(); + }); + it('should send the given text in a paste event', function () { - const expected = {_sQ: new Uint8Array(11), _sQlen: 0, - _sQbufferSize: 11, flush: () => {}}; - RFB.messages.clientCutText(expected, 'abc'); client.clipboardPasteFrom('abc'); - expect(client._sock).to.have.sent(expected._sQ); + expect(RFB.messages.clientCutText).to.have.been.calledOnce; + expect(RFB.messages.clientCutText).to.have.been.calledWith(client._sock, 'abc'); }); it('should flush multiple times for large clipboards', function () {