Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
From @longwave in #3123253-28: Remove usage of AssertLegacyTrait::pass() from base test classes:
+++ b/core/tests/Drupal/FunctionalTests/Installer/ConfigAfterInstallerTestBase.php
@@ -32,13 +32,7 @@ protected function assertInstalledConfig(array $skipped_config) {
+ $this->assertConfigDiff($result, $config_name, $skipped_config);
I checked what this method does and it doesn't actually perform assertions, rather it just throws an exception on failure. This means we have to use the $this->addToAssertionCount(1);
workaround at the top of the method as there are now no assertions here. Worth opening a new issue to make it more PHPUnit-y?
Comments