Commit6fe7fc2

authored

feat: add validation for list(string) (#114)

* feat: add validation for list(string)* fix

1 parent23a23a2 commit6fe7fc2Copy full SHA for 6fe7fc2

File tree

+21

-0

lines changed

+21

-0

lines changed

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -390,6 +390,12 @@ func (v *Validation) Valid(typ, value string) error {`
`390`	`390`	`ifv.Monotonic!=""&&v.Monotonic!=ValidationMonotonicIncreasing&&v.Monotonic!=ValidationMonotonicDecreasing {`
`391`	`391`	`returnfmt.Errorf("number monotonicity can be either %q or %q",ValidationMonotonicIncreasing,ValidationMonotonicDecreasing)`
`392`	`392`	`}`
	`393`	`+case"list(string)":`
	`394`	`+varlistOfStrings []string`
	`395`	`+err:=json.Unmarshal([]byte(value),&listOfStrings)`
	`396`	`+iferr!=nil {`
	`397`	`+returnfmt.Errorf("value %q is not valid list of strings",value)`
	`398`	`+}`
`393`	`399`	`}`
`394`	`400`	`returnnil`
`395`	`401`	`}`

Lines changed: 15 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -514,6 +514,19 @@ func TestValueValidatesType(t *testing.T) {`
`514`	`514`	`Min:0,`
`515`	`515`	`Max:2,`
`516`	`516`	`Monotonic:"decreasing",`
	`517`	`+}, {`
	`518`	`+Name:"ValidListOfStrings",`
	`519`	`+Type:"list(string)",`
	`520`	+Value:`["first","second","third"]`,
	`521`	`+}, {`
	`522`	`+Name:"InvalidListOfStrings",`
	`523`	`+Type:"list(string)",`
	`524`	+Value:`["first","second","third"`,
	`525`	`+Error:regexp.MustCompile("is not valid list of strings"),`
	`526`	`+}, {`
	`527`	`+Name:"EmptyListOfStrings",`
	`528`	`+Type:"list(string)",`
	`529`	+Value:`[]`,
`517`	`530`	`}} {`
`518`	`531`	`tc:=tc`
`519`	`532`	`t.Run(tc.Name,func(t*testing.T) {`
`@@ -529,6 +542,8 @@ func TestValueValidatesType(t *testing.T) {`
`529`	`542`	`iftc.Error!=nil {`
`530`	`543`	`require.Error(t,err)`
`531`	`544`	`require.True(t,tc.Error.MatchString(err.Error()),"got: %s",err.Error())`
	`545`	`+}else {`
	`546`	`+require.NoError(t,err)`
`532`	`547`	`}`
`533`	`548`	`})`
`534`	`549`	`}`

Comments

(0)