Suggest assertCount() over assertSame(n, count(...))
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #198 de @alies-dev — fusionado
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Read the plugin’s existing checks for assertSame(null|true|false, ...) and the referenced AssertSameWithCountRule in phpstan-phpunit. The issue proposes adding detection in afterMethodCallAnalysis for assertSame/assertEquals with count() or Countable::count() as the second argument, using the shared helper. Add tests for the new issue and confirm they cover both count forms.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
$this->assertSame(3, count($lines)) is clearer as $this->assertCount(3, $lines), and the failure message shows the collection.
phpstan-phpunit reports it (AssertSameWithCountRule).
Idea: in afterMethodCallAnalysis, detect assertSame/assertEquals whose second argument is a count() call (or ->count() on a Countable) and emit a plugin issue. It shares the helper with the assertSame(null|true|false, ...) checks. Roughly 55 lines with tests.
Idea borrowed from phpstan/phpstan-phpunit.
- Lenguaje dominante
- Gherkin
- Estrellas
- 79
- Forks
- 38
- Merge medio
- 3 h 54 min
- PR fusionados (30 d)
- 68
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de psalm/psalm-plugin-phpunit
-
InvalidArgument for `new ArrayIterator(...)` datasets passed to invariant generic parametersAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
psalm/psalm-plugin-phpunit#274 ·
Los mantenedores suelen responder en 1 día
Todos los issues de psalm/psalm-plugin-phpunit
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
radius-project/radius#13743 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Automattic/static-site-importer#2869 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
btclib-org/btclib#3047 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
freelawproject/courtlistener#8406 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
agentic-community/mcp-gateway-registry#2017 ·
Los mantenedores suelen responder en 1 día