ResolverFully doesn't resolve arrays recursively

[exlevan]

The logic in ResolverFully seems to be flawed:

swagger-parser/modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/util/ResolverFully.java

Lines 301 to 305 in a4a0667

	if(arrayModel.getItems().get$ref() != null) {
	arrayModel.setItems(resolveSchema(arrayModel.getItems()));
	} else {
	arrayModel.setItems(arrayModel.getItems());
	}

If array's items schema is a reference, it's resolved. If it isn't, then nothing is done (we're getting items from the array, then setting them back. Hmm…)
But the case when array's items schema is an array, object or a composed schema, which contain references, is unhandled; those references are never resolved.
I propose removing this check and replacing it with arrayModel.setItems(resolveSchema(arrayModel.getItems())), which will take care of resolving the references.

[gracekarina]

Hi, @exlevan can you please send us a PR with a test case where you show us how resolverFully it's failing now, and how it can be improved? Thanks.