浏览代码

AMBARI-10658. Refactor stack-config's 'property_depended_by' information into a sub-resource - test fixes (dsen via srimanth)

Srimanth Gunturi 10 年之前
父节点
当前提交
ac96799834

+ 52 - 37
ambari-server/src/test/python/TestStackAdvisor.py

@@ -399,23 +399,28 @@ class TestStackAdvisorInitialization(TestCase):
                 ],
                 "property_name" : "yarn.nodemanager.resource.cpu-vcores",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": []
             },
             {
               "StackConfigurations" : {
-                "property_depended_by" : [
-                  {
-                    "type" : "yarn-site",
-                    "name" : "yarn.scheduler.maximum-allocation-mb"
-                  },
-                  {
-                    "type" : "yarn-site",
-                    "name" : "yarn.scheduler.minimum-allocation-mb"
-                  }
-                ],
                 "property_name" : "yarn.nodemanager.resource.memory-mb",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": [
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "yarn.scheduler.maximum-allocation-mb",
+                    "dependency_type": "yarn-site"
+                  }
+                },
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "yarn.scheduler.minimum-allocation-mb",
+                    "dependency_type": "yarn-site"
+                  }
+                },
+              ]
             },
             {
               "StackConfigurations" : {
@@ -435,45 +440,55 @@ class TestStackAdvisorInitialization(TestCase):
                 ],
                 "property_name" : "yarn.scheduler.maximum-allocation-mb",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": []
             },
             {
               "StackConfigurations" : {
                 "property_depended_by" : [ ],
                 "property_name" : "yarn.scheduler.maximum-allocation-vcores",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": []
             },
             {
               "StackConfigurations" : {
-                "property_depended_by" : [
-                  {
-                    "type" : "hive-site",
-                    "name" : "hive.tez.container.size"
-                  },
-                  {
-                    "type" : "mapred-site",
-                    "name" : "yarn.app.mapreduce.am.resource.mb"
-                  },
-                  {
-                    "type" : "mapred-site",
-                    "name" : "mapreduce.map.memory.mb"
-                  },
-                  {
-                    "type" : "mapred-site",
-                    "name" : "mapreduce.reduce.memory.mb"
-                  }
-                ],
                 "property_name" : "yarn.scheduler.minimum-allocation-mb",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": [
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "hive.tez.container.size",
+                    "dependency_type": "hive-site"
+                  }
+                },
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "yarn.app.mapreduce.am.resource.mb",
+                    "dependency_type": "mapred-site"
+                  }
+                },
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "mapreduce.map.memory.mb",
+                    "dependency_type": "mapred-site"
+                  }
+                },
+                {
+                  "StackConfigurationDependency" : {
+                    "dependency_name": "mapreduce.reduce.memory.mb",
+                    "dependency_type": "mapred-site"
+                  }
+                },
+              ]
             },
             {
               "StackConfigurations" : {
-                "property_depended_by" : [ ],
                 "property_name" : "yarn.scheduler.minimum-allocation-vcores",
                 "type" : "yarn-site.xml"
-              }
+              },
+             "dependencies": []
             }
           ]
         }
@@ -490,9 +505,9 @@ class TestStackAdvisorInitialization(TestCase):
     properties_dict = default_stack_advisor.getAffectedConfigs(services)
     expected_properties_dict = [{'name': 'yarn.scheduler.maximum-allocation-mb', 'type': 'yarn-site'},
                                 {'name': 'yarn.scheduler.minimum-allocation-mb', 'type': 'yarn-site'},
+                                {'name': 'hive.tez.container.size', 'type': 'hive-site'},
                                 {'name': 'yarn.app.mapreduce.am.resource.mb', 'type': 'mapred-site'},
                                 {'name': 'mapreduce.map.memory.mb', 'type': 'mapred-site'},
-                                {'name': 'mapreduce.reduce.memory.mb', 'type': 'mapred-site'},
-                                {'name': 'hive.tez.container.size', 'type': 'hive-site'}]
+                                {'name': 'mapreduce.reduce.memory.mb', 'type': 'mapred-site'}]
 
     self.assertEquals(properties_dict, expected_properties_dict)

+ 11 - 9
ambari-web/test/mappers/configs/stack_config_properties_mapper_test.js

@@ -57,14 +57,16 @@ describe('App.stackConfigPropertiesMapper', function () {
                 "minimum": "512",
                 "maximum": "10240",
                 "unit": "MB"
-              },
-              "property_depended_by": [
-                {
-                  "property_type": "site4",
-                  "property_name": "p4"
+              }
+            },
+            "dependencies": [
+              {
+                "StackConfigurationDependency" : {
+                  "dependency_name" : "p4",
+                  "dependency_type" : "site4",
                 }
-              ]
-            }
+              }
+            ]
           }
         ]
       },
@@ -153,8 +155,8 @@ describe('App.stackConfigPropertiesMapper', function () {
       expect(App.StackConfigProperty.find('p1_site1').get('fileName')).to.eql('site1.xml');
       expect(App.StackConfigProperty.find('p1_site1').get('propertyDependedBy')).to.eql([
         {
-          "property_type": "site4",
-          "property_name": "p4"
+          "type": "site4",
+          "name": "p4"
         }
       ]);
       expect(App.StackConfigProperty.find('p1_site1').get('valueAttributes')).to.eql({