Merge pull request #6744 from elastic/jasper/backport/6736/6742/4.5

[backport] PR #6736 to 4.5
elastic · Apr 1, 2016 · 5412663 · 5412663
2 parents bbbb5a6 + 7c610b1
commit 5412663
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 4 deletions.
diff --git a/src/ui/public/agg_types/buckets/range.js b/src/ui/public/agg_types/buckets/range.js
@@ -26,7 +26,7 @@ define(function (require) {
 
         var id = RangeKey.idBucket(bucket);
 
-        var key = keys.get(id);
+        key = keys.get(id);
         if (!key) {
           key = new RangeKey(bucket);
           keys.set(id, key);

diff --git a/test/fixtures/scenario_manager.js b/test/fixtures/scenario_manager.js
@@ -120,7 +120,6 @@ ScenarioManager.prototype.loadIfEmpty = function (id) {
   var scenario = config[id];
   if (!scenario) throw new Error('No scenario found for ' + id);
 
-  var self = this;
   return Promise.all(scenario.bulk.map(function mapBulk(bulk) {
     var loadIndexDefinition;
 

diff --git a/test/functional/apps/discover/index.js b/test/functional/apps/discover/index.js
@@ -9,7 +9,6 @@ define(function (require) {
   var collapseExpand = require('./_collapse_expand');
 
   bdd.describe('discover app', function () {
-    var scenarioManager;
     var remote;
     var scenarioManager = new ScenarioManager(url.format(config.servers.elasticsearch));
     this.timeout = config.timeouts.default;

diff --git a/test/functional/apps/visualize/index.js b/test/functional/apps/visualize/index.js
@@ -20,7 +20,6 @@ define(function (require) {
 
   bdd.describe('visualize app', function () {
     var common;
-    var scenarioManager;
     var remote;
     var headerPage;
     var settingsPage;