Revise ext/standard/tests/file test suite
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.2k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
Occasionally, there are still test conflicts, and a couple of tests are marked as obscure_filename conflicting. Furthermore, some of the tests do not necessarily clean up properly after execution (they unlink some created files during test execution only). Also, some tests appear to have POSIX and Windows variants, although that doesn't seem to be necessary. See e.g. #16722 and #16854.
Ideally, all tests should only create and delete files which either have the same base name as the test case, or are placed in a directory with the base name of the test case. Since this may lead to overlong paths, we may need to take some shortcuts there (possibly using shorter test case names).
In any way, this test suite should be cleaned up/improved.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Prüfe zuerst ext/standard/tests/file und die zugehörigen Beispiele in #16722 und #16854. Erfasse die Konflikte, obscure_filename-Markierungen, das Cleanup-Verhalten und die POSIX/Windows-Varianten; abgeschlossen bedeutet, dass die Testsuite ihre Dateien oder Verzeichnisse konsistent isoliert und erstellte Dateien ohne unnötige Varianten entfernt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- testing-qa
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100