php / php/php-src

Revise ext/standard/tests/file test suite

Abierto
#16,872 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Category: Tests Extension: standard Feature Status: Needs Triage
Lenguaje dominante
C
Estrellas
40.4k
Forks
8.1k
Merge medio
2 d 13 h
PR fusionados (30 d)
96

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Revisa primero ext/standard/tests/file y los ejemplos relacionados en #16722 y #16854. Haz un inventario de los conflictos, las marcas obscure_filename, el comportamiento de limpieza y las variantes POSIX/Windows; se considera terminado cuando la suite aísla de forma coherente sus archivos o directorios y elimina los archivos creados sin variantes innecesarias.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
php
Área
testing-qa
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.