feat(SchemaUtil): parse a partial definition

dcos · Sep 11, 2017 · 552770a · 552770a
1 parent d3e0f08
commit 552770a
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/src/js/utils/SchemaUtil.js b/src/js/utils/SchemaUtil.js
@@ -268,7 +268,7 @@ const SchemaUtil = {
 
     Object.keys(schemaProperties).forEach(function(topLevelProp) {
       const topLevelPropertyObject = schemaProperties[topLevelProp];
-      const secondLevelProperties = topLevelPropertyObject.properties;
+      const secondLevelProperties = topLevelPropertyObject.properties || {};
       const requiredProps = topLevelPropertyObject.required;
       const definitionForm = (multipleDefinition[topLevelProp] = {});
 

diff --git a/src/js/utils/__tests__/SchemaUtil-test.js b/src/js/utils/__tests__/SchemaUtil-test.js
@@ -48,6 +48,29 @@ describe("SchemaUtil", function() {
       });
     });
 
+    describe("schema with no second level properties", function() {
+      const $scope = {};
+      beforeEach(function() {
+        var schema = {
+          properties: {
+            application: {
+              description: "This is a description"
+            }
+          }
+        };
+
+        $scope.subheaderRender = jasmine.createSpy();
+        $scope.result = SchemaUtil.schemaToMultipleDefinition({
+          schema,
+          renderSubheader: $scope.subheaderRender
+        });
+      });
+
+      it("creates the definition", function() {
+        expect($scope.result.application.definition).toEqual([]);
+      });
+    });
+
     describe("nested schema", function() {
       beforeEach(function() {
         var schema = {