Chore/configure blueprint

[IhGori]

O que foi feito?

Implementação do comando principal cuids:generate, responsável por orquestrar a geração de módulos CUIDS (MongoDB) utilizando o Laravel Blueprint.

Alterações principais

Obs: O comando utilizou como base o Cuids Generator (Backend) que já estava em funcionamento no Keldor

Dependências adicionadas

• Laravel/Prompts
• laravel-shift/blueprint
• jasonmccreary/laravel-test-assertions

Testando

Adicione o repositório em seu composer.json

"repositories": [
	{
		"name": "sysvale/cuids-generator",
		"type": "git",
		"url": "https://github.com/Sysvale/cuids-generator"
	}
]

Para executar uma branch específica, execute o seguinte comando:

vendor/bin/sail composer require sysvale/cuids-generator:dev-chore/configure-blueprint -W

Caso o projeta ja tenha configurado o cuids:generator antigo, comentar ou alterar o nome do comando para não chocar com o do pacote.

Execute o comando:

php artisan cuids:generate

[uesley]

eu esperava nao depender mais dos comentários no arquivo de rotas. Acredito que seja pra colocar no lugar certo, mas acho melhor manter limpo e ter o ínfimo trabalhinho manual de mudar a linha de lugar.

Só minha opinião, se outros discordarem posso me adequar tranquilamente à opinião divergente.

[uesley]

ao definir os nomes dos atributos do model, seria interessante usar sempre lower_snake_case, que é como esperamos os dados na base de dados, mesmo que o dev (vulgo, usuário) escreva em quAlquER-CasE poderiamos transformar para o esperado

[uesley]

vi que o wrap que ta sendo colocado por padrão no resource de collections é null, deveria ser data

[uesley]

o trait softDeletes usado nos models mongo deveria ser: MongoDB\Laravel\Eloquent\SoftDeletes; parece que tá usando o do eloquent, pode ser que dê problema

[uesley]

ele tá criando migrations, deveria? acredito que não é um problemão, posso apagar toda vez que gerar, mas se nem criar já é uma coisa a menos. 😬

[uesley]

os testes gerados estão falhando, como as rotas são definidas no espaço protegido pelo auth, nos testes deveria usar o actingAs para usar um acesso logado. Os testes que esperam códigos de sucesso falham porque recebem 401 Unauthenticated

[uesley]

sugiro, que, se possível, seja trocado nos testes o uso de assertStatus(###) para os respectivos assertOk, assertCreated, assertNoContent, etc...
Mas se não for possivel ou for dificil não é algo que impeça o merge

[uesley]

os testes de created deveriam olhar a base para assegurar que o registro foi criado, apenas a resposta do controller não garante o funcionamento, um método de controller vazio, retorna 200, sem fazer absolutamente nada

[IhGori]

o trait softDeletes usado nos models mongo deveria ser: MongoDB\Laravel\Eloquent\SoftDeletes; parece que tá usando o do eloquent, pode ser que dê problema

Fiquei nessa dúvida, primeiramente havia colocado ele, mas ai apareceu que estava depreciado e recomendando usar o do eloquent