From 410a0ac90741990b09aa9ee74130522f3e2705c7 Mon Sep 17 00:00:00 2001 From: Diana Derevyankina <54894989+DziyanaDzeraviankina@users.noreply.github.com> Date: Thu, 16 Sep 2021 11:42:02 +0300 Subject: [PATCH] Fix and re-enable Vega test to be consistent with development/CI (#110971) * Fix and re-enable Vega test to be consistent with development/CI * Remove skip comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> --- .../public/__snapshots__/vega_visualization.test.js.snap | 6 +++--- .../vis_types/vega/public/vega_visualization.test.js | 9 +++------ 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/src/plugins/vis_types/vega/public/__snapshots__/vega_visualization.test.js.snap b/src/plugins/vis_types/vega/public/__snapshots__/vega_visualization.test.js.snap index c70c4406a34f2..8915dbcc149c4 100644 --- a/src/plugins/vis_types/vega/public/__snapshots__/vega_visualization.test.js.snap +++ b/src/plugins/vis_types/vega/public/__snapshots__/vega_visualization.test.js.snap @@ -1,7 +1,7 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP -exports[`VegaVisualizations VegaVisualization - basics should show vega graph (may fail in dev env) 1`] = `"
"`; +exports[`VegaVisualizations VegaVisualization - basics should show vega graph (may fail in dev env) 1`] = `""`; -exports[`VegaVisualizations VegaVisualization - basics should show vegalite graph and update on resize (may fail in dev env) 1`] = `" "`; +exports[`VegaVisualizations VegaVisualization - basics should show vegalite graph and update on resize (may fail in dev env) 1`] = `" "`; -exports[`VegaVisualizations VegaVisualization - basics should show vegalite graph and update on resize (may fail in dev env) 2`] = `" "`; +exports[`VegaVisualizations VegaVisualization - basics should show vegalite graph and update on resize (may fail in dev env) 2`] = `" "`; diff --git a/src/plugins/vis_types/vega/public/vega_visualization.test.js b/src/plugins/vis_types/vega/public/vega_visualization.test.js index 8c47b2fdfd7c0..05a88880822ca 100644 --- a/src/plugins/vis_types/vega/public/vega_visualization.test.js +++ b/src/plugins/vis_types/vega/public/vega_visualization.test.js @@ -81,9 +81,7 @@ describe('VegaVisualizations', () => { mockWidth.mockRestore(); mockHeight.mockRestore(); }); - - // SKIP: https://github.com/elastic/kibana/issues/83385 - test.skip('should show vegalite graph and update on resize (may fail in dev env)', async () => { + test('should show vegalite graph and update on resize (may fail in dev env)', async () => { let vegaVis; try { vegaVis = new VegaVisualization(domNode, jest.fn()); @@ -105,7 +103,7 @@ describe('VegaVisualizations', () => { expect(domNode.innerHTML).toMatchSnapshot(); mockedWidthValue = 256; - mockedHeightValue = 256; + mockedHeightValue = 250; await vegaVis.vegaView.resize(); @@ -115,8 +113,7 @@ describe('VegaVisualizations', () => { } }); - // SKIP: https://github.com/elastic/kibana/issues/83385 - test.skip('should show vega graph (may fail in dev env)', async () => { + test('should show vega graph (may fail in dev env)', async () => { let vegaVis; try { vegaVis = new VegaVisualization(domNode, jest.fn());