Bug: NPE in PropertyConditionPredicate when processing empty schema

[beskrovnykh]

I'm submitting a ... [x] bug report [ ] feature request

What is the current behavior?

PropertyConditionPredicate выбрасывает NPE при попытке доступа к properties пустой Schema во время анализа покрытия. Ошибка возникает потому что SwaggerCoverageV3RestAssured создает Schema без инициализации properties, а PropertyConditionPredicate предполагает что properties всегда инициализированы.

If the current behavior is a bug, please provide steps to reproduce: Использовать фильтр SwaggerCoverageV3RestAssured с REST-assured: RestAssured.given().filter(new SwaggerCoverageV3RestAssured()) Сделать любой запрос с телом Попытаться сгенерировать отчет о покрытии Ошибка:

Exception in thread "main" java.lang.NullPointerException: Cannot invoke "java.util.Map.entrySet()" because the return value of "io.swagger.v3.oas.models.media.Schema.getProperties()" is null at com.github.viclovsky.swagger.coverage.core.predicate.PropertyConditionPredicate.lambda$check$2(PropertyConditionPredicate.java:32)

What is the expected behavior? PropertyConditionPredicate должен корректно обрабатывать null в properties, либо: Обеспечивая создание Schema с инициализированными properties в SwaggerCoverageV3RestAssured Либо добавляя проверку на null в PropertyConditionPredicate What is the motivation / use case for changing the behavior? Текущая реализация ломает анализ покрытия для любого запроса с телом из-за необработанного NPE. Это затрагивает всех пользователей независимо от того, используют они сгенерированные клиенты или нет, так как фильтр всегда создает новую пустую Schema.

Other information В документации указано что "Swagger-coverage работает хорошо с клиентами, сгенерированными из swagger", однако это не имеет отношения к проблеме, так как SwaggerCoverageV3RestAssured создает новую пустую Schema независимо от типа клиента:

// в SwaggerCoverageV3RestAssured
MediaType mediaType = new MediaType();
mediaType.setSchema(new Schema()); // properties == null
// в PropertyConditionPredicate
.flatMap(o -> (Stream<Map.Entry<String, Schema>>) o.getProperties().entrySet().stream()) // NPE здесь

[beskrovnykh]

Проблема решена инициализацией пустого Map для properties при создании Schema в SwaggerCoverageV3RestAssured.

Было:

MediaType mediaType = new MediaType(); mediaType.setSchema(new Schema()); // properties == null

Стало:

MediaType mediaType = new MediaType(); mediaType.setSchema(new Schema().properties(new HashMap<>())); // properties инициализирован

Это предотвращает NPE в PropertyConditionPredicate, так как теперь properties всегда инициализирован, даже если пуст.