From dd0d483f7d8531f9d13245be86d19b9a0c501afc Mon Sep 17 00:00:00 2001 From: Steve Sanderson Date: Wed, 24 Jul 2019 09:48:12 -0700 Subject: [PATCH] Fix Web.JS tests (#12513) --- .../Web.JS/tests/DefaultReconnectDisplay.test.ts | 1 - src/Components/Web.JS/tests/RenderQueue.test.ts | 7 ++++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Components/Web.JS/tests/DefaultReconnectDisplay.test.ts b/src/Components/Web.JS/tests/DefaultReconnectDisplay.test.ts index 0148da9e68..bce5ca2588 100644 --- a/src/Components/Web.JS/tests/DefaultReconnectDisplay.test.ts +++ b/src/Components/Web.JS/tests/DefaultReconnectDisplay.test.ts @@ -1,5 +1,4 @@ import { DefaultReconnectDisplay } from "../src/Platform/Circuits/DefaultReconnectDisplay"; -import { AutoReconnectCircuitHandler } from "../src/Platform/Circuits/AutoReconnectCircuitHandler"; import {JSDOM} from 'jsdom'; describe('DefaultReconnectDisplay', () => { diff --git a/src/Components/Web.JS/tests/RenderQueue.test.ts b/src/Components/Web.JS/tests/RenderQueue.test.ts index 0ee4c161e2..df22da3a8a 100644 --- a/src/Components/Web.JS/tests/RenderQueue.test.ts +++ b/src/Components/Web.JS/tests/RenderQueue.test.ts @@ -25,14 +25,15 @@ describe('RenderQueue', () => { }); - it('processBatch does not render previous batches', () => { + it('processBatch acknowledges previously rendered batches', () => { const queue = RenderQueue.getOrCreateQueue(3, NullLogger.instance); const sendMock = jest.fn(); const connection = { send: sendMock } as any as signalR.HubConnection; - queue.processBatch(1, new Uint8Array(0), connection); + queue.processBatch(2, new Uint8Array(0), connection); - expect(sendMock.mock.calls.length).toEqual(0); + expect(sendMock.mock.calls.length).toEqual(1); + expect(queue.getLastBatchid()).toEqual(2); }); it('processBatch does not render out of order batches', () => {