grafana/k6

Move Sobek option parsing to the mapping layer

Open

#5.305 geöffnet am 14. Okt. 2025

Auf GitHub ansehen
 (22 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Go (30.564 Stars) (1.537 Forks)batch import
area: browsergood first issuerefactor

Beschreibung

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
  • Locator options
  • ElementHandleBaseOptions
  • #5687
  • PageReloadOptions
  • PageScreenshotOptions
  • Size.Parse
  • #5302
  • #5560

Example

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

Related

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

Contributor Guide

Tech Stack
go
Domain
backend
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Go programmingk6 codebase structureSobek API
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Examine the existing mapping layer in files like locator mapping.go and browser mapping.go to understand the current pattern. Review the list of options to be moved, noting that some are already completed (e.g., #5687, #5302, #5560). Discuss with the assignee to coordinate work. Ensure each option is moved following the separation of concerns principle, with updated tests.