swiftlang/swift

[SR-4595] Tests generate lots of warnings

Open

#47.172 geöffnet am 14. Apr. 2017

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Swift (10.719 Forks)batch import
buggood first issue

Repository-Metriken

Stars
 (69.989 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T 17h) (510 gemergte PRs in 30 T)

Beschreibung
Previous ID SR-4595
Radar None
Original Reporter @dabrahams
Type Bug
Votes 0
Component/s Project Infrastructure
Labels Bug, StarterBug
Assignee None
Priority Medium

md5: 3bb31b4381d569556f0d983141a18d9c

Issue Description:

Because tests pass even when they produce warnings, most people don't see this, but when a test fails, it really matters that the important information is not buried under a bunch of irrelevancies. We need both:

  1. To clean up all the existing warnings in tests (obviously, other than those that are part of what we're testing for, and while keeping the tests running)

  2. To put a system in place that will cause us to keep the tests clean of warnings.

Contributor Guide

Research-Richtung
Das Issue fordert die Bereinigung aller vorhandenen Warnungen in Testdateien und die Implementierung eines Systems zur Verhinderung zukünftiger Warnungen. Untersuchen Sie die aktuelle Testsuite im Swift Repository, mit Schwerpunkt auf Dateien wie 'swift/test/' und verwandten Build Skripten. Suchen Sie nach vorhandenen CI Konfigurationen oder Testrichtlinien. Erwägen Sie, mit einer kleinen Teilmenge von Warnungen, z. B. Veraltungswarnungen, zu beginnen, und schlagen Sie einen Plan für eine systematische Bereinigung vor.
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.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
85