Unverified Commit 409af2de authored by alexpott's avatar alexpott
Browse files

Issue #3136389 by jyotimishra123, Hardik_Patel_12, Suresh Prabhu Parkala,...

Issue #3136389 by jyotimishra123, Hardik_Patel_12, Suresh Prabhu Parkala, shimpy, longwave, quietone: Remove '@group legacy' from tests that do not exercise deprecated code
parent 2a17e73d
......@@ -8,7 +8,6 @@
* Tests removing action module's configuration.
*
* @group Update
* @group legacy
*/
class ActionConfigTest extends UpdatePathTestBase {
......
......@@ -12,7 +12,6 @@
* Tests the views integration of content_moderation.
*
* @group content_moderation
* @group legacy
*/
class ViewsDataIntegrationTest extends ViewsKernelTestBase {
......
......@@ -8,7 +8,6 @@
* Ensures that update hook is run properly for deleting obsolete Hal settings.
*
* @group hal
* @group legacy
*/
class HalSettingsDeletionUpdateTest extends UpdatePathTestBase {
......
......@@ -8,7 +8,6 @@
* Ensures that update hook is run properly for deleting obsolete serialization settings.
*
* @group serialization
* @group legacy
*/
class SerializationSettingsDeletionUpdateTest extends UpdatePathTestBase {
......
......@@ -10,7 +10,6 @@
* @see https://www.drupal.org/node/2928906
*
* @group Update
* @group legacy
*/
class IdentifierFieldSchemaTypeUpdateTest extends UpdatePathTestBase {
......
......@@ -10,7 +10,6 @@
* @see https://www.drupal.org/node/3099789
*
* @group Update
* @group legacy
*/
class RemoveRevisionMetadataBcLayerUpdateTest extends UpdatePathTestBase {
......
......@@ -11,7 +11,6 @@
* Tries to update a module which has no pre-existing schema.
*
* @group Update
* @group legacy
*/
class NoPreExistingSchemaUpdateTest extends BrowserTestBase {
use RequirementsPageTrait;
......
......@@ -18,13 +18,6 @@ class RolesRidTest extends UnitTestCase {
* Tests the titleQuery method.
*
* @covers ::titleQuery
*
* @group legacy
*
* Note this is only a legacy test because it triggers a call to
* \Drupal\Core\Entity\EntityTypeInterface::getLabelCallback() which is mocked
* and triggers a deprecation error. Remove when ::getLabelCallback() is
* removed.
*/
public function testTitleQuery() {
$role1 = new Role([
......
......@@ -10,7 +10,6 @@
* Tests the update path base class.
*
* @group Update
* @group legacy
*/
class UpdatePathTestBaseTest extends UpdatePathTestBase {
......
......@@ -6,7 +6,6 @@
/**
* @group File
* @group legacy
*/
class FileSystemRequirementsTest extends KernelTestBase {
......
......@@ -212,8 +212,6 @@ public function getParametersDataProvider() {
* @covers ::getParameterCall
*
* @dataProvider getDefinitionsDataProvider
*
* @group legacy
*/
public function testGetServiceDefinitions($services, $definition_services) {
$this->containerDefinition['services'] = $definition_services;
......@@ -479,8 +477,6 @@ protected function getServiceCall($id, $invalid_behavior = ContainerInterface::E
* @covers ::getReferenceCall
*
* @dataProvider publicPrivateDataProvider
*
* @group legacy
*/
public function testGetServiceDefinitionWithReferenceToAlias($public) {
$bar_definition = new Definition('\stdClass');
......@@ -536,8 +532,6 @@ public function publicPrivateDataProvider() {
* getDecoratedService().
*
* @covers ::getServiceDefinition
*
* @group legacy
*/
public function testGetServiceDefinitionForDecoratedService() {
$bar_definition = new Definition('\stdClass');
......
......@@ -114,12 +114,6 @@ protected function setUp(): void {
/**
* @covers ::getCssAssets
* @dataProvider providerAttachedAssets
* @group legacy
*
* Note the legacy group is used here because
* ActiveTheme::getStyleSheetsRemove() is called and is deprecated. As this
* code path will still be triggered until Drupal 9 we have to add the group.
* We do not trigger a silenced deprecation.
*/
public function testGetCssAssets(AttachedAssetsInterface $assets_a, AttachedAssetsInterface $assets_b, $expected_cache_item_count) {
$this->assetResolver->getCssAssets($assets_a, FALSE);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment