testng-team/testng

Test Smell: it is not a good practice to write a return statement in the test case

Offen

#2.797 geöffnet am 29.08.2022

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (1.025 Forks)github user discovery
good first issuelogging

Repository-Metriken

Stars
 (2.059 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 17h 43m) (7 gemergte PRs in 30 T)

Beschreibung

Hi! I notice that you write a test case with a return statement. For example,

It is not a good idea to write the return statement in your test case based on the definition of the test case. So I would like to know if is it really a useful test case? And no assertion is in this test case. Maybe I think you should remove this test case from your test code. Thanks.

Contributor Guide

Research-Richtung
Identifizieren Sie die in der Ausgabe erwähnte Testdatei, suchen Sie den Testfall mit einer return Anweisung und ohne Assertions, und entfernen oder refaktorisieren Sie ihn, um geeignete Assertions einzufügen.
Tech Stack
java
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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
JavaTestNG
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70