Adiciona Gabriela Turquetti aos voluntários

[netlify[bot]]

✔️ Deploy Preview for dadosabertosdefeira ready!

🔨 Explore the source changes: 5165df55448ba8467dd8c76041c7c85d81cb99ee

🔍 Inspect the deploy log: https://app.netlify.com/sites/dadosabertosdefeira/deploys/61739ec1f713bd0007f74883

😎 Browse the preview: https://deploy-preview-91--dadosabertosdefeira.netlify.app

[matheusrocha89]

@Suburbanno Ainda não temos um padrão para abrir PR mas é sempre bom colocar uma descrição do que o código faz e um screenshot caso altere algo na interface.

[Suburbanno]

@Suburbanno Ainda não temos um padrão para abrir PR mas é sempre bom colocar uma descrição do que o código faz e um screenshot caso altere algo na interface.

ok, desculpa

[anapaulagomes]

Não tem o que se desculpar @Suburbanno! :) Estamos aqui pra ajudar! Qualquer dúvida em como fazer as correções é só postar aqui ou no Telegram.

[matheusrocha89]

@Suburbanno Exatamente cara, aqui é todo mundo se ajudando, e você não estava errado, não tinha template ainda 😄 Mas fica tranquilo. E obrigado pela contribuição.

[Suburbanno]

Obrigado pessoal, mais tarde eu vejo a questão da vírgula

[matheusrocha89]

O erro não é a vírgula não, é o snapshot desatualizado dos testes, é só atualizar que funciona 👍

On Tue, 19 Oct 2021 at 20:19, Vinicius @.***> wrote:

Obrigado pessoal, mais tarde eu vejo a questão da vírgula

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/DadosAbertosDeFeira/portal/pull/91#issuecomment-947033935, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4LZXXAR5QLFU4YLPPQ7DLUHXAEDANCNFSM5GICCIZQ .

-- Matheus Cruz Rocha

[Suburbanno]

@matheusrocha89 como eu arrumo o snapshot?

[matheusrocha89]

@matheusrocha89 como eu arrumo o snapshot?

@Suburbanno Bom, TLDR:

Você pode rodar o comando de testes do nosso projeto passando a flag -u (yarn test -u) e isso vai automaticamente rodar todos os testes de projeto e atualizar todos os snapshots necessários com suas respectivas alterações. Depois é só fazer commit dessas alterações.

Essa foi a explicação curta. Um conselho que eu dou é pesquisar sobre testes em react. Nós usamos a biblioteca Jest no nosso projeto, dá uma lida na documentação que vai ser interessante para você compreender mais sobre testes e você vai entender melhor o que são snapshots, asserts, expects, describes e por aí vai, além dos comandos extras e flags que podem ser passadas para rodar os testes.

Uma boa leitura também sobre snapshots é esse artigo aqui pois snapshots usados de maneira indevida podem causar mais problemas do que solução, então fica aqui a dica.

Qualquer outra dúvida é só perguntar aqui, ou no discord ou no telegram que eu vou responder o mais breve possível. Eu moro em Portugal e se eu demorar de responder pode ser por conta do fuso 😉 .

Espero ter ajudado.

[Suburbanno]

@matheusrocha89 rodei aqui e todos os testes passaram... Não tem como commitar nada, pois nada foi alterado durante o teste

[matheusrocha89]

@Suburbanno Cara, algum arquivo de snapshot tem que ter sido alterado para você fazer commit. Olha o log do deploy: https://app.netlify.com/sites/dadosabertosdefeira/deploys/616e36986b76d30007b85539 Foi exatamente o snapshot da alteração nova que foi adicionada 😄 Vou fazer checkout da sua branch e atualizar o snapshot aqui e fazer push

[Suburbanno]

Funcionou!!!!

Obrigado @matheusrocha89