sascha-egerer / phpstan-typo3

TYPO3 CMS class reflection extension for PHPStan & framework-specific rules
MIT License
42 stars 22 forks source link

[TASK] Make the data providers static #145

Closed oliverklee closed 9 months ago

oliverklee commented 9 months ago

This is required in newer PHPUnit versions.

oliverklee commented 9 months ago

I have no idea where the gatherAssertTypes method comes from and would appreciate a pointer for that.

sascha-egerer commented 9 months ago

I have no idea where the gatherAssertTypes method comes from and would appreciate a pointer for that.

It's defined in the parent abstract class PHPStan\Testing\TypeInferenceTestCase

oliverklee commented 9 months ago

Ah, I see. For some reasons, my vendor folder was excluded in my PhpStorm.

oliverklee commented 9 months ago

We probably need to update PHPStan so TypeInferenceTestCase::gatherAssertTypes is static.

oliverklee commented 9 months ago

https://github.com/phpstan/phpstan-src/commit/ea2670a086f0f699b92dd6b2535504771054498c

sascha-egerer commented 9 months ago

We probably need to update PHPStan so TypeInferenceTestCase::gatherAssertTypes is static.

Ok, then we've finally found a reason in increase the dependency version ;)

sascha-egerer commented 9 months ago

Then lets reopen https://github.com/sascha-egerer/phpstan-typo3/pull/148 ?

sascha-egerer commented 9 months ago

Ok also fine like that. Thanks

oliverklee commented 9 months ago

I'd like to bump only the PHPStan dependency then. PR coming …

oliverklee commented 9 months ago

Okay, assertFileAsserts still is not static. We probably need to avoid calling it in a data provider. https://github.com/phpstan/phpstan-src/blob/1.11.x/src/Testing/TypeInferenceTestCase.php

oliverklee commented 9 months ago

Ah, wait …

oliverklee commented 9 months ago

Done: #150

sascha-egerer commented 9 months ago

Done: #150

this one is merged

oliverklee commented 9 months ago

Rebased and repushed.

oliverklee commented 9 months ago

Thanks! That was my bunch of changes for now. :-)