Repository: ambari
Updated Branches:
  refs/heads/trunk b9e979578 -> 40aac602f


http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/utils/string_utils_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/utils/string_utils_test.js 
b/ambari-web/test/utils/string_utils_test.js
index 9df143f..3c7b077 100644
--- a/ambari-web/test/utils/string_utils_test.js
+++ b/ambari-web/test/utils/string_utils_test.js
@@ -16,10 +16,10 @@
  * limitations under the License.
  */
 
-var string_utils = require('utils/string_utils');
+var stringUtils = require('utils/string_utils');
 require('utils/helper');
 
-describe('string_utils', function () {
+describe('stringUtils', function () {
 
   describe('#underScoreToCamelCase', function () {
     var tests = [
@@ -30,7 +30,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it(test.m + ' ', function () {
-        expect(string_utils.underScoreToCamelCase(test.i)).to.equal(test.e);
+        expect(stringUtils.underScoreToCamelCase(test.i)).to.equal(test.e);
       });
     });
   });
@@ -50,7 +50,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it(test.m + ' ', function () {
-        expect(string_utils.pad(test.i, test.l, test.f, 
test.a)).to.equal(test.e);
+        expect(stringUtils.pad(test.i, test.l, test.f, 
test.a)).to.equal(test.e);
       });
     });
   });
@@ -70,7 +70,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it(test.m + ' ', function () {
-        expect(string_utils.compareVersions(test.v1, 
test.v2)).to.equal(test.e);
+        expect(stringUtils.compareVersions(test.v1, test.v2)).to.equal(test.e);
       });
     });
   });
@@ -84,7 +84,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it(test.t + ' ' + test.m + ' ', function () {
-        expect(string_utils.isSingleLine(test.t)).to.equal(test.e);
+        expect(stringUtils.isSingleLine(test.t)).to.equal(test.e);
       });
     });
   });
@@ -92,7 +92,7 @@ describe('string_utils', function () {
   describe('#arrayToCSV', function() {
     var test = [{a: 1, b:2, c:3}, {a: 1, b:2, c:3}, {a: 1, b:2, c:3}];
     it('array of object to csv-string', function () {
-      expect(string_utils.arrayToCSV(test)).to.equal("1,2,3\n1,2,3\n1,2,3\n");
+      expect(stringUtils.arrayToCSV(test)).to.equal("1,2,3\n1,2,3\n1,2,3\n");
     });
   });
 
@@ -110,7 +110,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it('Check ' + typeof test.t, function () {
-        expect(string_utils.getFileFromPath(test.t)).to.equal(test.e);
+        expect(stringUtils.getFileFromPath(test.t)).to.equal(test.e);
       });
     });
   });
@@ -131,7 +131,7 @@ describe('string_utils', function () {
       ];
       tests.forEach(function(test) {
           it('Check ' + typeof test.t, function () {
-            expect(string_utils.getPath(test.t)).to.equal(test.e);
+            expect(stringUtils.getPath(test.t)).to.equal(test.e);
           });
       });
   });
@@ -156,7 +156,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it(test.i + ' to ' + test.e + ' ', function () {
-        expect(string_utils.getCamelCase(test.i)).to.equal(test.e);
+        expect(stringUtils.getCamelCase(test.i)).to.equal(test.e);
       });
     });
   });
@@ -251,7 +251,7 @@ describe('string_utils', function () {
     ];
     tests.forEach(function(test) {
       it('Check ' + typeof test.t, function () {
-        expect(string_utils.htmlEntities(test.t)).to.equal(test.e);
+        expect(stringUtils.htmlEntities(test.t)).to.equal(test.e);
       });
     });
   });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/utils/ui_effects_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/utils/ui_effects_test.js 
b/ambari-web/test/utils/ui_effects_test.js
index c4736ce..2c9ce7a 100644
--- a/ambari-web/test/utils/ui_effects_test.js
+++ b/ambari-web/test/utils/ui_effects_test.js
@@ -16,8 +16,7 @@
  * limitations under the License.
  */
 
