Browse Source

Add objectCheck

Squeeze fix for required object on this PR
bubble
HelderSepu 7 years ago
parent
commit
87d11f99dd
2 changed files with 20 additions and 1 deletions
  1. +2
    -1
      src/core/utils.js
  2. +18
    -0
      test/core/utils.js

+ 2
- 1
src/core/utils.js View File

@@ -568,6 +568,7 @@ export const validateParam = (param, isXml, isOAS3 = false) => {
// These checks should evaluate to true if the parameter's value is valid
let stringCheck = type === "string" && value && !validateString(value)
let arrayCheck = type === "array" && Array.isArray(value) && value.length
let objectCheck = type === "object" && value.length
let listCheck = type === "array" && Im.List.isList(value) && value.count()
let fileCheck = type === "file" && value instanceof win.File
let booleanCheck = type === "boolean" && !validateBoolean(value)
@@ -589,7 +590,7 @@ export const validateParam = (param, isXml, isOAS3 = false) => {
if (err) errors.push(err)
}

if ( required && !(stringCheck || arrayCheck || listCheck || fileCheck || booleanCheck || numberCheck || integerCheck) ) {
if ( required && !(stringCheck || objectCheck || arrayCheck || listCheck || fileCheck || booleanCheck || numberCheck || integerCheck) ) {
errors.push("Required field is not provided")
return errors
}


+ 18
- 0
test/core/utils.js View File

@@ -326,6 +326,24 @@ describe("utils", function() {
expect( result ).toEqual( [] )
})

it("validates required objects", function() {
// invalid object
param = {
required: true,
type: "object",
value: ""
}
assertValidateParam(param, ["Required field is not provided"])

// valid object
param = {
required: true,
type: "object",
value: "test"
}
assertValidateParam(param, [])
})

it("validates required strings", function() {
// invalid string
param = {


Loading…
Cancel
Save