Skip to content

Commit

Permalink
MDL-66979 behat: Add a step to set a timeout factor for a test
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewnicols committed Jan 29, 2021
1 parent adc8204 commit bead2a1
Show file tree
Hide file tree
Showing 3 changed files with 33 additions and 7 deletions.
21 changes: 21 additions & 0 deletions lib/behat/classes/behat_session_trait.php
Expand Up @@ -1355,4 +1355,25 @@ public function get_sesskey(): string {

return $this->evaluate_script($script);
}

/**
* Set the timeout factor for the remaining lifetime of the session.
*
* @param int $factor A multiplication factor to use when calculating the timeout
*/
public function set_test_timeout_factor(int $factor = 1): void {
$driver = $this->getSession()->getDriver();

if (!$driver instanceof \OAndreyev\Mink\Driver\WebDriver) {
// This is a feature of the OAndreyev MinkWebDriver.
return;
}

// Use `get_real_timeout` to multiply the timeout by the global behat_increasetimeout value, and again by the
// factor specified.
$this->getSession()->getDriver()->setTimeouts([
// The standard script timeout is 30000 ms. (aka, 30 seconds).
'script' => self::get_real_timeout(30) * 1000 * $factor,
]);
}
}
11 changes: 11 additions & 0 deletions lib/tests/behat/behat_general.php
Expand Up @@ -2065,4 +2065,15 @@ public function i_visit($localurl): void {
$localurl = new moodle_url($localurl);
$this->getSession()->visit($this->locate_path($localurl->out_as_local_url(false)));
}

/**
* Increase the webdriver timeouts.
*
* This should be reset between scenarios, or can be called again to decrease the timeouts.
*
* @Given I mark this test as slow setting a timeout factor of :factor
*/
public function i_mark_this_test_as_long_running(int $factor = 2): void {
$this->set_test_timeout_factor($factor);
}
}
8 changes: 1 addition & 7 deletions lib/tests/behat/behat_hooks.php
Expand Up @@ -303,13 +303,7 @@ protected function restart_session(): void {
protected function start_session(): void {
$this->getSession()->start();

if ($this->running_javascript()) { // Goutte driver doesn't implement this.
$this->getSession()->getDriver()->setTimeouts([
// The standard script timeout is 30000 ms. (aka, 30 seconds).
// Use `get_real_timeout` to multiply this by the behat increased timeout factor.
'script' => self::get_real_timeout(30) * 1000,
]);
}
$this->set_test_timeout_factor(1);
}

/**
Expand Down

0 comments on commit bead2a1

Please sign in to comment.