Add @ApplicationScoped to generated api to allow mocking api for testing

[miguelchico]

PR to fix issue #86.
Adding @ApplicationScoped annotation to generated API will allow us to mock it in unit test according to official Quarkus guidelines https://quarkus.io/blog/mocking/#using-injectmock

[ricardozanini]

@miguelchico were you able to test this approach in your application? I mean, with this PR you're able to mock the generated stub?

Another way to achieve mocking is to use Wiremock and change the endpoint to the mocked server.

I'm just wondering if this annotation won't bring any side effects to the user's project.

[miguelchico]

Hi @ricardozanini
I tested my application with the extra annotation and I couldn't find any side effects.
The idea here is to be able to mock that client in a more pure unit test rather than an integration test with a real webserver (mocked with wiremock)

[ricardozanini]

@miguelchico I'll prepare a new release since I understand this is quite important for you?

[miguelchico]

@ricardozanini
It would be great if you do so!
I'm also working on a PR for #85 but I don't think I'll be able to get it done this week I'm afraid. I'll keep you posted!

[ricardozanini]

@all-contributors please add @miguelchico for code

[allcontributors]

@ricardozanini

I've put up a pull request to add @miguelchico! 🎉

[ricardozanini]

@miguelchico do you think we can wait for #85 then?

[miguelchico]

@ricardozanini give me some time in order to do just one new release with both if you are ok, I'll need both anyway so it might be easier for both

[ricardozanini]

Yes, that's what I was thinking. Let's include both in the new release. Take your time, please. Thanks for the contribution!