envoyproxy/envoy

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

Open

#4,531 创建于 2018年9月25日

在 GitHub 查看
 (0 评论) (0 反应) (0 负责人)C++ (5,373 fork)batch import
help wantedtech debt

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

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.

贡献者指南

研究方向
检查 test/config/utility.cc 中的测试配置，找出那些仅为了通过启动而指定管理地址的测试。在验证 Admin 控制台类可以在没有 HTTP 监听器的情况下实例化后，移除这些测试中的管理地址。
技术栈
cpp
领域
backend
议题类型
测试
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitC++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70