coder / coder/internal

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

Offen
#90 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
tech-debt
Vorherrschende Sprache
Keine Sprachdaten
Sterne
3
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start in the enterprise/coderd/wsproxy package and reproduce the failures from adding TestMain with golek.VerifyTestMain(m). Trace the reported leaks to their sources and consider the work done when the package's leak checks and tests pass without resulting errors.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
testing-qa
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.