php / php/php-src

Revise ext/standard/tests/file test suite

Ouverte
#16,872 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Category: Tests Extension: standard Feature Status: Needs Triage
Langage dominant
C
Étoiles
40.4k
Forks
8.1k
Merge moyen
2 j 13 h
PR mergées (30 j)
96

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Examinez d’abord ext/standard/tests/file et les exemples associés dans #16722 et #16854. Dressez l’inventaire des conflits, des marquages obscure_filename, du comportement de nettoyage et des variantes POSIX/Windows ; le travail est considéré comme terminé lorsque la suite isole systématiquement ses fichiers ou répertoires et supprime les fichiers créés sans variantes inutiles.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
testing-qa
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.