From 2a967e9b25eb1e0cf1113a6d0d9e1712367cf46e Mon Sep 17 00:00:00 2001
From: ishuen <12228644+ishuen@users.noreply.github.com>
Date: Wed, 26 Oct 2022 02:01:07 +0800
Subject: [PATCH] fix(parameters): allowedValues for enum and boolean types
 (#8231)

* Change parameter with empty map

* Change allowValues data type

Co-authored-by: Tim Lai <timothy.lai@smartbear.com>
---
 src/core/json-schema-components.jsx    |  6 ++---
 src/helpers/get-parameter-schema.js    |  2 +-
 test/unit/components/parameter-row.jsx | 32 ++++++++++++++++++++++++++
 3 files changed, 36 insertions(+), 4 deletions(-)

diff --git a/src/core/json-schema-components.jsx b/src/core/json-schema-components.jsx
index 7d75df1c..647340ac 100644
--- a/src/core/json-schema-components.jsx
+++ b/src/core/json-schema-components.jsx
@@ -85,7 +85,7 @@ export class JsonSchema_string extends Component {
       const Select = getComponent("Select")
       return (<Select className={ errors.length ? "invalid" : ""}
                       title={ errors.length ? errors : ""}
-                      allowedValues={ enumValue }
+                      allowedValues={ [...enumValue] }
                       value={ value }
                       allowEmptyValue={ !required }
                       disabled={disabled}
@@ -335,14 +335,14 @@ export class JsonSchema_boolean extends Component {
     errors = errors.toJS ? errors.toJS() : []
     let enumValue = schema && schema.get ? schema.get("enum") : null
     let allowEmptyValue = !enumValue || !required
-    let booleanValue = !enumValue && fromJS(["true", "false"])
+    let booleanValue = !enumValue && ["true", "false"]
     const Select = getComponent("Select")
 
     return (<Select className={ errors.length ? "invalid" : ""}
                     title={ errors.length ? errors : ""}
                     value={ String(value) }
                     disabled={ disabled }
-                    allowedValues={ enumValue || booleanValue }
+                    allowedValues={ enumValue ? [...enumValue] : booleanValue }
                     allowEmptyValue={ allowEmptyValue }
                     onChange={ this.onEnumChange }/>)
   }
diff --git a/src/helpers/get-parameter-schema.js b/src/helpers/get-parameter-schema.js
index 8b2ff607..cdf463c6 100644
--- a/src/helpers/get-parameter-schema.js
+++ b/src/helpers/get-parameter-schema.js
@@ -86,7 +86,7 @@ export default function getParameterSchema(parameter, { isOAS3 } = {}) {
   }
 
   return {
-    schema: parameter.get("schema", Im.Map()),
+    schema: parameter.get("schema") ? parameter.get("schema", Im.Map()): Im.Map(),
     parameterContentMediaType: null,
   }
 }
diff --git a/test/unit/components/parameter-row.jsx b/test/unit/components/parameter-row.jsx
index 821b457b..b452682b 100644
--- a/test/unit/components/parameter-row.jsx
+++ b/test/unit/components/parameter-row.jsx
@@ -49,6 +49,21 @@ describe("<ParameterRow/>", () => {
     expect(wrapper.find(".parameter__type").text()).toEqual("string")
   })
 
+  it("Can render Swagger 2 parameter type boolean without format", () => {
+    const param = fromJS({
+      name: "hasId",
+      in: "path",
+      description: "boolean value to indicate if the pet has an id",
+      type: "boolean"
+    })
+
+    const props = createProps({ param, isOAS3: false })
+    const wrapper = render(<ParameterRow {...props}/>)
+
+    expect(wrapper.find(".parameter__type").length).toEqual(1)
+    expect(wrapper.find(".parameter__type").text()).toEqual("boolean")
+  })
+
   it("Can render OAS3 parameter type with format", () => {
     const param = fromJS({
       name: "petUuid",
@@ -83,6 +98,23 @@ describe("<ParameterRow/>", () => {
     expect(wrapper.find(".parameter__type").length).toEqual(1)
     expect(wrapper.find(".parameter__type").text()).toEqual("string")
   })
+
+  it("Can render OAS3 parameter type boolean without format", () => {
+    const param = fromJS({
+      name: "hasId",
+      in: "path",
+      description: "boolean value to indicate if the pet has an id",
+      schema: {
+        type: "boolean"
+      }
+    })
+
+    const props = createProps({ param, isOAS3: true })
+    const wrapper = render(<ParameterRow {...props}/>)
+
+    expect(wrapper.find(".parameter__type").length).toEqual(1)
+    expect(wrapper.find(".parameter__type").text()).toEqual("boolean")
+  })
 })
 
 describe("bug #5573: zero default and example values", function () {