Verified Commit 261aa81c authored by Lee Rowlands's avatar Lee Rowlands Committed by Jess
Browse files

Issue #3273626 by xjm, dww, Spokje: Drupal Media JavaScript test suite causes... 

Issue #3273626 by xjm, dww, Spokje: Drupal Media JavaScript test suite causes database locks on SQLite

(cherry picked from commit 4990c413)
parent 409d23cd

core/modules/ckeditor5/tests/src/FunctionalJavascript/MediaTest.php

+8 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 


namespace Drupal\Tests\ckeditor5\FunctionalJavascript;



use Drupal\Core\Database\Database;

use Drupal\editor\Entity\Editor;

use Drupal\file\Entity\File;

use Drupal\filter\Entity\FilterFormat;
@@ -267,6 +268,13 @@ function (ConstraintViolation $v) { 
   * Tests that failed media embed preview requests inform the end user.

   */

  public function testErrorMessages() {

    // This test currently frequently causes the SQLite database to lock, so

    // skip the test on SQLite until the issue can be resolved.

    // @todo https://www.drupal.org/project/drupal/issues/3273626

    if (Database::getConnection()->driver() === 'sqlite') {

      $this->markTestSkipped('Test frequently causes a locked database on SQLite');

    }



    // Assert that a request to the `media.filter.preview` route that does not

    // result in a 200 response (due to server error or network error) is

    // handled in the JavaScript by displaying the expected error message.

core/modules/media/tests/src/FunctionalJavascript/CKEditorIntegrationTest.php

+8 −0
Original line number Diff line number Diff line
@@ -3,6 +3,7 @@ 
namespace Drupal\Tests\media\FunctionalJavascript;



use Drupal\Component\Utility\Html;

use Drupal\Core\Database\Database;

use Drupal\Core\Entity\Entity\EntityViewDisplay;

use Drupal\editor\Entity\Editor;

use Drupal\field\Entity\FieldConfig;
@@ -193,6 +194,13 @@ public function testOnlyDrupalMediaTagProcessed() { 
   * Tests that failed media embed preview requests inform the end user.

   */

  public function testErrorMessages() {

    // This test currently frequently causes the SQLite database to lock, so

    // skip the test on SQLite until the issue can be resolved.

    // @todo https://www.drupal.org/project/drupal/issues/3273626

    if (Database::getConnection()->driver() === 'sqlite') {

      $this->markTestSkipped('Test frequently causes a locked database on SQLite');

    }



    // Assert that a request to the `media.filter.preview` route that does not

    // result in a 200 response (due to server error or network error) is

    // handled in the JavaScript by displaying the expected error message.

core/modules/media/tests/src/FunctionalJavascript/MediaStandardProfileTest.php

+8 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 


namespace Drupal\Tests\media\FunctionalJavascript;



use Drupal\Core\Database\Database;

use Drupal\Core\Entity\Entity\EntityViewDisplay;

use Drupal\field\Entity\FieldConfig;

use Drupal\field\Entity\FieldStorageConfig;
@@ -49,6 +50,13 @@ protected function setUp(): void { 
   * increases the performance of this test.

   */

  public function testMediaSources() {

    // This test currently frequently causes the SQLite database to lock, so

    // skip the test on SQLite until the issue can be resolved.

    // @todo https://www.drupal.org/project/drupal/issues/3273626

    if (Database::getConnection()->driver() === 'sqlite') {

      $this->markTestSkipped('Test frequently causes a locked database on SQLite');

    }



    $storage = FieldStorageConfig::create([

      'entity_type' => 'node',

      'field_name' => 'field_related_media',