KREST-11230 Fix tests to expect media-type application/json for errors

confluentinc / kafka-rest

Confluent REST Proxy for Kafka

https://docs.confluent.io/current/kafka-rest/docs/index.html

Other

22 stars 641 forks source link

KREST-11230 Fix tests to expect media-type application/json for errors #1189

Closed msn-tldr closed 11 months ago

msn-tldr commented 11 months ago

V2 APIs returned errors in the esoteric MediaTypes like application/vnd.kafka.binary.v2+json. Errors should just be application/json which this PR fixed(Applicable to both v2/v3).

This PR updates V2 tests to expect application/json for errors.

cla-assistant[bot] commented 11 months ago

All committers have signed the CLA.

cla-assistant[bot] commented 11 months ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

msn-tldr commented 11 months ago

Jenkins Ci failing due to unrelated reasons 😞

Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check (validate) on project ce-kafka-rest-parent: Failed during checkstyle configuration: cannot initialize module TreeWalker - cannot initialize module ImportControl - illegal value 'checkstyle/import_control.xml' for property 'file': com.puppycrawl.tools.checkstyle.api.CheckstyleException: Unable to find: checkstyle/import_control.xml -> [Help 1]