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

Issue #3316816 by Wim Leers, nod_: Stabilize FunctionalJavascript testing...

Issue #3316816 by Wim Leers, nod_: Stabilize FunctionalJavascript testing AJAX: make ::setValue() trigger both "input" and "formUpdated" events

(cherry picked from commit 6a1bac3c)
parent 7770bcc5
No related branches found
No related tags found
23 merge requests!8394[warning] array_flip(): Can only flip STRING and INTEGER values, when saving a non-revisionable custom content entity,!7780issue 3443822: fix for 'No route found for the specified format html. Supported formats: json, xml.',!5013Issue #3071143: Table Render Array Example Is Incorrect,!4848Issue #1566662: Update module should send notifications on Thursdays,!4792Issue #2230689: Remove redundant "Italic" style,!4220Issue #3368223: Link field > Access to internal links is not checked on display.,!3884Issue #3356842,!3870Issue #3087868,!3812Draft: Issue #3339373 by alexpott, andypost, mondrake:...,!3686Issue #3219967 against 9.5.x,!3683Issue #2939397: Clearing AliasManager cache with root path raises warning,!3543Issue #3344259: Allow ajax dialog to have focus configurable,!3356Issue #3209129: Scrolling problems when adding a block via layout builder,!2921Issue #1383696: Allow a custom HTML element to be selected for a grouping field,!2280Issue #3280415: Metapackage Generator Breaks Under Composer --no-dev,!2205Quote all names in the regions section.,!2050Issue #3272969: Remove UnqiueField constraint.,!1956Issue #3268872: hook_views_invalidate_cache not called when a view is deleted,!1893Issue #3217260: Add a way to make media captions not editable in CKEditor,!1459Issue #3087632: menu_name max length is too long,!878Issue #3221534: throw an exception when IDs passed to loadMultiple() are badly formed,!866Issue #2845319: The highlighting of the 'Home' menu-link does not respect query strings and fragment identifiers,!204Issue #3040556: It is not possible to react to an entity being duplicated
......@@ -86,14 +86,8 @@ protected function addImage() {
$this->waitForEditor();
$this->pressEditorButton('Insert image');
$panel = $page->find('css', '.ck-dropdown__panel.ck-image-insert__panel');
// Do not use setValue method as it triggers a blur event by default that
// closes the CKEditor 5 panel, making it impossible to click on the Insert
// button.
$this->getSession()->executeScript('
const input = document.querySelector(".ck-dropdown__panel.ck-image-insert__panel input[type=text]");
input.value = "' . $src . '";
input.dispatchEvent(new Event("input", {bubbles:true}));
');
$src_input = $panel->find('css', 'input[type=text]');
$src_input->setValue($src);
$panel->find('xpath', "//button[span[text()='Insert']]")->click();
}
......
......@@ -104,14 +104,8 @@ public function testImageUrlWidget(): void {
$this->pressEditorButton('Insert image');
$panel = $page->find('css', '.ck-dropdown__panel.ck-image-insert__panel');
// Do not use setValue method as it triggers a blur event by default that
// closes the CKEditor 5 panel, making it impossible to click on the Insert
// button.
$this->getSession()->executeScript('
const input = document.querySelector(".ck-dropdown__panel.ck-image-insert__panel input[type=text]");
input.value = "' . $src . '";
input.dispatchEvent(new Event("input", {bubbles:true}));
');
$src_input = $panel->find('css', 'input[type=text]');
$src_input->setValue($src);
$panel->find('xpath', "//button[span[text()='Insert']]")->click();
$this->assertNotEmpty($assert_session->waitForElementVisible('css', $image_selector));
......
......@@ -142,6 +142,24 @@ public function setValue($xpath, $value) {
$not_clickable_exception = NULL;
$result = $this->waitFor(10, function () use (&$not_clickable_exception, $xpath, $value) {
try {
// \Behat\Mink\Driver\Selenium2Driver::setValue() will call .blur() on
// the element, modify that to trigger the "input" and "change" events
// instead. They indicate the value has changed, rather than implying
// user focus changes.
$this->executeJsOnXpath($xpath, <<<JS
var node = {{ELEMENT}};
var original = node.blur;
node.blur = function() {
node.dispatchEvent(new Event("input", {bubbles:true}));
node.dispatchEvent(new Event("change", {bubbles:true}));
// Do not wait for the debounce, which only triggers the 'formUpdated` event
// up to once every 0.3 seconds. In tests, no humans are typing, hence there
// is no need to debounce.
// @see Drupal.behaviors.formUpdated
node.dispatchEvent(new Event("formUpdated", {bubbles:true}));
node.blur = original;
};
JS);
parent::setValue($xpath, $value);
return TRUE;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment