loadsys / loadsys/loadsys_codesniffer
Add coverage for all sniffs identified by testUntested()
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
The test suite now compares the full list of sniffs employed by a coding standard against those in the test sample file //~ annotations and throws an assertion failure if the sample files fail to "cover" all tests.
This acts as a crude kind of "coverage" metric of the sniffs defined in the ruleset, but right now we're only covering about 31 out of 120+ sniffs.
To resolve this, uncomment the markTestIncomplete() in LoadsysStandardTest::testUntested(). Then repeatedly run vendor/bin/phpunit and add new sample files for each remaining sniff name listed. Ideally, there should be two files for each sniff-- one confirming that incorrect style is caught by our ruleset, and another that correct behavior is not caught. The new files should be arranged in the tests/files/must/ and tests/files/must_not/ folders. Read the ## Testing section of the README for more naming details and goals.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza en LoadsysStandardTest::testUntested(), descomenta markTestIncomplete() y ejecuta vendor/bin/phpunit para identificar los nombres de los sniff restantes. Lee la sección Testing del README y añade después archivos de ejemplo en tests/files/must/ y tests/files/must_not/ para los sniff restantes; el trabajo estará terminado cuando esté cubierta la lista completa de reglas y la suite de pruebas se ejecute correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- testing
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 35/100