Adicionar etapa extra no GitHub Action de PRs para correr testes

As-Raparigas-do-Codigo / ardc-website

Novo website d'As Raparigas do Código

https://raparigasdocodigo.pt

MIT License

4 stars 8 forks source link

Adicionar etapa extra no GitHub Action de PRs para correr testes #195

Closed tmanso9 closed 2 months ago

tmanso9 commented 3 months ago

Descrição

Adicionado script para correr o jest em modo CI
Adicionada etapa test ao GitHub Action de PR e push para o dev

Falta ainda:

[x] Confirmar que aparece o check (apenas possível depois de criar PR)
[x] Confirmar que o check aparece a verde sem erros nos testes
[x] Testar se o check aparece a vermelho com testes falhados - vou verificar num próximo commit

Fixes #192

Como testar esta modificação?

Confirmar que aparece um check chamado Build website / test no CI/CD.

Checklist:

[x] O meu PR segue as regras de estilo de código deste projeto
[x] Eu fiz uma autoavaliação (review) do meu próprio código ou materiais

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
ardc-website	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 28, 2024 6:23pm

tmanso9 commented 3 months ago

Se os testes falharem, este é o output.

tmanso9 commented 3 months ago

@isabelcosta sugiro também adicionar o jest ao pre-commit hook, para garantir que alterações aos componentes não causam regressões de funcionalidades. Não o fiz porque achei que não se inseria no âmbito deste PR, mas é muito fácil de implementar também.

isabelcosta commented 3 months ago

@isabelcosta sugiro também adicionar o jest ao pre-commit hook, para garantir que alterações aos componentes não causam regressões de funcionalidades. Não o fiz porque achei que não se inseria no âmbito deste PR, mas é muito fácil de implementar também.

@tmanso9 parece me uma boa sugestão. Acho que podes adicionar isso noutra PR, para que cada PR tenha o seu propósito. Parece te bem? Eu penso em mergir esta PR em breve.

tmanso9 commented 3 months ago

@isabelcosta sugiro também adicionar o jest ao pre-commit hook, para garantir que alterações aos componentes não causam regressões de funcionalidades. Não o fiz porque achei que não se inseria no âmbito deste PR, mas é muito fácil de implementar também.

@tmanso9 parece me uma boa sugestão. Acho que podes adicionar isso noutra PR, para que cada PR tenha o seu propósito. Parece te bem? Eu penso em mergir esta PR em breve.

Ok! Não sei como costumam fazer as coisas aqui no repo, espero que seja criado um issue relacionado com isto ou posso só fazer directamente outra PR?

isabelcosta commented 3 months ago

Ok! Não sei como costumam fazer as coisas aqui no repo, espero que seja criado um issue relacionado com isto ou posso só fazer directamente outra PR?

Acho que neste caso, podes simplesmente criar a PR, não temos regras estritas aqui :) Mas também se quiseres, podes criar uma issue com a explicação da tarefa para te fazer assign e teres uma issue a referenciar. Mas de qualquer forma a PR é bem vinda. @tmanso9