swiftlang/swift

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

Open

#47.936 geöffnet am 3. Juli 2017

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Swift (10.719 Forks)batch import
buggood first issuestandard library

Repository-Metriken

Stars
 (69.989 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 7T 6h) (556 gemergte PRs in 30 T)

Beschreibung
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.

Contributor Guide

Research-Richtung
Untersuche den aktuellen Stand doppelter Testnamen in der Swift Testsuite. Das Problem verweist auf zwei Tests in StringViews.swift (alter Commit), beginne also mit der Suche nach ähnlich benannten Tests in der aktuellen validation test/stdlib/StringViews.swift. Grep außerdem im gesamten Testverzeichnis nach doppelten Testfunktionsnamen, die über StdlibUnittest registriert sind. Sieh dir den StdlibUnittest Quellcode (wahrscheinlich in swift/stdlib/private/StdlibUnittest) an, um zu verstehen, wie Tests registriert werden und wo ein Hard Fehler für doppelte Namen hinzugefügt werden kann. Identifiziere alle Duplikate, entferne ungültige und benenne diejenigen um, die umbenannt werden müssen. Prüfe die Kommentare zum Issue auf etwaige aktuelle Aktivitäten oder Anleitungen.
Tech Stack
swift
Domain
testing
Issue Type
Fehler
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.
1-2 Tage
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
Swift
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75