Atualizando o pacote do composer do Laravel Sail para 1.31 no Server

[moreirayokoyama]

Mudanças propostas

Atualizando o Laravel Sail para corrigir o problema da versão obsoleta do Nodejs (16) e o erro EBADENGINE durante o sail up.

Para conseguir rodar a aplicação corretamente, a versão do pacote do Laravel Sail 1.0.1 estava tentando subir um container com uma imagem que instalava uma versão obsoleta do NodeJs (o nodejs 16). A consequência disto era um erro do tipo EBADENGINE na hora de subir o servidor, e isto impedia o startup do server.

A alteração da lib Laravel Sail exigiu também a atualização das dependências dela, já que várias dependências estavam com versões incompatíveis com a nova.

Mudanças secundárias

Alguns testes estavam quebrados, e algumas alterações secundárias foram feitas para corrigir os testes de acordo: No commit 664010c, os campos bio e image do modelo User, embora sejam nullable, estão forçando uma string vazia na serialização do Resource, e isto quebrou alguns testes.

Para não mexer na forma como o Resource está sendo serializado, eu mudei o teste para validar o comportamento atual da aplicação. Mas criei uma sugestão de dívida técnica (#45) para lidar com a questão da serialização destes campos.

O commit d9d965b adequa os testes com estes campos.

Outros testes estavam deixando de passar pelos seguintes motivos (causados pelo commit 664010c):

• No resource Article, o campo favorited passou a ser renderizado sempre (os testes faziam o assert de que, em certas situações, ele seria omitido)
• No resource Profile, o campo following passou a ser renderizado sempre (os testes faziam o assert de que, em certas situações, ele seria omitido)

Também o teste de JwtGuard que avaliava o token vindo como um Header, estava considerando uma string começando com Token em vez de Bearer (que é como o Guard avalia).

O commit e658410 adequa todos os testes que estavam falhando pelos motivos acima.

Tipo de mudança

• [x] Correção de bug
• [ ] Nova feature
• [x] Atualização de documentação

Checklist

• [x] Testei minhas mudanças
• [ ] Escrevi pelo menos 1 teste automatizado - N/A

[moreirayokoyama]

@marcelaodev Eu não sei se os testes estavam funcionando antes das minhas alterações, mas alguns deles não estão passando no momento. Mais tarde eu pretendo dar uma olhada com calma nos motivos de não estarem passando (e pretendo configurar o CI, se ninguém pegar esta tarefa).

Se você achar que os testes deveriam passar antes de aceitar o PR, só dá um toque aqui que eu tento correr com isto. Mas se estes testes já estiverem falhando na versão anterior, eu acho que não faz sentido barrar o PR por causa disso, e eu subo outro depois.

De qualquer forma, só me dê as orientações que eu sigo de acordo.

Valeu.

[moreirayokoyama]

Alguns testes já estão passando. Mas ainda tem 5 testes falhando.

[marcelaodev]

Olá tio @moreirayokoyama, obrigado pela contribuição!

Em relação aos testes, o elivandro fez uma série de correções de testes no PR-32. Minha sugestão aqui seria mergear suas modificações para dentro da PR dele antes de enviá-las para a main, já que ambas alterações me parecem críticas para o projeto como um todo. Elivandro também fez uma grande contribução corrigindo testes que começaram a falhar no Laravel 11, mas ainda falta alguns. Hoje não vou ter tempo, mas pretendo fazer esse procedimento amanhã durante a live.

[moreirayokoyama]

Puxa, @marcelaodev . Eu nem vi o PR dele (se duvidar eu devo ter feito coisas que ele já havia feito).

Eu corrigi todos os testes. Vou ver se consigo fazer um merge.