Issue #3468829 by Spokje: Follow-up Speed up WorkspacesContentModerationStateTest

(cherry picked from commit 7f653a7d)

core/modules/content_moderation/tests/src/Kernel/WorkspacesContentModerationStateTest.php

@@ -40,10 +40,30 @@ class WorkspacesContentModerationStateTest extends ContentModerationStateTest {
    */
   protected $revEntityTypeId = 'entity_test_revpub';
 
+  const SKIP_METHODS = [
+    // This test creates published default revisions in Live, which can not be
+    // deleted in a workspace. A test scenario for the case when Content
+    // Moderation and Workspaces are used together is covered in
+    // parent::testContentModerationStateRevisionDataRemoval().
+    'testContentModerationStateDataRemoval',
+    // This test does not assert anything that can be workspace-specific.
+    'testModerationWithFieldConfigOverride',
+    // This test does not assert anything that can be workspace-specific.
+    'testWorkflowDependencies',
+    // This test does not assert anything that can be workspace-specific.
+    'testWorkflowNonConfigBundleDependencies',
+    // This test does not assert anything that can be workspace-specific.
+    'testGetCurrentUserId',
+  ];
+
   /**
    * {@inheritdoc}
    */
   protected function setUp(): void {
+    if (in_array($this->name(), static::SKIP_METHODS, TRUE)) {
+      $this->markTestSkipped('Irrelevant for this test');
+    }
+
     parent::setUp();
 
     $this->initializeWorkspacesModule();
@@ -181,43 +201,45 @@ public static function basicModerationTestCases() {
    * {@inheritdoc}
    */
   public function testContentModerationStateDataRemoval($entity_type_id = NULL): void {
-    // This test creates published default revisions in Live, which can not be
-    // deleted in a workspace. A test scenario for the case when Content
-    // Moderation and Workspaces are used together is covered in
-    // parent::testContentModerationStateRevisionDataRemoval().
-    $this->markTestSkipped();
+    // Deliberately empty body. This test will be skipped in the setUp() of this
+    // class. However, for it to be picked up there, in PHPUnit 9, it still
+    // needs to be defined in this class.
   }
 
   /**
    * {@inheritdoc}
    */
   public function testModerationWithFieldConfigOverride(): void {
-    // This test does not assert anything that can be workspace-specific.
-    $this->markTestSkipped();
+    // Deliberately empty body. This test will be skipped in the setUp() of this
+    // class. However, for it to be picked up there, in PHPUnit 9, it still
+    // needs to be defined in this class.
   }
 
   /**
    * {@inheritdoc}
    */
   public function testWorkflowDependencies(): void {
-    // This test does not assert anything that can be workspace-specific.
-    $this->markTestSkipped();
+    // Deliberately empty body. This test will be skipped in the setUp() of this
+    // class. However, for it to be picked up there, in PHPUnit 9, it still
+    // needs to be defined in this class.
   }
 
   /**
    * {@inheritdoc}
    */
   public function testWorkflowNonConfigBundleDependencies(): void {
-    // This test does not assert anything that can be workspace-specific.
-    $this->markTestSkipped();
+    // Deliberately empty body. This test will be skipped in the setUp() of this
+    // class. However, for it to be picked up there, in PHPUnit 9, it still
+    // needs to be defined in this class.
   }
 
   /**
    * {@inheritdoc}
    */
   public function testGetCurrentUserId(): void {
-    // This test does not assert anything that can be workspace-specific.
-    $this->markTestSkipped();
+    // Deliberately empty body. This test will be skipped in the setUp() of this
+    // class. However, for it to be picked up there, in PHPUnit 9, it still
+    // needs to be defined in this class.
   }
 
   /**