testng-team/testng

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

Ouverte

#2 797 ouverte le 29 août 2022

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)Java (1 025 forks)github user discovery
good first issuelogging

Métriques du dépôt

Stars
 (2 059 étoiles)
Métriques de merge PR
 (Merge moyen 17h 43m) (7 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Identifiez le fichier de test mentionné dans le problème, localisez le cas de test avec une instruction return et sans assertion, puis supprimez le ou refactorez le pour inclure des assertions appropriées.
Stack technique
java
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaTestNG
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70