grafana/k6

Move Sobek option parsing to the mapping layer

Open

#5305 aperta il 14 ott 2025

Vedi su GitHub
 (28 commenti) (0 reazioni) (1 assegnatario)Go (1537 fork)batch import
area: browsergood first issuerefactor

Metriche repository

Star
 (30.564 star)
Metriche merge PR
 (Merge medio 11g 15h) (47 PR mergiate in 30 g)

Descrizione

What

  • Move Sobek option parsing to the mapping layer.

Why

  • Separation of concerns.
  • Option parsing is only required by the mapping layer.
  • To simplify and make the code and tests easy to reason about.

Current options to move

  • Frame options
  • ElementHandleBaseOptions
  • PageReloadOptions
  • PageScreenshotOptions
  • Locator options
  • Size.Parse
  • #5687
  • #5302
  • #5560

Example

  • In this code, the parsing happens in the mapping layer.

Related

  • This issue is a continuation of the epic issue #4219.

Guida contributor

Direzione di ricerca
Esamina il parsing delle opzioni esistente nel livello di mappatura (ad es. locator mapping.go) e identifica dove avviene il parsing delle opzioni di Sobek. Quindi rifattorizza per spostare quelle funzioni di parsing nel livello di mappatura, assicurandoti che i test siano aggiornati di conseguenza.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50