quarkus-openapi-generator icon indicating copy to clipboard operation
quarkus-openapi-generator copied to clipboard

Add moqu and wiremock module

Open mcruzdev opened this issue 1 year ago • 10 comments

Why

This pull request aims to add a new module where we get a OpenAPI Specification and transform it to a Wiremock (this pull request, we can have more after) stubs.

This was created from this Discussion: https://github.com/quarkiverse/quarkus-openapi-generator/discussions/575

DevUI

https://github.com/user-attachments/assets/63e4ca32-e481-4013-90a3-5483f9b28696

mcruzdev avatar Oct 23 '24 17:10 mcruzdev

This is interesting but i don't see any Doc updates explaining what this exactly does?

melloware avatar Nov 19 '24 18:11 melloware

Hi @melloware, yeah I need to add documentation ;/ I will try to do it on this week.

mcruzdev avatar Nov 19 '24 21:11 mcruzdev

Could you add your review too? @ricardozanini @hbelmiro @rmanibus @fjtirado

There are some things to do yet, but we can do in another step.

  • Generate wiremock-mappings.json on output directory.
  • Add a DevService for hosting the Wiremock for test/dev purpose (maybe to integrate with quarkisver-wiremock)

mcruzdev avatar Nov 20 '24 16:11 mcruzdev

cc: @chberger

mcruzdev avatar Nov 20 '24 16:11 mcruzdev

Do we have an issue explaining the motivations of this PR?

ricardozanini avatar Nov 21 '24 15:11 ricardozanini

Do we have an issue explaining the motivations of this PR?

This is related to this discussion https://github.com/quarkiverse/quarkus-openapi-generator/discussions/575

mcruzdev avatar Nov 21 '24 20:11 mcruzdev

@ricardozanini @hbelmiro This is being labeled as Stale.

github-actions[bot] avatar Jan 21 '25 12:01 github-actions[bot]

@mcruzdev I'd like to include this extension, but it looks like you are still working on it or...?

ricardozanini avatar Jan 21 '25 14:01 ricardozanini

@mcruzdev I'd like to include this extension, but it looks like you are still working on it or...?

This is the first version, it is available for reviewing. I need to apply the @fjtirado suggestions

mcruzdev avatar Jan 21 '25 14:01 mcruzdev

Hi @ricardozanini @hbelmiro @fjtirado suggestions applied, could you take a review? I would like to add strength to this this year, I've been thinking about this for a while and I always put it off.

mcruzdev avatar Feb 18 '25 12:02 mcruzdev

Do we have enough to merge and move on?

ricardozanini avatar Mar 11 '25 14:03 ricardozanini

@fjtirado can you please give a second look? I guess we are waiting for your vote :)

ricardozanini avatar Mar 11 '25 14:03 ricardozanini