search

authorjapps / zerocode

A community-developed, free, opensource, automated testing framework for microservices APIs, Kafka(Data Streams) and Load testing. Zerocode Open Source enables you to create, change and maintain your automated test scenarios via simple JSON or YAML files. Visit documentation below:
https://zerocode-tdd.tddfy.com
Apache License 2.0
894 stars 395 forks source link

[TECH-DEBT] Move to jackson json provider in JsonPath instead of jsonsmart #648

Closed a1shadows closed 6 months ago

a1shadows commented 6 months ago

We can have uniformity across the repo if we do this and can avoid pulling in the json-smart dependency

was raised during #647

baulea commented 6 months ago

@a1shadows You may assign this issue to me. I'll have a closer look at json array handling :-)

baulea commented 6 months ago

@a1shadows Do you know the reason, why Gson is used in the Kafka part? Couldn't that also be changed to Jackson?

a1shadows commented 6 months ago

shadows Do you know the reason, why Gson is used in the Kafka part? Couldn't that also be changed to Jackson?

Hey @authorjapps any idea on this?

authorjapps commented 6 months ago

Gson is used in the Kafka part?

@a1shadows, I vaguely remember, a collaborator introduced for a quick fix. I think it was introduced for Kafka "headers" (and elsewhere) to allow generic object structure ser/de using Gson easily over Jackson. Yes, Jackson can be used instead of Gson 👍 .

Worth checking anywhere else Gson is not used before completely getting rid of it.