swiftlang/swift

[SR-5362] StdlibUnittest allows redeclaration of the same-named test; ignores all but last

Open

#47 936 ouverte le 3 juil. 2017

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Swift (10 719 forks)batch import
buggood first issuestandard library

Métriques du dépôt

Stars
 (69 989 stars)
Métriques de merge PR
 (Merge moyen 7j 6h) (556 PRs mergées en 30 j)

Description
Previous ID SR-5362
Radar rdar://problem/33107335
Original Reporter @dabrahams
Type Bug
Status In Progress
Resolution
Votes 0
Component/s Standard Library
Labels Bug, StarterBug
Assignee None
Priority Medium

md5: fc8e733c68f3cbbcf633f37b9a56c975

Issue Description:

For example, these two tests have the same name. The first one wouldn't pass, but it never gets run because of the second one.

This affects the reliability and maintainability of our testing. Someone needs to make StdlibUnittest cause a hard error in these cases, identify all the duplicated tests, strip out the bogus ones, and rename any others.

Note: the particular bogus tests referenced here will probably be gone from master by the time this bug is addressed. The links point to an older version of the file.

Guide contributeur

Direction de recherche
Étudiez l'état actuel des noms de tests en double dans la suite de tests Swift. Le problème fait référence à deux tests dans StringViews.swift (ancien commit), commencez donc par rechercher des tests similaires dans le fichier validation test/stdlib/StringViews.swift actuel. De plus, effectuez un grep dans l'ensemble du répertoire de tests pour les noms de fonctions de test en double enregistrés via StdlibUnittest. Examinez le code source de StdlibUnittest (probablement dans swift/stdlib/private/StdlibUnittest) pour comprendre comment les tests sont enregistrés et où ajouter une erreur stricte pour les noms en double. Identifiez tous les doublons, supprimez ceux qui sont erronés et renommez ceux qui doivent l'être. Considérez la vérification des commentaires sur le problème pour toute activité récente ou orientation.
Stack technique
swift
Domaine
testing
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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
Swift
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75