Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| id: string; | ||
| } | ||
|
|
||
| export function Button(props: Id) { |
There was a problem hiding this comment.
Aqui como conversamos na aula, o botão possui muitas responsabilidade o que dificulta até mesmo os testes. O botão ele precisa ser o elemento que inicia as alterações de estado, o gatilho, mas isso não quer dizer que o código que faz essas alterações deveriam estar dentro do componente de botão.
| import Home from 'pages/home/Home'; | ||
| import { Header } from './Header'; | ||
|
|
||
| describe('<Header/>', () => { |
There was a problem hiding this comment.
Este teste em específico não te ajuda muito. Aqui você tá testando se a url do browser muda, mas o que faz a url do browser mudar é o react router, que não é um código que você escreveu, evitamos testar códigos de bibliotecas por que normalmente as bibliotecas possuem testes, a maioria pelo menos.
| (acc += currentValue.price), | ||
| 0 | ||
| ); | ||
|
|
There was a problem hiding this comment.
O único problema desse teste é que você calculou o total aqui usando o reduce, então parece que o teste tá testando o reduce o que não faz muito sentido. Se você usasse a função que calcula o total do carrinho aqui, este teste faria mais sentido.
| @@ -1,127 +1,127 @@ | |||
| @import '../../../styles/var.scss'; | |||
|
|
|||
There was a problem hiding this comment.
poderia ter deletado este arquivo por completo.
| const cart = useCartState(); | ||
| const setCart = useSetCartState(); |
There was a problem hiding this comment.
aqui poderia ser um useState normal, se não houver um motivo muito forte pra separar o estado da função que altera esse estado, acredito que não seja uma boa ideia fazer, por que agora, você tem dois lugares para dar atenção sobre o que um estado significa e como as alterações sobre este estado acontecem.
2 participants
No description provided.