Button

[kelvincharlesdev]

• Add button component with style and tests
• Add test model
• Add fix for test bug in jest and tenting library suggested in the link : toHaveStyles always report hover style testing-library/jest-dom#594
• Set home to receive Button component
• Adjusts theme with new colors and sizes
• test:covarage for test covarage

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
space-tourism-multi-page	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 17, 2024 11:33pm

[lucasmarques73]

Primeiro ponto sobre esse PR. Os arquivos dentro de coverage poderiam ser ignorados pelo git. Como eles são gerados/atualizados sempre que rodar os testes e não vamos deixar eles visível em algum lugar.. Geralmente é só pra vermos no nosso PC.. não precisamos versionar esses arquivos.

Sobre os testes, vc colocou um teste com as 3 variações.. e está bom, vai garantir que as variações funcione.. Mas geralmente é criado um teste para cada variação. Por exemplo must have primary variation , must have secondary variation, etc.
Isso pq, digamos que vc troque a segunda variação, vai quebrar esse teste que testa tudo.. e não somente o teste da alteração que vc está fazendo.

E por último, vc que vc colocou o nome das cores nos temas. gray100 e o hexadecimal e essa é a terceira variação.. Mas digamos que agora mudou pra azul..
Vc teria que ir em todo lugar e trocar o gray100 para azul.
Certo que hj é fácil por causa da IDE.. mas se a cor chama terceira e não gray100, vc troca em um arquivo só e já era... não corre o risco de esquecer em nenhum lugar.

---

Mano, está tudo muito bom, realmente bom.
Eu não faria metade disso, vc está demonstrando uma grande preocupação com os detalhes, e isso faz muita diferença.
Meus Parabéns 🥇