envoyproxy/envoy

Clean up tests that start an admin HTTP server just to get by

Ouverte

#4 531 ouverte le 25 sept. 2018

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
help wantedtech debt

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

After #4486 is submitted, it will be possible to create an instance of the Admin admin console class without automatically starting an HTTP listener. Tests that specify an admin address in a bootstrap config - and so start an HTTP listener - just to get through startup correctly can now have that listening address removed. At least one example (I think?) is in test/config/utility.cc, ConfigHelper constructor.

Guide contributeur

Direction de recherche
Examinez la configuration de test dans test/config/utility.cc et identifiez les tests qui spécifient une adresse d'administration uniquement pour passer le démarrage. Après avoir vérifié que la classe Admin console peut être instanciée sans écouteur HTTP, supprimez l'adresse d'administration de ces tests.
Stack technique
cpp
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70