I believe this should be removed when using BTB.

Comments

Sam152 created an issue. See original summary.

Sam152’s picture

Sam152 CreditAttribution: Sam152 at PreviousNext commented

Things that we should try and remove:

./modules/video_embed_wysiwyg/tests/src/FunctionalJavascript/EmbedDialogTest.php:6:use Drupal\simpletest\ContentTypeCreationTrait;
./tests/src/Functional/AdminUserTrait.php:5:use Drupal\simpletest\UserCreationTrait;
./tests/src/Functional/AssertionsTrait.php:6: * Some legacy assertions that existed in simpletest.
./tests/src/Functional/EntityDisplaySetupTrait.php:6:use Drupal\simpletest\ContentTypeCreationTrait;
./tests/src/Functional/EntityDisplaySetupTrait.php:7:use Drupal\simpletest\NodeCreationTrait;
./tests/src/Functional/FieldConfigurationTest.php:4:use Drupal\simpletest\ContentTypeCreationTrait;
Sam152’s picture

Sam152 CreditAttribution: Sam152 at PreviousNext commented
Status: Active » Fixed

I thought these were inherently incompatibly, but it looks like there is specific handling for phpunit based tests. Some methods on the traits do blow up based on not being in the context of SimpleTestBase, however I think the benefit of these utilities is too much to remove just yet.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.