[backport] PR #8537 to 4.6 - Reduce memory leaks in visualization tests

src/ui/public/vislib/__tests__/lib/dispatch.js

@@ -11,8 +11,7 @@ let data = require('fixtures/vislib/mock_data/date_histogram/_series');
 describe('Vislib Dispatch Class Test Suite', function () {
 
   function destroyVis(vis) {
-    $(vis.el).remove();
-    vis = null;
+    vis.destroy();
   }
 
   function getEls(el, n, type) {

src/ui/public/vislib/__tests__/lib/handler/handler.js

@@ -41,8 +41,7 @@ dateHistogramArray.forEach(function (data, i) {
     }));
 
     afterEach(function () {
-      $(vis.el).remove();
-      vis = null;
+      vis.destroy();
     });
 
     describe('render Method', function () {

src/ui/public/vislib/__tests__/lib/layout/layout.js

@@ -43,8 +43,7 @@ dateHistogramArray.forEach(function (data, i) {
     });
 
     afterEach(function () {
-      $(vis.el).remove();
-      vis = null;
+      vis.destroy();
     });
 
     describe('createLayout Method', function () {

src/ui/public/vislib/__tests__/vis.js

@@ -41,9 +41,8 @@ dataArray.forEach(function (data, i) {
     }));
 
     afterEach(function () {
-      $(vis.el).remove();
-      $(secondVis.el).remove();
-      vis = null;
+      vis.destroy();
+      secondVis.destroy();
     });
 
     describe('render Method', function () {

src/ui/public/vislib/__tests__/visualizations/area_chart.js

@@ -37,8 +37,7 @@ _.forOwn(someOtherVariables, function (variablesAreCool, imaVariable) {
     }));
 
     afterEach(function () {
-      $(vis.el).remove();
-      vis = null;
+      vis.destroy();
     });
 
     describe('checkIfEnoughData method throws an error when not enough data', function () {

src/ui/public/vislib/__tests__/visualizations/chart.js

@@ -105,6 +105,7 @@ describe('Vislib _chart Test Suite', function () {
 
   afterEach(function () {
     el.remove();
+    vis.destroy();
   });
 
   it('should be a constructor for visualization modules', function () {

src/ui/public/vislib/__tests__/visualizations/column_chart.js

@@ -49,8 +49,7 @@ dataTypesArray.forEach(function (dataType, i) {
     }));
 
     afterEach(function () {
-      $(vis.el).remove();
-      vis = null;
+      vis.destroy();
     });
 
     describe('stackData method', function () {

src/ui/public/vislib/__tests__/visualizations/line_chart.js

@@ -47,8 +47,7 @@ describe('Vislib Line Chart', function () {
       }));
 
       afterEach(function () {
-        $(vis.el).remove();
-        vis = null;
+        vis.destroy();
       });
 
       describe('addCircleEvents method', function () {

src/ui/public/vislib/__tests__/visualizations/pie_chart.js

@@ -105,8 +105,8 @@ describe('No global chart settings', function () {
   }));
 
   afterEach(function () {
-    chart1 = null;
-    chart2 = null;
+    chart1.destroy();
+    chart2.destroy();
   });
 
   it('should render chart titles for all charts', function () {
@@ -187,8 +187,7 @@ aggArray.forEach(function (dataAgg, i) {
     }));
 
     afterEach(function () {
-      $(vis.el).remove();
-      vis = null;
+      vis.destroy();
     });
 
     describe('addPathEvents method', function () {