feat: add generate controller cli #39
Conversation
rizerkrof
had a problem deploying
to
github-pages
GitHub Actions
Error
rizerkrof
force-pushed
the
feat/cli-for-controllers
branch
2 times, most recently
from
d8d513a
to
5476700
Compare
EwenQuim
force-pushed
the
feat/cli-for-controllers
branch
from
5476700
to
06b5835
Compare
EwenQuim
force-pushed
the
feat/cli-for-controllers
branch
from
06b5835
to
a4a610b
Compare
cmd/fuego/commands/controller.go
Outdated
| ) | ||
|
|
||
| func ControllerCommand() *cli.Command { |
There was a problem hiding this comment.
On l'appelle de l'extérieur commands.Controller donc on comprend que c'est la commande pour créer des controllers
| func createController(controllerName string) error { | ||
| controllerDir := "./controllers/" | ||
| if _, err := os.Stat(controllerDir); os.IsNotExist(err) { | ||
| err = os.Mkdir(controllerDir, 0755) | ||
| err = os.Mkdir(controllerDir, 0o755) |
There was a problem hiding this comment.
Je mets explicitement que c'est de l'octal
|
|
||
| newContent := strings.ReplaceAll(string(templateContent), "newController", controllerName) | ||
| newContent = strings.ReplaceAll(newContent, "NewController", strings.Title(controllerName)) | ||
| newContent = strings.ReplaceAll(newContent, "NewController", titler.String(controllerName)) |
There was a problem hiding this comment.
askip mon IDE me dit qu'il faut préférer utiliser cases
| func TestCreateController(t *testing.T) { | ||
| err := createController("books") | ||
| require.NoError(t, err) | ||
| } |
There was a problem hiding this comment.
Ca fait toujours plez pour vérifier que rien ne crash :)
There was a problem hiding this comment.
Renommé parce que comme ça on est sûrs de pas faire de bêtises, on a le formattting, les imports etc...
| GetNewController(id string) (NewController, error) | ||
| CreateNewController(NewControllerCreate) (NewController, error) | ||
| GetAllNewController() ([]NewController, error) | ||
| UpdateNewController(id string, input NewControllerUpdate) (NewController, error) | ||
| DeleteNewController(id string) (any, error) |
There was a problem hiding this comment.
J'ai rajouté un corps aux fonctions et les signatures d'interface :
Comme ça, plus qu'à implémenter les interfaces (faire la partie BDD) et on a notre CRUD fini !
| controllerName := cCtx.Args().First() | ||
|
|
||
| if controllerName == "" { | ||
| controllerName = "newController" | ||
| fmt.Println("Note: You can add a controller name as an argument. Example: `fuego controller books`") | ||
| } |
There was a problem hiding this comment.
D'abord on prend la valeur, et seulement si elle n'est pas précisée, on lui assigne la valeur par défaut
| Flags: []cli.Flag{ | ||
| &cli.StringFlag{ | ||
| Name: "output", | ||
| Usage: "output file", | ||
| Aliases: []string{"o"}, | ||
| }, | ||
| }, |
There was a problem hiding this comment.
Ce sera une feature demandée je pense
| @@ -7,6 +7,9 @@ import ( | |||
| ) | |||
|
|
|||
| func TestCreateController(t *testing.T) { | |||
| err := createController("books") | |||
| res, err := createController("books", "/dev/null") | |||
There was a problem hiding this comment.
Et on peut direct utiliser notre feature pour que les tests polluent pas notre workspace :)
command
fuego controller myControllerwill generate./controller/myController.gofile