moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-04 16:36:37 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-58023 behat: Improve find_node_in_navigation function

Browse source
This commit is contained in:
gthomas2 2017-02-22 15:04:23 +00:00
parent 1034421264
commit 01f2a530b7
2 changed files with 19 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

18
lib/behat/behat_base.php
View file

@ -944,4 +944,22 @@ class behat_base extends Behat\MinkExtension\Context\RawMinkContext {
} }
return $DB->get_record('user', ['id' => $userid]); return $DB->get_record('user', ['id' => $userid]);
} }
/**
* Trigger click on node via javascript instead of actually clicking on it via pointer.
*
* This function resolves the issue of nested elements with click listeners or links - in these cases clicking via
* the pointer may accidentally cause a click on the wrong element.
* Example of issue: clicking to expand navigation nodes when the config value linkadmincategories is enabled.
* @param NodeElement $node
*/
protected function js_trigger_click($node) {
if (!$this->running_javascript()) {
$node->click();
}
$this->ensure_node_is_visible($node); // Ensures hidden elements can't be clicked.
$xpath = $node->getXpath();
$script = "Syn.click({{ELEMENT}})";
$this->getSession()->getDriver()->triggerSynScript($xpath, $script);
}
} }

16
lib/tests/behat/behat_navigation.php
View file

@ -290,21 +290,7 @@ class behat_navigation extends behat_base {
if ($pnode && $this->running_javascript() && $pnode->hasAttribute('aria-expanded') && if ($pnode && $this->running_javascript() && $pnode->hasAttribute('aria-expanded') &&
($pnode->getAttribute('aria-expanded') == "false")) { ($pnode->getAttribute('aria-expanded') == "false")) {
$this->ensure_node_is_visible($pnode); $this->js_trigger_click($pnode);
// If node is a link then some driver click in the middle of the node, which click on link and
// page gets redirected. To ensure expansion works in all cases, check if the node to expand is a
// link and if yes then click on link and wait for it to navigate to next page with node expanded.
$nodetoexpandliteral = behat_context_helper::escape($parentnodes[$i]);
$nodetoexpandxpathlink = $pnodexpath . "/a[normalize-space(.)=" . $nodetoexpandliteral . "]";
if ($nodetoexpandlink = $node->find('xpath', $nodetoexpandxpathlink)) {
$behatgeneralcontext = behat_context_helper::get('behat_general');
$nodetoexpandlink->click();
$behatgeneralcontext->wait_until_the_page_is_ready();
} else {
$pnode->click();
}
// Wait for node to load, if not loaded before. // Wait for node to load, if not loaded before.
if ($pnode->hasAttribute('data-loaded') && $pnode->getAttribute('data-loaded') == "false") { if ($pnode->hasAttribute('data-loaded') && $pnode->getAttribute('data-loaded') == "false") {