From d2144ed6a3d178f96405e5c724f2d9c9206525a4 Mon Sep 17 00:00:00 2001 From: Diana Derevyankina Date: Thu, 2 Sep 2021 15:10:55 +0300 Subject: [PATCH 1/2] Fix and re-enable Vega test to be consistent with development/CI --- .../public/__snapshots__/vega_visualization.test.js.snap | 6 +++--- src/plugins/vis_type_vega/public/vega_visualization.test.js | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/plugins/vis_type_vega/public/__snapshots__/vega_visualization.test.js.snap b/src/plugins/vis_type_vega/public/__snapshots__/vega_visualization.test.js.snap index c70c4406a34f2..8915dbcc149c4 100644 --- a/src/plugins/vis_type_vega/public/__snapshots__/vega_visualization.test.js.snap +++ b/src/plugins/vis_type_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_type_vega/public/vega_visualization.test.js b/src/plugins/vis_type_vega/public/vega_visualization.test.js index ba1121b8894e0..cbc853423e76a 100644 --- a/src/plugins/vis_type_vega/public/vega_visualization.test.js +++ b/src/plugins/vis_type_vega/public/vega_visualization.test.js @@ -83,7 +83,7 @@ describe('VegaVisualizations', () => { }); // 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 +105,7 @@ describe('VegaVisualizations', () => { expect(domNode.innerHTML).toMatchSnapshot(); mockedWidthValue = 256; - mockedHeightValue = 256; + mockedHeightValue = 250; await vegaVis.vegaView.resize(); @@ -116,7 +116,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()); From 35fb6d610ca20459ecc99fb1a67a65d23baeb2b3 Mon Sep 17 00:00:00 2001 From: Diana Derevyankina Date: Tue, 14 Sep 2021 10:15:55 +0300 Subject: [PATCH 2/2] Remove skip comments --- src/plugins/vis_types/vega/public/vega_visualization.test.js | 3 --- 1 file changed, 3 deletions(-) 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 4d7a45f7dbddd..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,8 +81,6 @@ describe('VegaVisualizations', () => { mockWidth.mockRestore(); mockHeight.mockRestore(); }); - - // SKIP: https://github.com/elastic/kibana/issues/83385 test('should show vegalite graph and update on resize (may fail in dev env)', async () => { let vegaVis; try { @@ -115,7 +113,6 @@ describe('VegaVisualizations', () => { } }); - // SKIP: https://github.com/elastic/kibana/issues/83385 test('should show vega graph (may fail in dev env)', async () => { let vegaVis; try {