-
-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHPUnit\Framework\TestCase is not an interface #202
Comments
Seeing this same error well, after reverting to |
I encountered with the same issue and after changing version of |
likewise - changed version of phpstan to 0.12.42 in the drupal-check/composer.json and reinstalled composer, it worked like a charm. |
We need to bump mglaman/phpstan-drupal to 0.12.8 to fix this. |
@mglaman consider locking up the phpstan version in phpstan-drupal. This has been appearing already few times. This re-appears with phpstan 0.12.70 and newers (currently 0.12.81), and with the current composer requirement in phpstan-drupal is ^0.12.64, which means that if you have installed/updated Jan 27th, drupal-check is most likely broken for you. Simple fix until phpstan version is locked by default is to manually require lower phpstan version. |
Locking phpstan can lead to other problems. That is one reason I shipped a phar for drupal-check, because it provides locked dependencies. I've received funding, so drupal-check will get some love once I get phpstan-drupal stabilized more. |
This should be all fixed after the latest phpstan-drupal work. |
How is drupal-check installed?
drupal-check is installed as a dependency to my project
Environment:
Describe the bug
I recently ran composer update, and phpstan was updated to 0.12.70 and drupal-check 1.1.6
Before the update, I could run:
vendor/bin/drupal-check -d web/modules/custom
and not got any errors.
Now after the update, I get errors:
Line Block/ChildrenNavBlock.php
If I modify CheckCommand.php by adding --debug to the phpstan $command args and re-run, this is what I get:
Console output
The text was updated successfully, but these errors were encountered: