Issue #3468829 by Spokje: Follow-up Speed up WorkspacesContentModerationStateTest (7f653a7d) · Commits · project / drupal

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

+35 −13

Original line number	Diff line number	Diff line
		@@ -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.
		}

		/**