Feature/ci tests

[rodrigondec]

Descrição das Mudanças

Adiciona no ci:

• testes do 'npm'
• lintagem do 'npm'
• coverage

Relacionado a issue #24

Mudanças na API

N/A

Mudanças de comportamento

N/A

Testing Procedure

Rodando o CI nesse PR

PR Checklist

• Possui testes (se não, por favor descreva o motivo de não ter teste)
• Foi feito Rebase para a master no momento que o PR foi aberto
• Modificações estão de acordo com o padrão de código
• Atualizado documentação

PR Checklist do time

• O PR está apontando para a branch correta
• Os testes estão passando (ou as falha não são relacionadas a este PR)

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (dev@cba80a1). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             dev       #25   +/-   ##
=======================================
  Coverage       ?   100.00%           
=======================================
  Files          ?        11           
  Lines          ?        73           
  Branches       ?         5           
=======================================
  Hits           ?        73           
  Misses         ?         0           
  Partials       ?         0           

[pabrrs]

Muito bom @rodrigondec 🎉

Como o branch dev não possui nenhum relatório de coverage para comparar, acho que vale desativar apenas nessa PR o required para o step codecov/project.

[deniojunior]

@rodrigondec Excelente!

Os pontos que o @pabrrs levantou são válidos. Além deles, sugeri somente adicionar uma quebra de linha no final dos arquivos, nada demais

[pictos]

@rodrigondec , essa falha não é relacionada a esse PR?

[rodrigondec]

@rodrigondec , essa falha não é relacionada a esse PR?

Não. É pq o codecov compara a cobertura do PR atual com a cobertura da branch de destino.

Que no caso não existe uma cobertura anterior para a dev. Então ele falha.

[pabrrs]

🚀

[deniojunior]

Boa!