Browse Source

AMBARI-9013. Alerts UI: type=aggregate should have interval as editable.(xiwang)

Xi Wang 10 years ago
parent
commit
f90ac9e67e

+ 3 - 0
ambari-web/app/controllers/main/alerts/definition_configs_controller.js

@@ -304,6 +304,9 @@ App.MainAlertDefinitionConfigsController = Em.Controller.extend({
       App.AlertConfigProperties.Description.create({
         value: isWizard ? '' : alertDefinition.get('description')
       }),
+      App.AlertConfigProperties.Interval.create({
+        value: isWizard ? '' : alertDefinition.get('interval')
+      }),
       App.AlertConfigProperties.Thresholds.OkThreshold.create({
         label: 'Thresholds',
         showInputForValue: false,

+ 2 - 1
ambari-web/test/controllers/main/alerts/definitions_configs_controller_test.js

@@ -283,6 +283,7 @@ describe('App.MainAlertDefinitionConfigsController', function () {
       controller.set('content', Em.Object.create({
         name: 'alertDefinitionName',
         description: 'alertDefinitionDescription',
+        interval: 60,
         reporting: [
           Em.Object.create({
             type: 'warning',
@@ -301,7 +302,7 @@ describe('App.MainAlertDefinitionConfigsController', function () {
 
       var result = controller.renderAggregateConfigs();
 
-      expect(result.length).to.equal(4);
+      expect(result.length).to.equal(5);
     });
 
   });