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