From f73f850b1d53cb0d7d59f11456a75db373920b3f Mon Sep 17 00:00:00 2001 From: Matt Seddon Date: Wed, 2 Aug 2023 14:30:24 +1000 Subject: [PATCH] remove .only from integration tests --- .../src/test/suite/experiments/index.test.ts | 35 +++++++++---------- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/extension/src/test/suite/experiments/index.test.ts b/extension/src/test/suite/experiments/index.test.ts index e926fe2e6a..1b29e7ce3c 100644 --- a/extension/src/test/suite/experiments/index.test.ts +++ b/extension/src/test/suite/experiments/index.test.ts @@ -1297,27 +1297,24 @@ suite('Experiments Test Suite', () => { expect(mockShowPlots).to.be.calledWith(dvcDemoPath) }).timeout(WEBVIEW_TEST_TIMEOUT) - it.only( - 'should be able to handle a message to toggle only changed columns', - async () => { - const { columnsModel, messageSpy, mockMessageReceived } = - await buildExperimentsWebview({ - disposer: disposable - }) - - expect(columnsModel.getShowOnlyChanged()).to.be.false - messageSpy.resetHistory() - - mockMessageReceived.fire({ - type: MessageFromWebviewType.TOGGLE_SHOW_ONLY_CHANGED + it('should be able to handle a message to toggle only changed columns', async () => { + const { columnsModel, messageSpy, mockMessageReceived } = + await buildExperimentsWebview({ + disposer: disposable }) - expect(columnsModel.getShowOnlyChanged()).to.be.true - expect(messageSpy).to.be.calledWithMatch({ - showOnlyChanged: true - }) - } - ).timeout(WEBVIEW_TEST_TIMEOUT) + expect(columnsModel.getShowOnlyChanged()).to.be.false + messageSpy.resetHistory() + + mockMessageReceived.fire({ + type: MessageFromWebviewType.TOGGLE_SHOW_ONLY_CHANGED + }) + + expect(columnsModel.getShowOnlyChanged()).to.be.true + expect(messageSpy).to.be.calledWithMatch({ + showOnlyChanged: true + }) + }).timeout(WEBVIEW_TEST_TIMEOUT) it('should handle a message to stop experiments running', async () => { const { experiments, dvcExecutor } =