- Issue created by @Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Was able to reproduce locally:
kristenpol@drupal101-web:/var/www/html$ ./vendor/bin/phpunit -c ./web/core ./web/modules/contrib/consultation PHPUnit 9.6.16 by Sebastian Bergmann and contributors. Testing /var/www/html/web/modules/contrib/consultation E 1 / 1 (100%)R Time: 00:01.072, Memory: 4.00 MB There was 1 error: 1) Drupal\Tests\consultation\Functional\ConsultationNodeHelperTest::testConsultationHelper Exception: Drupal\Tests\BrowserTestBase::$defaultTheme is required. See https://www.drupal.org/node/3083055, which includes recommendations on which theme to use. /var/www/html/web/core/lib/Drupal/Core/Test/FunctionalTestSetupTrait.php:427 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:556 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:367 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:728 -- There was 1 risky test: 1) Drupal\Tests\consultation\Functional\ConsultationNodeHelperTest::testConsultationHelper This test did not perform any assertions /var/www/html/web/core/tests/Drupal/Tests/Listeners/DrupalListener.php:65 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:452 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestSuite.php:684 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestSuite.php:684 /var/www/html/vendor/phpunit/phpunit/src/TextUI/TestRunner.php:651 /var/www/html/vendor/phpunit/phpunit/src/TextUI/Command.php:144 /var/www/html/vendor/phpunit/phpunit/src/TextUI/Command.php:97 ERRORS! Tests: 1, Assertions: 0, Errors: 1, Risky: 1.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
After adding the $defaultTheme, get this:
There was 1 error: 1) Drupal\Tests\consultation\Functional\ConsultationNodeHelperTest::testConsultationHelper Drupal\Core\Config\UnmetDependenciesException: Configuration objects provided by <em class="placeholder">consultation</em> have unmet dependencies: <em class="placeholder">views.view.consultations (views)</em> /var/www/html/web/core/lib/Drupal/Core/Config/UnmetDependenciesException.php:100 /var/www/html/web/core/lib/Drupal/Core/Config/ConfigInstaller.php:527 /var/www/html/web/core/lib/Drupal/Core/ProxyClass/Config/ConfigInstaller.php:132 /var/www/html/web/core/lib/Drupal/Core/Extension/ModuleInstaller.php:199 /var/www/html/web/core/lib/Drupal/Core/ProxyClass/Extension/ModuleInstaller.php:83 /var/www/html/web/core/lib/Drupal/Core/Test/FunctionalTestSetupTrait.php:465 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:557 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:367 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:728
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
After adding views as dependency, here's the next error:
There was 1 error: 1) Drupal\Tests\consultation\Functional\ConsultationNodeHelperTest::testConsultationHelper Drupal\Core\Config\Schema\SchemaIncompleteException: No schema for consultation.settings /var/www/html/web/core/lib/Drupal/Core/Config/Development/ConfigSchemaChecker.php:87 /var/www/html/web/core/lib/Drupal/Component/EventDispatcher/ContainerAwareEventDispatcher.php:111 /var/www/html/web/core/lib/Drupal/Core/Config/Config.php:229 /var/www/html/web/core/lib/Drupal/Core/Config/ConfigInstaller.php:396 /var/www/html/web/core/lib/Drupal/Core/Config/ConfigInstaller.php:149 /var/www/html/web/core/lib/Drupal/Core/ProxyClass/Config/ConfigInstaller.php:75 /var/www/html/web/core/lib/Drupal/Core/Extension/ModuleInstaller.php:327 /var/www/html/web/core/lib/Drupal/Core/ProxyClass/Extension/ModuleInstaller.php:83 /var/www/html/web/core/lib/Drupal/Core/Test/FunctionalTestSetupTrait.php:465 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:557 /var/www/html/web/core/tests/Drupal/Tests/BrowserTestBase.php:367 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:728
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
We'll need to make a schema file for the settings form:
https://www.drupal.org/docs/drupal-apis/configuration-api/configuration-... β
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Running into more errors. Given that the test doesn't actually DO anything, then it should be deleted for now.
/** * Tests if the paragraph type class is present when added. */ public function testConsultationHelper() { return; $this->loginAsAdmin();
-
Kristen Pol β
committed 0d1d93c9 on 1.x
Issue #3423257 by Kristen Pol, Toby Wild: Remove unused consultation...
-
Kristen Pol β
committed 0d1d93c9 on 1.x
- Issue was unassigned.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Removed and will be part of next release.
- Status changed to Fixed
about 1 year ago 5:36pm 22 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.