Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix - Sonar Test Coverage Report #512

Merged
merged 14 commits into from
Aug 12, 2021
Merged

Conversation

LuanEdCosta
Copy link
Contributor

@LuanEdCosta LuanEdCosta commented Aug 5, 2021

I found the exact same problem and one possible solution HERE

This PR includes:

  • Add test-report.xml to .gitignore
  • Improve sonar-project.properties comments
  • Update package.json test:ci script to fix jest bug - StackOverflow Answer

@github-actions
Copy link

github-actions bot commented Aug 5, 2021

Dicas para revisão de código

Commits

  • Título (1a linha do commit): apresentar resumo do que foi alterado/adicionado/removido.
    ex: adiciona action que salva parametros no backend; exibe rótulo no componente de selecao de dataset;
  • Descrição (outras linhas): dar mais detalhes de cada alteração:
    • motivos das alterações
      ex: havia um bug que causava...; nova funcionalidade que faz isso...; código foi movido para...;
    • bibliotecas adicionadas e versões (package.json)
      ex: atualiza para antd v4.6;
    • testes unitários criados/alterados
      ex: adiciona testes para a action fetchProjectSuccess;
  • Mensagens auto-explicativas! Quem revisa o código deve entender o que foi feito (e porque foi feito) sem perguntar para quem fez o commit.
  • Não devem ter conflitos. Solicitar que sejam resolvidas as ocorrências de "This branch has conflicts that must be resolved".

SonarCloud Quality Gate

  • Coverage > 80.0%, e sempre que possível = 100%
  • 0 Bugs, 0 Code Smells, 0 Vulnerabilities

Build Github actions COM SUCESSO

ReactJS

  • Usar Node 12 ou 14.
  • Remover console.log.
  • Não deixar código-fonte comentado.
  • Pasta de módulos devem ser escritas em lower case
  • Pasta de componentes devem ser escritas em camel case
  • Nunca utilizar spread em retorno de reducer sem alteração
  • Sempre utilizar hooks ao invés de HOCs
  • Itens em uma lista devem ser renderizados com uma chave única (preferir uuid)
  • Comentar apenas códigos que não estão muito claros
  • Não utilizar comentários para blocos
  • Action types devem conter @

Redux

As instruções a seguir foram retiradas do Redux Style Guide.

@fberanizo fberanizo force-pushed the fix/test-coverage-reports branch 4 times, most recently from 742e0ef to 8553e6c Compare August 12, 2021 19:22
@fberanizo fberanizo force-pushed the fix/test-coverage-reports branch 7 times, most recently from aa700d6 to b6e2504 Compare August 12, 2021 22:48
@fberanizo fberanizo force-pushed the fix/test-coverage-reports branch from b6e2504 to 41da51a Compare August 12, 2021 23:05
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@fberanizo fberanizo merged commit e431b9d into master Aug 12, 2021
@fberanizo fberanizo deleted the fix/test-coverage-reports branch August 13, 2021 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants