apache/beam

WindowFnTestUtils#runWindowFn should be removed

Open

#18.458 geöffnet am 3. Juni 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (4.097 Forks)batch import
P3bugcoregood first issuejava

Repository-Metriken

Stars
 (7.313 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T) (243 gemergte PRs in 30 T)

Beschreibung

Its use makes test methods (like most SlidingWIndow tests) very difficult to read, as there's no obvious behavior that is being tested for.

Imported from Jira BEAM-2525. Original Jira may contain additional context. Reported by: tgroh.

Contributor Guide

Research-Richtung
Entfernen Sie die Methode WindowFnTestUtils.runWindowFn und ersetzen Sie alle ihre Verwendungen durch direkte Test Assertions, die das erwartete Fensterverhalten überprüfen. Suchen Sie alle Aufrufer im Codebestand und aktualisieren Sie diese entsprechend.
Tech Stack
java
Domain
data
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Java
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70