Enhancement in restful controller stub about vapor-maker-commands HOT 4 OPEN

@hsharghi I pushed a commit regarding this feature, feel free to suggest any change via pull request as before 🙂

Still investigating the tests topic, not easy for me since I'm new to both Swift and Vapor 😅

from vapor-maker-commands.

That would be enough for now, sure 👍

But I don't quite like that kind of controller, hiding routes inside the controllers is a pattern that prevents you to see your routes well organized in a single file.

I would like to have a flag for this, like vapor run make:controller PlayersController --boot and use a new BootController.stub.

What do you think about that?

from vapor-maker-commands.

I like the idea of having a flag for this kind of controller.
But I would like having a testable generated code for all kinds of controllers too.

I don't think there is a single preferred approach for defining routes in Vapor as it's for Laravel.

I think creating single routes file for each controller will bloat the project structure, What do you think about adding routes to routes.swift file?

from vapor-maker-commands.

The command can drop a message on how to add this bootable controller into the routes.swift file, I don't think the command should edit that file, it's beyond the scope of the project and also the user would like to order the routes or split them into separate files as they want.

I would like to have tests too! But first I need to figure out a way for my tests to detect the resources shared via Packages.swift. I tried adding the resources: [ .copy("Resources")] key to the testTarget but no luck 😅

from vapor-maker-commands.