apache/beam

WindowFnTestUtils#runWindowFn should be removed

Offen

#18.458 geöffnet am 03.06.2022

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

Repository-Metriken

Stars
 (7.313 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

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