golang/go

x/tools/gopls: source.addTest: emit t.Context() when module is Go 1.24+

Chiusa

#79.647 aperta il 25 mag 2026

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)Go (19.008 fork)batch import
Toolsgoplshelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Proposal

The source.addTest code action ("Add test for FUNC") passes context.Background() whenever the function under test takes context.Context as its first parameter:

got := a.Foo(context.Background(), tt.in)

When the module's go.mod declares Go 1.24 or later, the generated scaffolding should use t.Context() instead:

got := a.Foo(t.Context(), tt.in)

Implementation

I have a draft CL ready. It gates on versions.AtLeast(fileVersion, versions.Go1_24); older modules keep the current context.Background() behavior.

Guida contributor

Direzione di ricerca
Locate the source.addTest code action implementation in the gopls source (likely in internal/lsp/source or similar). Examine how it currently generates test scaffolding with context.Background(). Understand the Go version detection mechanism using versions.AtLeast and the Go1 24 constant. Review the draft CL (if linked) or follow the discussion in the issue to understand the desired change. Ensure familiarity with the testing.T.Context() method introduced in Go 1.24.
Tech stack
go
Dominio
developer experiencetesting
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go programming languagegopls internalsGo module versioning
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70