envoyproxy/envoy

Reduce boilerplate and enforce InSequence in RouterTest

Open

#4 916 ouverte le 30 oct. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
help 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

The tests in RouterTest (test/common/router/router_test.cc) are very verbose and hard to work with. We should 1) reduce the amount of boilerplate necessary and 2) use InSequence in every test.

Guide contributeur

Direction de recherche
Familiarisez vous avec la configuration existante de RouterTest, étudiez comment InSequence est utilisé dans d'autres tests et identifiez les modèles de code répétitif qui peuvent être abstraits ou supprimés.
Stack technique
cpp
Domaine
testing
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Google TestBazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65