idvogados / idvogados-api

Repositório para o backend

26 stars 11 forks source link

Feature/ci tests #25

Closed rodrigondec closed 4 years ago

rodrigondec commented 4 years ago

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

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

PR Checklist do time

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

codecov[bot] commented 4 years ago

Codecov Report

:exclamation: 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

pictos commented 4 years ago

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

rodrigondec commented 4 years ago

@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.