Detect risky PHPUnit test
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
Piste de recherche
Start from the issue's try/catch example and review the repository's existing PHPUnit-related rules and tests. Define whether PHPStan should report tests that can finish successfully after the expected Throwable is caught without an explicit failure, then add coverage for the agreed behavior.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Feature request
In the following scenario:
function test_exception_state(): void {
$myService = new MyService();
try {
$myService->doSomethingWrong();
// missing: $this->fail();
} catch (MyException $exception) {
$this->assertSame('Custom message', $exception->getMessage());
$this->assertSame('Another property', $exception->smthNotTypicalOfException);
}
}
Essentially, a case where we want to do an explicit try/catch because expectExceptionObject() doesn't cut it (want to catch a Throwable instead of test some state of the exception besides message/code), it is easy to forget a fail() statement and leading to a risky test.
Maybe it leading as a risky is fine... But maybe it's a kinda of issue that PHPStan could detect & report. Just an idea.
Did PHPStan help you today? Did it make you happy in any way?
Today it did not help, but I didn't run it either so cannot blame it :) I like PHPStan otherwise!
- Langage dominant
- PHP
- Étoiles
- 545
- Forks
- 58
- Merge moyen
- 10 h 48 min
- PR mergées (30 j)
- 10
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de phpstan/phpstan-phpunit
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
phpstan/phpstan-phpunit#338 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
phpstan/phpstan-phpunit#321 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
phpstan/phpstan-phpunit#309 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
phpstan/phpstan-phpunit#299 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
phpstan/phpstan-phpunit#292 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de phpstan/phpstan-phpunit
Issues similaires
-
Bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 1 jour
-
0. Needs triage 35-feedback bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Notification e-mails are sent without Date and Message-ID headersPeut-être pris @nofuturekid l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
unraid/webgui#2781 · 1 commentaire ·
Les mainteneurs répondent en général sous 6 jours
-
thrift: security issues < 0.25.0Ouverte1.severity: security
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
NixOS/nixpkgs#569828 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Use FormEvents constants when generating subscriberPeut-être pris @GromNaN l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
symfony/maker-bundle#1841 ·
Les mainteneurs répondent en général sous 1 jour