support multiple ApiImplicitParameters

[kerwin612]

What's this PR do/fix?

support multiple ApiImplicitParameters

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

None.

Any background context you want to provide?

@Target({ ElementType.METHOD, ElementType.ANNOTATION_TYPE, ElementType.TYPE })
@Retention(RetentionPolicy.RUNTIME)
@ApiImplicitParams({
	@ApiImplicitParam(
			name = "common1",
			value = "common1",
			paramType = "header",
			dataType = "String",
			required = true
	),
	@ApiImplicitParam(
			name = "common2",
			value = "common2",
			paramType = "header",
			dataType = "String",
			required = true
	)
})
public @interface ApiImplicitParamsCommon {
}


@ApiImplicitParamsCommon
@ApiImplicitParams({
	@ApiImplicitParam(
			name = "test1",
			value = "test1",
			paramType = "path",
			dataType = "String",
			required = true
	)
})
public void test1(String test1) {}

@ApiImplicitParamsCommon
@ApiImplicitParams({
	@ApiImplicitParam(
			name = "test2",
			value = "test2",
			paramType = "path",
			dataType = "String",
			required = true
	)
})
public void test2(String test2) {}

What are the relevant issues?

None.

[dilipkrish]

Thanks for the PR. Unfortunately this PR has a compile error @ileler. Also could you perhaps add a test?

[dilipkrish]

This PR supercedes #2331

[kerwin612]

Thanks for the PR. Unfortunately this PR has a compile error @ileler. Also could you perhaps add a test?

get it.However, regarding the test cases, I may have to take a moment to look at it. I am not familiar with the test cases of the project.

[kerwin612]

@dilipkrish The test case has been added.

[dilipkrish]

The build is still failing @ileler It seems like a checkstyle issue

[codecov]

Codecov Report

Merging #3064 into master will increase coverage by <.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #3064      +/-   ##
============================================
+ Coverage     92.94%   92.94%   +<.01%     
- Complexity     3462     3464       +2     
============================================
  Files           375      375              
  Lines          9214     9217       +3     
  Branches        758      760       +2     
============================================
+ Hits           8564     8567       +3     
+ Misses          464      463       -1     
- Partials        186      187       +1

Impacted Files	Coverage Δ	Complexity Δ
...s/operation/OperationImplicitParametersReader.java	86.66% <66.66%> (-13.34%)	7 <4> (+1)
...umentation/spring/web/scanners/ApiModelReader.java	95.46% <0%> (+0.56%)	88% <0%> (+1%)	⬆️

[codecov]

Codecov Report

Merging #3064 into master will increase coverage by <.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #3064      +/-   ##
============================================
+ Coverage     92.94%   92.94%   +<.01%     
- Complexity     3462     3464       +2     
============================================
  Files           375      375              
  Lines          9214     9217       +3     
  Branches        758      760       +2     
============================================
+ Hits           8564     8567       +3     
+ Misses          464      463       -1     
- Partials        186      187       +1

Impacted Files	Coverage Δ	Complexity Δ
...s/operation/OperationImplicitParametersReader.java	86.66% <66.66%> (-13.34%)	7 <4> (+1)
...umentation/spring/web/scanners/ApiModelReader.java	95.46% <0%> (+0.56%)	88% <0%> (+1%)	⬆️

[dilipkrish]

Thank you @ileler!