grafana/k6

Move Sobek option parsing to the mapping layer

Open

#5.305 geöffnet am 14. Okt. 2025

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

Repository-Metriken

Stars
 (30.564 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11T 15h) (47 gemergte PRs in 30 T)

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
  • 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.

Contributor Guide

Research-Richtung
Untersuche das vorhandene Parsen von Optionen in der Mapping Ebene (z. B. locator mapping.go) und identifiziere, wo das Parsen von Sobek Optionen stattfindet. Refaktoriere dann, um diese Parsing Funktionen in die Mapping Ebene zu verschieben, und stelle sicher, dass die Tests entsprechend aktualisiert werden.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50