PHPUnit 9.5 Call to undefined method ::getAnnotations()

Thanks! Queued on PHP 7.3 as well, as we are also compatible with PHPUnit 8 (which is used on PHP 7.3 only now) and so need the tests to still run there as well.

Found the same issue today

The patch worked fine, in the sense that it fixed the error message.

But it caused another issue:
FATAL Drupal\Tests\boxout\Functional\BoxoutTest: test runner returned a non-zero error code (2)

Steps to reproduce:
- Install Drupal 9
- Run tests without excluding deprecation i.e.

sudo -u www-data php web/core/scripts/run-tests.sh \
	    --php /usr/local/bin/php \
	    --verbose \
	    --keep-results \
	    --color \
	    --concurrency "32" \
	    --repeat "1" \
	    --types "Simpletest,PHPUnit-Unit,PHPUnit-Kernel,PHPUnit-Functional" \
	    --sqlite sites/default/files/.ht.sqlite \
	    --url http://localhost \
	    --directory "modules/contrib/boxout"'

Pre-requisites: the test must contain a check for response code i.e. $this->assertResponse(200, 'Test response');

The results will report this deprecation:

      1x: AssertLegacyTrait::assertResponse() is deprecated in drupal:8.2.0 and
    is removed from drupal:10.0.0. Use
    $this->assertSession()->statusCodeEquals() instead. See
    https://www.drupal.org/node/3129738
        1x in BoxoutTest::testBoxoutArticle from Drupal\Tests\boxout\Functional

Now, change $this->assertResponse() to $this->assertSession()->statusCodeEqual() and re-run the tests

The fatal error happens.

Bumping to critical since now the PHP 7.4 and 8 tests are all failing.

IMHO we should consider to set the PHPUnit constraint in composer.json to limit the upper version, PHPUnit are very aggressive in removing internal methods and we may end up with failing tests at any PHPUnit minor.

@mondrake I think this is fine. I don't think we should have upper constraints here - it'll just stop us from fixing things like this.

@marcelovani the other reported in #5 is because the test is using deprecated code. You could either disable deprecation testing or fix it.

Committed and pushed 754b8ffab2 to 9.2.x and adc897b798 to 9.1.x. Thanks!

For anyone facing this while core 9.1.1 isn't released... simply dropping back to phpunit 9.4.3 is sufficient to get tests running again.

composer require --dev phpunit/phpunit:9.4.3 --with-all-dependencies

@heddn That works.

Note: You have to be using composer >= 2 to use the --with-all-dependencies switch

Update composer with the following:
composer selfupdate --2