AMBARI-9268. Config changes doesn't affect client components (alexantonenko)

ambari-web/app/models/service.js

@@ -95,7 +95,7 @@ App.Service = DS.Model.extend({
    * actual_configs, then a restart is required.
    */
   isRestartRequired: function () {
-    var rhc = this.get('hostComponents').filterProperty('staleConfigs', true);
+    var rhc = App.HostComponent.find().filterProperty('service.serviceName', this.get('serviceName')).filterProperty('staleConfigs', true);
     var hc = {};
     rhc.forEach(function(_rhc) {
       var hostName = _rhc.get('hostName');

ambari-web/test/models/service_test.js

@@ -140,7 +140,9 @@ var service,
     ],
     [
       {
-        serviceName: 'HIVE',
+        service: {
+          serviceName: 'HIVE'
+        },
         staleConfigs: false
       }
     ]
@@ -148,6 +150,9 @@ var service,
   hostComponentsDataTrue = [
     [
       Em.Object.create({
+        service: {
+          serviceName: 'HDFS'
+        },
         staleConfigs: true,
         displayName: 'service0'
       })
@@ -157,6 +162,9 @@ var service,
         host: {
           publicHostName: 'host0'
         },
+        service: {
+          serviceName: 'HDFS'
+        },
         staleConfigs: true,
         displayName: 'service1'
       })
@@ -214,19 +222,28 @@ describe('App.Service', function () {
   });
 
   describe('#isRestartRequired', function () {
+    var mockHostComponentModel = function (mock) {
+      sinon.stub(App.HostComponent, 'find', function () {
+        return mock;
+      });
+    }
+    beforeEach(function () {
+      service.reopen({
+        serviceName: 'HDFS'
+      });
+    });
+    afterEach(function () {
+      App.HostComponent.find.restore();
+    });
     hostComponentsDataFalse.forEach(function (item) {
       it('should be false', function () {
-        service.reopen({
-          hostComponents: item
-        });
+        mockHostComponentModel(item);
         expect(service.get('isRestartRequired')).to.be.false;
       });
     });
     hostComponentsDataTrue.forEach(function (item) {
       it('should be true', function () {
-        service.reopen({
-          hostComponents: item
-        });
+        mockHostComponentModel(item);
         expect(service.get('isRestartRequired')).to.be.true;
       });
     });