Problem/Motivation
The following tests are using WebTestBase, but never use HTTP, and can be DrupalUnitTestBase instead:
Drupal\config\Tests\DefaultConfigTest->testDefaultConfig()
Drupal\config\Tests\ConfigOtherModuleTest->testUninstall()
Drupal\config\Tests\ConfigOtherModuleTest->testInstallConfigEnityModuleFirst()
Drupal\config\Tests\ConfigOtherModuleTest->testInstallOtherModuleFirst()
Drupal\config\Tests\ConfigInstallWebTest->testInstallProfileConfigOverwrite()
Drupal\config\Tests\ConfigInstallWebTest->testIntegrationModuleReinstallation()
Drupal\config\Tests\ConfigEntityTest->testCRUD()
Drupal\config\Tests\ConfigEntityListTest->testList()
Proposed resolution
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#1 | config-2258217-1.patch | 42.15 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettUgh, d.o ate my comment. I did everything but ConfigInstallWebTest, which I can't figure out.
Comment #3
sunMoving those helper methods into a trait is a great idea! :-)
I can look into
ConfigInstallWebTest
— do you want to address theKeyValueConfigEntityStorageTest
failure first?Comment #12
Mile23All the tests touched by the patch have been converted to PHPUnit-based tests.
Calling this outdated.