-var ui_utils = require('utils/ui_effects');
-var App = require('app');
+var uiUtils = require('utils/ui_effects');
 
 describe('utils/ui_effects', function(){
   describe('#pulsate()', function(){
@@ -34,13 +33,13 @@ describe('utils/ui_effects', function(){
 
     it('opacity should be 0.2 on 5-th iteration', function() {
       var domEl = $('#pulsate-test-dom');
-      ui_utils.pulsate(domEl, 1000);
+      uiUtils.pulsate(domEl, 1000);
       this.clock.tick(300);
       expect(parseFloat(domEl.css('opacity')).toFixed(1)).to.eql('0.2');
     });
     it('should call callback at the end', function() {
       var domEl = $('#pulsate-test-dom');
-      ui_utils.pulsate(domEl, 1000, this.clb);
+      uiUtils.pulsate(domEl, 1000, this.clb);
       this.clock.tick(2000);
       expect(this.clb.calledOnce).to.be.ok;
     });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/utils/validator_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/utils/validator_test.js 
b/ambari-web/test/utils/validator_test.js
index 5591aee..5ff6185 100644
--- a/ambari-web/test/utils/validator_test.js
+++ b/ambari-web/test/utils/validator_test.js
@@ -232,8 +232,9 @@ describe('validator', function () {
       { obj: 1, detect: false },
       { obj: true, detect: false }
     ];
-    testable.forEach(function(value){
-      it('should ' + (value.detect ? '' : 'not') + ' detect empty value in `' 
+ new String(value.obj) + '`', function(){
+    testable.forEach(function(value) {
+      var detect = value.detect ? '' : 'not';
+      it('should {0} detect empty value in `{1}`'.format(detect, 
JSON.stringify(value.obj)), function() {
         expect(validator.empty(value.obj)).to.eql(value.detect);
       });
     });
@@ -420,7 +421,7 @@ describe('validator', function () {
         { value: 'a{1,2}{', expected: false }
       ];
     tests.forEach(function(test) {
-      it(message.format(test.value, (test.expected) ? 'valid' : 'not valid'), 
function() {
+      it(message.format(test.value, test.expected ? 'valid' : 'not valid'), 
function() {
         
expect(validator.isValidMatchesRegexp(test.value)).to.equal(test.expected);
       })
     });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/chart/linear_time_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/common/chart/linear_time_test.js 
b/ambari-web/test/views/common/chart/linear_time_test.js
index d21f830..377e392 100644
--- a/ambari-web/test/views/common/chart/linear_time_test.js
+++ b/ambari-web/test/views/common/chart/linear_time_test.js
@@ -382,7 +382,7 @@ describe('App.ChartLinearTimeView', function () {
         view.setYAxisFormatter();
         view.yAxisFormatter();
         methodNames.forEach(function (name) {
-          expect(App.ChartLinearTimeView[name].callCount).to.equal(Number(name 
== item.formatter));
+          expect(App.ChartLinearTimeView[name].callCount).to.equal(Number(name 
=== item.formatter));
         });
       });
     });
@@ -443,16 +443,16 @@ describe('App.ChartLinearTimeView.LoadAggregator', 
function () {
       result = aggregator.groupRequests(requests);
     });
     it("result['r1'].subRequests.length", function () {
-      expect(result['r1'].subRequests.length).to.equal(1);
+      expect(result.r1.subRequests.length).to.equal(1);
     });
     it("result['r1'].fields.length", function () {
-      expect(result['r1'].fields.length).to.equal(1);
+      expect(result.r1.fields.length).to.equal(1);
     });
     it("result['r2'].subRequests.length", function () {
-      expect(result['r2'].subRequests.length).to.equal(2);
+      expect(result.r2.subRequests.length).to.equal(2);
     });
     it("result['r2'].fields.length", function () {
-      expect(result['r2'].fields.length).to.equal(3);
+      expect(result.r2.fields.length).to.equal(3);
     });
   });
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/configs/overriddenProperty_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/common/configs/overriddenProperty_view_test.js 
b/ambari-web/test/views/common/configs/overriddenProperty_view_test.js
index 7520cef..16ab7e4 100644
--- a/ambari-web/test/views/common/configs/overriddenProperty_view_test.js
+++ b/ambari-web/test/views/common/configs/overriddenProperty_view_test.js
@@ -44,7 +44,7 @@ describe('App.ServiceConfigView.SCPOverriddenRowsView', 
function () {
 
   describe('#setSwitchText', function () {
 
-    var view = App.ServiceConfigView.SCPOverriddenRowsView.create({
+    view = App.ServiceConfigView.SCPOverriddenRowsView.create({
       serviceConfigProperty: {
         overrides: [
           Em.Object.create({

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/configs/service_config_container_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/common/configs/service_config_container_view_test.js 
b/ambari-web/test/views/common/configs/service_config_container_view_test.js
index 0cdef1e..e7aaa62 100644
--- a/ambari-web/test/views/common/configs/service_config_container_view_test.js
+++ b/ambari-web/test/views/common/configs/service_config_container_view_test.js
@@ -22,10 +22,7 @@ require('views/common/configs/services_config');
 
 describe('App.ServiceConfigContainerView', function () {
 
-  var view,
-    selectedService = {
-      configCategories: []
-    };
+  var view;
 
   beforeEach(function () {
     view = App.ServiceConfigContainerView.create({

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/configs/service_configs_by_category_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/common/configs/service_configs_by_category_view_test.js 
b/ambari-web/test/views/common/configs/service_configs_by_category_view_test.js
index 3451964..5b443b6 100644
--- 
a/ambari-web/test/views/common/configs/service_configs_by_category_view_test.js
+++ 
b/ambari-web/test/views/common/configs/service_configs_by_category_view_test.js
@@ -26,8 +26,6 @@ describe('App.ServiceConfigsByCategoryView', function () {
     serviceConfigs: []
   });
 
-  var result = [1, 2, 3, 4];
-
   var testData = [
     {
       title: 'four configs in correct order',
@@ -97,6 +95,7 @@ describe('App.ServiceConfigsByCategoryView', function () {
   App.TestAliases.testAsComputedIfThenElse(view, 'isCategoryBodyVisible', 
'category.isCollapsed', 'display: none;', 'display: block;');
 
   describe('#sortByIndex', function () {
+    var result = [1, 2, 3, 4];
     testData.forEach(function(_test){
       it(_test.title, function () {
         
expect(view.sortByIndex(_test.configs).mapProperty('resultId')).to.deep.equal(result);
@@ -352,9 +351,8 @@ describe('App.ServiceConfigsByCategoryView', function () {
   });
 
   describe('#categoryConfigs', function () {
-    var view,
-      result = [1,2,3,4,5],
-      cases = [
+    var result = [1, 2, 3, 4, 5];
+    var cases = [
         {
           categoryNname: 'TestCategory',
           serviceConfigs: [

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/filter_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/common/filter_view_test.js 
b/ambari-web/test/views/common/filter_view_test.js
index 3bf8152..81abddf 100644
--- a/ambari-web/test/views/common/filter_view_test.js
+++ b/ambari-web/test/views/common/filter_view_test.js
@@ -328,9 +328,10 @@ describe('filters.getFilterByType', function () {
       }
     ];
 
-    testData.forEach(function(item){
-      it('Condition: ' + item.condition + ((item.result) ? ' - match ' : ' - 
doesn\'t match ' + 'value: ') +
-        item.value.mapProperty('componentName').join(" "), function () {
+    testData.forEach(function(item) {
+      var substr = item.condition + (item.result ? ' - match ' : ' - doesn\'t 
match ');
+      var components = item.value.mapProperty('componentName').join(' ');
+      it('Condition: {0} value: {1}'.format(substr, components), function () {
         expect(filter(item.value, item.condition)).to.equal(item.result);
       })
     });
@@ -389,9 +390,10 @@ describe('filters.getFilterByType', function () {
     ];
 
     testData.forEach(function(item){
-      it('Condition: ' + item.condition + ((item.result) ? ' - match ' : ' - 
doesn\'t match ' + 'value: ') + item.value, function () {
+      var substr = item.condition + (item.result ? ' - match ' : ' - doesn\'t 
match ');
+      it('Condition: {0} value: {1}'.format(substr, item.value), function () {
         expect(filter(item.value, item.condition)).to.equal(item.result);
-      })
+      });
     });
   });
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/quick_link_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/common/quick_link_view_test.js 
b/ambari-web/test/views/common/quick_link_view_test.js
index 1c985a1..cae4715 100644
--- a/ambari-web/test/views/common/quick_link_view_test.js
+++ b/ambari-web/test/views/common/quick_link_view_test.js
@@ -299,7 +299,7 @@ describe('App.QuickViewLinks', function () {
       quickViewLinks.setEmptyLinks();
       expect(quickViewLinks.get('quickLinks')).to.eql([{
         label: quickViewLinks.t('quick.links.error.label'),
-        url: 'javascript:alert("' + quickViewLinks.t('contact.administrator') 
+ '");return false;'
+        url: 'javascript:alert("' + quickViewLinks.t('contact.administrator') 
+ '");return false;' // eslint-disable-line no-script-url
       }]);
       expect(quickViewLinks.get('isLoaded')).to.be.true;
     });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/common/sort_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/common/sort_view_test.js 
b/ambari-web/test/views/common/sort_view_test.js
index 9bfae34..1820250 100644
--- a/ambari-web/test/views/common/sort_view_test.js
+++ b/ambari-web/test/views/common/sort_view_test.js
@@ -16,7 +16,6 @@
  * limitations under the License.
  */
 
-var App = require('app');
 var sort = require('views/common/sort_view');
 require('utils/misc');
 require('utils/string_utils');

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/login_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/login_test.js 
b/ambari-web/test/views/login_test.js
index 844b0f6..6050211 100644
--- a/ambari-web/test/views/login_test.js
+++ b/ambari-web/test/views/login_test.js
@@ -21,16 +21,17 @@ var App = require('app');
 require('views/login');
 
 var view,
-  controller,
   pass;
 
 describe('App.LoginView', function () {
 
   before(function () {
     sinon.stub(App, 'get', function(k) {
-      if (k === 'router') return {
-        login: Em.K
-      };
+      if (k === 'router') {
+        return {
+          login: Em.K
+        };
+      }
       return Em.get(App, k);
     });
   });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/admin/stack_upgrade/upgrade_wizard_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/admin/stack_upgrade/upgrade_wizard_view_test.js 
b/ambari-web/test/views/main/admin/stack_upgrade/upgrade_wizard_view_test.js
index a5b0def..25182a1 100644
--- a/ambari-web/test/views/main/admin/stack_upgrade/upgrade_wizard_view_test.js
+++ b/ambari-web/test/views/main/admin/stack_upgrade/upgrade_wizard_view_test.js
@@ -286,7 +286,7 @@ describe('App.upgradeWizardView', function () {
     afterEach(function () {
       this.mock.restore();
     });
-    var testCases = [
+    [
       {
         data: {
           failedItem: null,
@@ -442,7 +442,7 @@ describe('App.upgradeWizardView', function () {
   });
 
   describe("#upgradeStatusLabel", function () {
-    var testCases = [
+    [
       {
         data: {
           status: 'QUEUED',

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js 
b/ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js
index b151ee4..c4eaa74 100644
--- a/ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js
+++ b/ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js
@@ -207,7 +207,7 @@ describe('App.mainAdminStackVersionsView', function () {
 
     beforeEach(function () {
       sinon.stub(App, 'get', function (key) {
-        return key == 'supports.displayOlderVersions' ? displayOlderVersions : 
Em.get(App, key);
+        return key === 'supports.displayOlderVersions' ? displayOlderVersions 
: Em.get(App, key);
       });
     });
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/alert_definitions_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/alert_definitions_view_test.js 
b/ambari-web/test/views/main/alert_definitions_view_test.js
index 66d1188..d4344c3 100644
--- a/ambari-web/test/views/main/alert_definitions_view_test.js
+++ b/ambari-web/test/views/main/alert_definitions_view_test.js
@@ -51,8 +51,8 @@ describe('App.MainAlertDefinitionsView', function () {
     it('Add Ambari service to filters', function () {
       var serviceFilterClass = view.serviceFilterView;
       var content = serviceFilterClass.create({}).get('content');
-      expect(content[0].label==Em.I18n.t('common.all'));
-      expect(content[content.length-1].label==Em.I18n.t('app.name'));
+      expect(content[0].label === Em.I18n.t('common.all'));
+      expect(content[content.length - 1].label === Em.I18n.t('app.name'));
     });
   });
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/alerts/manage_alert_groups_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/alerts/manage_alert_groups_view_test.js 
b/ambari-web/test/views/main/alerts/manage_alert_groups_view_test.js
index 5bb005d..64e5c84 100644
--- a/ambari-web/test/views/main/alerts/manage_alert_groups_view_test.js
+++ b/ambari-web/test/views/main/alerts/manage_alert_groups_view_test.js
@@ -18,8 +18,6 @@
 
 var App = require('app');
 
-var view;
-
 function getView() {
   return App.MainAlertsManageAlertGroupView.create({
     controller: Em.Object.create()
@@ -28,10 +26,6 @@ function getView() {
 
 describe('App.MainAlertsManageAlertGroupView', function () {
 
-  beforeEach(function () {
-    view = getView();
-  });
-
   App.TestAliases.testAsComputedIfThenElse(getView(), 'removeButtonTooltip', 
'controller.isRemoveButtonDisabled', 
Em.I18n.t('alerts.actions.manage_alert_groups_popup.removeButtonDisabled'), 
Em.I18n.t('alerts.actions.manage_alert_groups_popup.removeButton'))
 
 });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/alerts/manage_alert_notifications_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/alerts/manage_alert_notifications_view_test.js 
b/ambari-web/test/views/main/alerts/manage_alert_notifications_view_test.js
index 6e8b993..eb68f2f 100644
--- a/ambari-web/test/views/main/alerts/manage_alert_notifications_view_test.js
+++ b/ambari-web/test/views/main/alerts/manage_alert_notifications_view_test.js
@@ -89,7 +89,7 @@ describe('App.ManageAlertNotificationsView', function () {
         selectedAlertNotification: null,        
         m: 'some alert notification is not selected and user is a non-admin 
operator',
         p: {
-          isAddButtonDisabled: true,
+          isAddButtonDisabled: true,
           isEditButtonDisabled: false,
           isRemoveButtonDisabled: false,
           isDuplicateButtonDisabled: false

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard/widgets/hbase_master_uptime_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/dashboard/widgets/hbase_master_uptime_test.js 
b/ambari-web/test/views/main/dashboard/widgets/hbase_master_uptime_test.js
index 3a43b54..4f19a35 100644
--- a/ambari-web/test/views/main/dashboard/widgets/hbase_master_uptime_test.js
+++ b/ambari-web/test/views/main/dashboard/widgets/hbase_master_uptime_test.js
@@ -28,7 +28,7 @@ describe('App.HBaseMasterUptimeView', function () {
   var tests = [
     {
       model: Em.Object.create({
-        masterStartTime: ((new Date()).getTime() - 192.1 * 24 * 3600 * 1000)
+        masterStartTime: new Date().getTime() - 192.1 * 24 * 3600 * 1000
       }),
       e: {
         isGreen: true,

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard/widgets/namenode_uptime_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/dashboard/widgets/namenode_uptime_test.js 
b/ambari-web/test/views/main/dashboard/widgets/namenode_uptime_test.js
index 607ecfd..bfd101c 100644
--- a/ambari-web/test/views/main/dashboard/widgets/namenode_uptime_test.js
+++ b/ambari-web/test/views/main/dashboard/widgets/namenode_uptime_test.js
@@ -28,7 +28,7 @@ describe('App.NameNodeUptimeView', function() {
   var tests = [
     {
       model: Em.Object.create({
-        nameNodeStartTime: ((new Date()).getTime() - 192.1*24*3600*1000)
+        nameNodeStartTime: new Date().getTime() - 192.1*24*3600*1000
       }),
       e: {
         isRed: false,

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard/widgets/resource_manager_uptime_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/dashboard/widgets/resource_manager_uptime_test.js 
b/ambari-web/test/views/main/dashboard/widgets/resource_manager_uptime_test.js
index bf4b637..1dd4df1 100644
--- 
a/ambari-web/test/views/main/dashboard/widgets/resource_manager_uptime_test.js
+++ 
b/ambari-web/test/views/main/dashboard/widgets/resource_manager_uptime_test.js
@@ -28,7 +28,7 @@ describe('App.ResourceManagerUptimeView', function() {
   var tests = [
     {
       model: Em.Object.create({
-        resourceManagerStartTime: ((new Date()).getTime() - 192.1*24*3600*1000)
+        resourceManagerStartTime: (new Date()).getTime() - 192.1*24*3600*1000
       }),
       e: {
         isRed: false,

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard/widgets/uptime_text_widget_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/dashboard/widgets/uptime_text_widget_test.js 
b/ambari-web/test/views/main/dashboard/widgets/uptime_text_widget_test.js
index 5179353..808e67b 100644
--- a/ambari-web/test/views/main/dashboard/widgets/uptime_text_widget_test.js
+++ b/ambari-web/test/views/main/dashboard/widgets/uptime_text_widget_test.js
@@ -80,8 +80,8 @@ describe('App.UptimeTextDashboardWidgetView', function() {
     ];
     timestamps.forEach(function(timestamp) {
       var uptimeTextDashboardWidgetView = 
App.UptimeTextDashboardWidgetView.create({thresh1:40, thresh2:70});
-      it('timestamp ' + timestamp.t + '. timeUnit should be ' + '"' + 
timestamp.e.timeUnit + '"', function() {
-        var result = uptimeTextDashboardWidgetView.uptimeProcessing(((new 
Date()).getTime() - timestamp.diff));
+      it('timestamp {0}. timeUnit should be "{1}"'.format(timestamp.t, 
timestamp.e.timeUnit), function() {
+        uptimeTextDashboardWidgetView.uptimeProcessing(new Date().getTime() - 
timestamp.diff);
         
expect(uptimeTextDashboardWidgetView.get('timeUnit')).to.equal(timestamp.e.timeUnit);
       });
     });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard/widgets_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/dashboard/widgets_test.js 
b/ambari-web/test/views/main/dashboard/widgets_test.js
index 35f0673..eb27237 100644
--- a/ambari-web/test/views/main/dashboard/widgets_test.js
+++ b/ambari-web/test/views/main/dashboard/widgets_test.js
@@ -19,7 +19,6 @@
 
 var App = require('app');
 require('messages');
-var filters = require('views/common/filter_view');
 require('mixins/common/userPref');
 require('mixins/common/localStorage');
 require('views/main/dashboard/widgets');
@@ -29,11 +28,11 @@ describe('App.MainDashboardWidgetsView', function () {
   var view = App.MainDashboardWidgetsView.create();
 
   describe('#setInitPrefObject', function () {
-    var host_metrics_widgets_count = 4;
-    var hdfs_widgets_count = 7;
-    var hbase_widgets_count = 4;
-    var yarn_widgets_count = 4;
-    var total_widgets_count = 20;
+    var hostMetricsWidgetsCount = 4;
+    var hdfsWidgetsCount = 7;
+    var hbaseWidgetsCount = 4;
+    var yarnWidgetsCount = 4;
+    var totalWidgetsCount = 20;
     var tests = Em.A([
       {
         models: {
@@ -43,7 +42,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: null
         },
         e: {
-          visibleL: total_widgets_count - host_metrics_widgets_count - 
hdfs_widgets_count - hbase_widgets_count - yarn_widgets_count - 1,
+          visibleL: totalWidgetsCount - hostMetricsWidgetsCount - 
hdfsWidgetsCount - hbaseWidgetsCount - yarnWidgetsCount - 1,
           hiddenL: 0
         },
         m: 'All models are null'
@@ -56,7 +55,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: null
         },
         e: {
-          visibleL: total_widgets_count - hdfs_widgets_count - 
hbase_widgets_count - yarn_widgets_count - 1,
+          visibleL: totalWidgetsCount - hdfsWidgetsCount - hbaseWidgetsCount - 
yarnWidgetsCount - 1,
           hiddenL: 0
         },
         m: 'hdfs_model, hbase_model, yarn_model are null'
@@ -69,7 +68,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: null
         },
         e: {
-          visibleL: total_widgets_count - hbase_widgets_count - 
yarn_widgets_count - 1,
+          visibleL: totalWidgetsCount - hbaseWidgetsCount - yarnWidgetsCount - 
1,
           hiddenL: 0
         },
         m: 'hbase_model, yarn_model are null'
@@ -82,7 +81,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: null
         },
         e: {
-          visibleL: total_widgets_count - hbase_widgets_count - 
yarn_widgets_count - 1,
+          visibleL: totalWidgetsCount - hbaseWidgetsCount - yarnWidgetsCount - 
1,
           hiddenL: 0
         },
         m: 'hbase_model and yarn_model are null'
@@ -95,7 +94,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: null
         },
         e: {
-          visibleL: total_widgets_count - yarn_widgets_count - 1,
+          visibleL: totalWidgetsCount - yarnWidgetsCount - 1,
           hiddenL: 1
         },
         m: 'yarn_model is null'
@@ -108,7 +107,7 @@ describe('App.MainDashboardWidgetsView', function () {
           yarn_model: {}
         },
         e: {
-          visibleL: total_widgets_count,
+          visibleL: totalWidgetsCount,
           hiddenL: 1
         },
         m: 'All models are not null'
@@ -253,7 +252,6 @@ describe('App.MainDashboardWidgetsView', function () {
     });
 
     describe("#applyFilterComplete()", function () {
-      var widget = {checked: true};
       beforeEach(function () {
         sinon.stub(view, 'postUserPref');
         sinon.stub(view, 'translateToReal');
@@ -496,7 +494,7 @@ describe('App.MainDashboardWidgetsView', function () {
     Em.keys(widgetsMap).forEach(function (item, index, array) {
       it(notEmptyModelTitle.format(item), function () {
         array.forEach(function (modelName) {
-          view.set(modelName, modelName == item ? {} : null);
+          view.set(modelName, modelName === item ? {} : null);
         });
         
expect(view.checkServicesChange(emptyCurrentPref).visible).to.eql(widgetsMap[item]);
       });
@@ -506,7 +504,7 @@ describe('App.MainDashboardWidgetsView', function () {
       it(emptyModelTitle.format(item), function () {
         var expected = [];
         array.forEach(function (modelName) {
-          if (modelName == item) {
+          if (modelName === item) {
             view.set(modelName, null);
           } else {
             view.set(modelName, {});

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/dashboard_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/dashboard_test.js 
b/ambari-web/test/views/main/dashboard_test.js
index 1236e2d..70e2eb2 100644
--- a/ambari-web/test/views/main/dashboard_test.js
+++ b/ambari-web/test/views/main/dashboard_test.js
@@ -16,16 +16,4 @@
  * limitations under the License.
  */
 
-
-var App = require('app');
-require('messages');
-var filters = require('views/common/filter_view');
-require('views/main/dashboard');
-var mainDashboardView;
-describe('App.MainDashboardView', function() {
-
-   beforeEach(function() {
-     mainDashboardView = App.MainDashboardView.create();
-   });
-
-});
+describe('App.MainDashboardView', function() {});

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/host/details/host_component_views/decommissionable_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/host/details/host_component_views/decommissionable_test.js
 
b/ambari-web/test/views/main/host/details/host_component_views/decommissionable_test.js
index 7c02870..b5cbbe5 100644
--- 
a/ambari-web/test/views/main/host/details/host_component_views/decommissionable_test.js
+++ 
b/ambari-web/test/views/main/host/details/host_component_views/decommissionable_test.js
@@ -28,13 +28,15 @@ describe('App.Decommissionable', function() {
 
   beforeEach(function() {
     sinon.stub(App.router, 'get', function (k) {
-      if (k === 'mainHostDetailsController.content') return Em.Object.create({
-        hostComponents: [
-          {
-            componentName: 'component'
-          }
-        ]
-      });
+      if (k === 'mainHostDetailsController.content') {
+        return Em.Object.create({
+          hostComponents: [
+            {
+              componentName: 'component'
+            }
+          ]
+        });
+      }
       return Em.get(App.router, k);
     });
   });

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/host/details/host_component_views/nodemanager_view_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/host/details/host_component_views/nodemanager_view_test.js
 
b/ambari-web/test/views/main/host/details/host_component_views/nodemanager_view_test.js
index 3f72c27..3aafe70 100644
--- 
a/ambari-web/test/views/main/host/details/host_component_views/nodemanager_view_test.js
+++ 
b/ambari-web/test/views/main/host/details/host_component_views/nodemanager_view_test.js
@@ -64,7 +64,7 @@ describe('App.NodeManagerComponentView', function () {
       expect(view.setStatusAs.calledWith('DECOMMISSIONED')).to.be.true;
     });
     it("RM contains host", function () {
-      var curObj =  {rm_metrics: {
+      var curObj = {rm_metrics: {
         cluster: {
           nodeManagers: [{
             HostName: 'host1'
@@ -75,7 +75,7 @@ describe('App.NodeManagerComponentView', function () {
       expect(view.setStatusAs.calledWith('DECOMMISSIONING')).to.be.true;
     });
     it("RM does not contain host", function () {
-      var curObj =  {rm_metrics: {
+      var curObj = {rm_metrics: {
         cluster: {
           nodeManagers: []
         }

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/host/summary_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/host/summary_test.js 
b/ambari-web/test/views/main/host/summary_test.js
index 3481f15..eb70014 100644
--- a/ambari-web/test/views/main/host/summary_test.js
+++ b/ambari-web/test/views/main/host/summary_test.js
@@ -331,7 +331,7 @@ describe('App.MainHostSummaryView', function() {
       sinon.stub(App.StackServiceComponent, 'find', function(component) {
         var customCommands = [];
 
-        if (component == 'WITH_CUSTOM_COMMANDS') {
+        if (component === 'WITH_CUSTOM_COMMANDS') {
           customCommands = ['CUSTOMCOMMAND'];
         }
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/service/info/metrics/ambari_metrics/regionserver_base_test.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/test/views/main/service/info/metrics/ambari_metrics/regionserver_base_test.js
 
b/ambari-web/test/views/main/service/info/metrics/ambari_metrics/regionserver_base_test.js
index 4ba93ac..3b0ee44 100644
--- 
a/ambari-web/test/views/main/service/info/metrics/ambari_metrics/regionserver_base_test.js
+++ 
b/ambari-web/test/views/main/service/info/metrics/ambari_metrics/regionserver_base_test.js
@@ -54,7 +54,7 @@ describe('App.ChartServiceMetricsAMS_RegionServerBaseView', 
function () {
     });
 
     it('data.length', function () {
-      
expect(this.result[0].data.length).to.equal(jsonData.metrics.hbase.regionserver['test'].length);
+      
expect(this.result[0].data.length).to.equal(jsonData.metrics.hbase.regionserver.test.length);
     });
 
     it('y-property', function () {

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/service/info/summary_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/service/info/summary_test.js 
b/ambari-web/test/views/main/service/info/summary_test.js
index df3aa24..72e468d 100644
--- a/ambari-web/test/views/main/service/info/summary_test.js
+++ b/ambari-web/test/views/main/service/info/summary_test.js
@@ -447,7 +447,7 @@ describe('App.MainServiceInfoSummaryView', function() {
   describe("#restartAllStaleConfigComponents", function () {
 
     describe('trigger restartAllServiceHostComponents', function () {
-      var view;
+
       beforeEach(function () {
         view = App.MainServiceInfoSummaryView.create({
           controller: Em.Object.create({
@@ -476,7 +476,6 @@ describe('App.MainServiceInfoSummaryView', function() {
 
     describe('trigger check last check point warning before triggering 
restartAllServiceHostComponents', function () {
 
-      var view;
       var mainServiceItemController;
 
       beforeEach(function () {

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/main/service/item_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/main/service/item_test.js 
b/ambari-web/test/views/main/service/item_test.js
index 8319699..af94f01 100644
--- a/ambari-web/test/views/main/service/item_test.js
+++ b/ambari-web/test/views/main/service/item_test.js
@@ -59,16 +59,16 @@ describe('App.MainServiceItemView', function () {
     var actionMap = App.HostComponentActionMap.getMap(view);
 
     var customActionsArray = [];
-    for (var iter in actionMap) {
+    Object.keys(actionMap).forEach(function (iter) {
       customActionsArray.push(actionMap[iter]);
-    }
+    });
     var customActions = 
customActionsArray.mapProperty('customCommand').filter(function (action) {
       return !nonCustomAction.contains(action);
     }).uniq();
 
     // remove null and undefined from the list
     customActions = customActions.filter(function (value) {
-      return value != null;
+      return !Em.isNone(value);
     });
 
     customActions.forEach(function (action) {
@@ -402,7 +402,7 @@ describe('App.MainServiceItemView', function () {
             {"action": "restartAllHostComponents", "context": "KNOX", "label": 
"Restart All", "cssClass": "icon-repeat", "disabled": false},
             {"action": "runSmokeTest", "label": "Run Service Check", 
"cssClass": "icon-thumbs-up-alt", "disabled": false},
             {"action": "turnOnOffPassive", "context": "Turn On Maintenance 
Mode for Knox", "label": "Turn On Maintenance Mode", "cssClass": "icon-medkit", 
"disabled": false},
-            {"action": "startLdapKnox", "customCommand": "STARTDEMOLDAP", 
"context": "Start Demo LDAP",  "label": "Start Demo LDAP", "cssClass": 
"icon-play-sign", "disabled": false},
+            {"action": "startLdapKnox", "customCommand": "STARTDEMOLDAP", 
"context": "Start Demo LDAP", "label": "Start Demo LDAP", "cssClass": 
"icon-play-sign", "disabled": false},
             {"action": "stopLdapKnox", "customCommand": "STOPDEMOLDAP", 
"context": "Stop Demo LDAP", "label": "Stop Demo LDAP", "cssClass": 
"icon-stop", "disabled": false},
             {"action": "downloadClientConfigs", "label": "Download Client 
Configs", "cssClass": "icon-download-alt", "isHidden": true, "disabled": false, 
"hasSubmenu": false, "submenuOptions": []}
           ]
@@ -436,7 +436,7 @@ describe('App.MainServiceItemView', function () {
       sinon.stub(App, 'get', function (k) {
         switch (k) {
           case 'isSingleNode':
-            return view.get('controller.content.serviceName') == 'HDFS';
+            return view.get('controller.content.serviceName') === 'HDFS';
           case 'supports.autoRollbackHA':
           case 'isRMHaEnabled':
           case 'isHaEnabled':
@@ -479,8 +479,9 @@ describe('App.MainServiceItemView', function () {
         ];
       });
 
-      sinon.stub(App.StackServiceComponent, 'find', function () {
-        switch (arguments[0]) {
+      /*eslint-disable complexity */
+      sinon.stub(App.StackServiceComponent, 'find', function (id) {
+        switch (id) {
           case 'NAMENODE':
             return Em.Object.create({ customCommands: ["DECOMMISSION", 
"REBALANCEHDFS"] });
           case 'RESOURCEMANAGER':
@@ -510,6 +511,7 @@ describe('App.MainServiceItemView', function () {
             ];
         }
       });
+      /*eslint-enable complexity */
     });
 
     afterEach(function () {

http://git-wip-us.apache.org/repos/asf/ambari/blob/40aac602/ambari-web/test/views/wizard/step1_view_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/views/wizard/step1_view_test.js 
b/ambari-web/test/views/wizard/step1_view_test.js
index 1cfb7ff..0dacad8 100644
--- a/ambari-web/test/views/wizard/step1_view_test.js
+++ b/ambari-web/test/views/wizard/step1_view_test.js
@@ -279,7 +279,7 @@ describe('App.WizardStep1View', function () {
 
   });
 
-  App.TestAliases.testAsComputedSomeBy(getView(), 'invalidUrlExist', 
'allRepositories', 'validation', App.Repository.validation['INVALID']);
+  App.TestAliases.testAsComputedSomeBy(getView(), 'invalidUrlExist', 
'allRepositories', 'validation', App.Repository.validation.INVALID);
 
   App.TestAliases.testAsComputedSomeBy(getView(), 'invalidFormatUrlExist', 
'allRepositories', 'invalidFormatError', true);
 
@@ -448,7 +448,7 @@ describe('App.WizardStep1View', function () {
       })
     ];
 
-    var controller = {
+    var ctrl = {
       content: {
         stacks: [
           Em.Object.create({
@@ -485,7 +485,7 @@ describe('App.WizardStep1View', function () {
     it('target group isn\'t isSelected', function () {
       view.reopen({
         operatingSystems: operatingSystems,
-        controller: controller
+        controller: ctrl
       });
       view.updateByCheckbox();
       var targetGroup = 
view.get('operatingSystems.firstObject.repositories.firstObject');
@@ -496,11 +496,11 @@ describe('App.WizardStep1View', function () {
     });
 
     it('target group is isSelected, skipValidationisSelected = true', function 
() {
-      controller.content.stacks[0].operatingSystems[0].selected = true;
+      ctrl.content.stacks[0].operatingSystems[0].selected = true;
       operatingSystems[0].set('isSelected', true);
       view.reopen({
         operatingSystems: operatingSystems,
-        controller: controller
+        controller: ctrl
       });
       view.updateByCheckbox();
       var targetGroup = 
view.get('operatingSystems.firstObject.repositories.firstObject');

Reply via email to