remove `.py` empty file creation after python client generation

[Adi-204]

when we run npm run test inside python websocket client are generated.
In all the clients except client_slack an empty file .py is created which need to be fix.

For ref -

[derberg]

hmmm, I saw this in my PR, it was related to models generation, for files where there are no reach schemas and no model generation available. This if: https://github.com/asyncapi/generator/blob/master/packages/components/src/components/models.js#L76 solved it. Weird you see this behaviour

[Aditya08Vashisht]

@derberg it indeed prevents the generation of model files when there are no schemas. However, it appears that in the Python WebSocket clients (excluding client_slack), an empty .py file is still being created even when no models are generated.

To address this, I propose adding an additional check in the template logic to ensure that a model file is only created if there are schemas to process. This would prevent the creation of empty .py files.

[derberg]

@Aditya08Vashisht feel free to open a PR 🙏🏼

[Aditya08Vashisht]

@Adi-204 and @derberg
Well i have done a lot of work and research in this issue but still

am not facing as such empty .py file things that you are talking about..
Please correct me if am wrong somewhere!

what i did till now is first searching for the particular folders in which i was clearly wrong but then i just simple ran "npm run test" in my particular branch inside the codespace
then the temp dir opened and i simply seen this(seeing only 2 .py files inside it!)

That's the story yet from yesterday till now!

still working and figuring out the things currently!!
But your suggestions are valuable!

[Aditya08Vashisht]

hmmm, I saw this in my PR, it was related to models generation, for files where there are no reach schemas and no model generation available. This if: https://github.com/asyncapi/generator/blob/master/packages/components/src/components/models.js#L76 solved it. Weird you see this behavior

Yes @derberg now i think you are totally right!

someone may have already solved it and made the necessary changes needed before!!
Guide me wherever i am wrong or if directionless anywhere!

[Adi-204]

@Aditya08Vashisht I think you branch is not updated because output should be in folder structure not files directly. Pull latest changes locally.

[Aditya08Vashisht]

@Aditya08Vashisht I think you branch is not updated because output should be in folder structure not files directly. Pull latest changes locally.

okay let me try with the updated branch way
stay tuned for future updates and comments.

[Aditya08Vashisht]

@Aditya08Vashisht I think you branch is not updated because output should be in folder structure not files directly. Pull latest changes locally.

after understanding your concern I've updated and even created the whole thing in a new codespace very freshly then done npm install for all the dependencies to be assured already and then done npm run test
after all i got this and seeing no anytype of empty file (.py)

branch is now up-to-date, and your test run shows no more stray .py placeholders

but still i think to raise a pr
i feel i should now make a comment in each of these files in the last that its working according to the latest version and there are no any stray .py files

like this i am feeling to create a PR
right @derberg ??

[Adi-204]

@Aditya08Vashisht ok I tried again and .py was not created a bit weird not sure what happen initially we will close this issue so you can work on other.

[Adi-204]

@derberg we can close the issue not sure what happened first time when I ran test after updating branch. thanks : )

[Aditya08Vashisht]

@Aditya08Vashisht ok I tried again and .py was created we will close this issue so you can work on other.

okay then no worries wonder if that was a real issue and could have tackle some difficulties.. :))

[Aditya08Vashisht]

why is still open @derberg @Adi-204

[Adi-204]

@Aditya08Vashisht
@derberg will close the issue.

[Aditya08Vashisht]

@Aditya08Vashisht @derberg will close the issue.

@Adi-204 okay sure!