Make sure types are passed through correctly for non-body items

readmeio · Dec 1, 2017 · 4c28df0 · 4c28df0
1 parent ab1406c
commit 4c28df0
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 3 deletions.
diff --git a/packages/api-explorer-ui/__tests__/lib/parameters-to-json-schema.test.js b/packages/api-explorer-ui/__tests__/lib/parameters-to-json-schema.test.js
@@ -197,6 +197,36 @@ test('should pass through defaults', () => {
   ]);
 });
 
+test('it should pass through type for non-body parameters', () => {
+  expect(
+    parametersToJsonSchema({
+      parameters: [
+        {
+          in: 'query',
+          name: 'checkbox',
+          schema: {
+            type: 'boolean',
+          },
+        },
+      ],
+    }),
+  ).toEqual([
+    {
+      label: 'Query Params',
+      type: 'query',
+      schema: {
+        type: 'object',
+        properties: {
+          checkbox: {
+            type: 'boolean',
+          },
+        },
+        required: [],
+      },
+    },
+  ]);
+});
+
 test('it should pass through description', () => {
   expect(
     parametersToJsonSchema({

diff --git a/packages/api-explorer-ui/dist/index.js b/packages/api-explorer-ui/dist/index.js
diff --git a/packages/api-explorer-ui/src/lib/parameters-to-json-schema.js b/packages/api-explorer-ui/src/lib/parameters-to-json-schema.js
@@ -43,6 +43,7 @@ function getOtherParams(pathOperation) {
 
         if (current.schema.default) schema.default = current.schema.default;
         if (current.schema.enum) schema.enum = current.schema.enum;
+        if (current.schema.type) schema.type = current.schema.type;
       }
 
       prev[current.name] = schema;