Skip to content
Snippets Groups Projects
Commit d00c4eb2 authored by catch's avatar catch
Browse files

Revert "Issue #3317520 by mstrelan, nod_, wim leers: [random test failure]...

Revert "Issue #3317520 by mstrelan, nod_, wim leers: [random test failure] Drupal\Tests\settings_tray\FunctionalJavascript\SettingsTrayBlockFormTest::testEditModeEnableDisable"

This reverts commit 853031b6.
parent fd1519be
No related branches found
No related tags found
3 merge requests!5423Draft: Resolve #3329907 "Test2",!3478Issue #3337882: Deleted menus are not removed from content type config,!579Issue #2230909: Simple decimals fail to pass validation
Pipeline #470342 passed with warnings
Pipeline: drupal

#470358

    Pipeline: drupal

    #470352

      Pipeline: drupal

      #470347

        ...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
        use Drupal\settings_tray_test\Plugin\Block\SettingsTrayFormAnnotationIsClassBlock; use Drupal\settings_tray_test\Plugin\Block\SettingsTrayFormAnnotationIsClassBlock;
        use Drupal\settings_tray_test\Plugin\Block\SettingsTrayFormAnnotationNoneBlock; use Drupal\settings_tray_test\Plugin\Block\SettingsTrayFormAnnotationNoneBlock;
        use Drupal\Tests\WaitTerminateTestTrait;
        use Drupal\user\Entity\Role; use Drupal\user\Entity\Role;
        /** /**
        ...@@ -16,8 +15,6 @@ ...@@ -16,8 +15,6 @@
        */ */
        class SettingsTrayBlockFormTest extends SettingsTrayTestBase { class SettingsTrayBlockFormTest extends SettingsTrayTestBase {
        use WaitTerminateTestTrait;
        /** /**
        * {@inheritdoc} * {@inheritdoc}
        */ */
        ...@@ -47,7 +44,6 @@ protected function setUp(): void { ...@@ -47,7 +44,6 @@ protected function setUp(): void {
        'search content', 'search content',
        ]); ]);
        $this->drupalLogin($user); $this->drupalLogin($user);
        $this->setWaitForTerminate();
        } }
        /** /**
        ...@@ -235,6 +231,7 @@ public function getBlockTests() { ...@@ -235,6 +231,7 @@ public function getBlockTests() {
        * Tests enabling and disabling Edit Mode. * Tests enabling and disabling Edit Mode.
        */ */
        public function testEditModeEnableDisable(): void { public function testEditModeEnableDisable(): void {
        $this->markTestSkipped("Skipped due to frequent random test failures. See https://www.drupal.org/project/drupal/issues/3317520");
        foreach (static::getTestThemes() as $theme) { foreach (static::getTestThemes() as $theme) {
        $this->enableTheme($theme); $this->enableTheme($theme);
        $block = $this->placeBlock('system_powered_by_block'); $block = $this->placeBlock('system_powered_by_block');
        ......
        ...@@ -49,10 +49,11 @@ protected function openBlockForm($block_selector, $contextual_link_container = ' ...@@ -49,10 +49,11 @@ protected function openBlockForm($block_selector, $contextual_link_container = '
        // When page first loads Edit Mode is not triggered until first contextual // When page first loads Edit Mode is not triggered until first contextual
        // link is added. // link is added.
        $this->assertNotEmpty($this->assertSession()->waitForElementVisible('css', '.dialog-off-canvas-main-canvas.js-settings-tray-edit-mode')); $this->assertNotEmpty($this->assertSession()->waitForElementVisible('css', '.dialog-off-canvas-main-canvas.js-settings-tray-edit-mode'));
        // @todo https://www.drupal.org/project/drupal/issues/3317520 Work why the
        // sleep is necessary in.
        usleep(100000);
        $block = $this->assertSession()->waitForElementVisible('css', $block_selector); $block = $this->getSession()->getPage()->find('css', $block_selector);
        $this->assertNotEmpty($block);
        $block->mouseOver(); $block->mouseOver();
        $block->click(); $block->click();
        $this->waitForOffCanvasToOpen(); $this->waitForOffCanvasToOpen();
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Please register or to comment