coder / coder/internal

Add `gotestleak` to wsproxy pkg and fix all resulting errors

オープン
#90 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
tech-debt
主要言語
言語のデータがありません
スター
3
フォーク
0
PR マージ指標
30日以内にマージされた PR はありません

説明

I tried adding a
```
func TestMain(m *testing.M) {
goleak.VerifyTestMain(m)
}
```
to the `enterprise/coderd/wsproxy` package and there were a ton of failures as a result. We should find some time to figure out where all the leaks are originating from.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

enterprise/coderd/wsproxy パッケージから始め、golek.VerifyTestMain(m) を使用して TestMain を追加したことによる失敗を再現します。報告されたリークをその発生源まで追跡し、パッケージのリークチェックとテストが結果としてエラーを発生させずにパスした時点で作業完了とみなします。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
go
領域
testing-qa
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。