envoyproxy/envoy

beginner: add coverage

Open

#1 963 ouverte le 28 oct. 2017

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
beginnerhelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Going to leave open a persistent beginner ticket for adding coverage: https://s3.amazonaws.com/lyft-envoy/coverage/report-master/coverage.html

There are always a few missing things here and there especially regarding random error handling. If you are a beginner this is a great way to learn and do something valuable.

Rough steps:

  • Look at the coverage report and find something (often a single line) that looks like it should be covered.
  • Find the relevant unit test that should cover it (often in the test tree with the same file name)
  • Write the test
  • Confirm in PR coverage build that line is covered. (Coverage report is posted as a build artifact).

Guide contributeur