Issue 3255

[shouldnotappearcalm]

What's this PR do/fix?

fix @Size annotation does not work properly on List

Are there unit tests? If not how should this be manually tested?

You can write the following code in the bean

@ApiModelProperty("list1")
@Size(min = 1, max = 200, message = "1-200")
private List<String> list1;

Then you can see in the generated swagger json

list1:- {
type: "array",
description: "list1",
items:- {
  type: "string"
},
maxItems: 200,
minItems: 1
}

Any background context you want to provide?

As above

What are the relevant issues?

3255

[codecov]

Codecov Report

Merging #3259 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3259   +/-   ##
=========================================
  Coverage     92.91%   92.91%           
- Complexity     3512     3513    +1     
=========================================
  Files           382      382           
  Lines          9330     9335    +5     
  Branches        768      769    +1     
=========================================
+ Hits           8669     8674    +5     
  Misses          472      472           
  Partials        189      189           

[shouldnotappearcalm]

I looked at the test methods in the EnumMapperSpec class, but I feel that ArrayProperty class does not seem to be particularly suitable for current methods, because it does not have properties such as enums and minimum. Do I need to add a test method? @dilipkrish

[dilipkrish]

Thank you for your contribution 🤘 . You can add a new method for your test. Without that there is no safety net for your change.

[shouldnotappearcalm]

Hi,I have added the corresponding unit tests for the new code. Please take a look when you are free. @dilipkrish

[dilipkrish]

Thank you @shouldnotappearcalm