mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 05:58:34 +01:00
Code Issues Projects Releases Wiki Activity

Merge branch 'MDL-69278-master' of git://github.com/andrewnicols/moodle

Browse Source
This commit is contained in:
Eloy Lafuente (stronk7) 2020-07-18 11:55:48 +02:00
commit a16ab5697d
1 changed files with 35 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
lib/tests/behat/behat_hooks.php
View File

@ -86,6 +86,17 @@ class behat_hooks extends behat_base {
*/
protected static $currentstepexception = null;
/**
* If an Exception is thrown in the BeforeScenario hook it will cause the Scenario to be skipped, and the exit code
* to be non-zero triggering a potential rerun.
*
* To combat this the exception is stored and re-thrown when looking for exceptions.
* This allows the test to instead be failed and re-run correctly.
*
* @var null|Exception
*/
protected static $currentscenarioexception = null;
/**
* If we are saving any kind of dump on failure we should use the same parent dir during a run.
*
@ -173,7 +184,7 @@ class behat_hooks extends behat_base {
$message = <<<EOF
Your behat test site is outdated, please run the following command from your Moodle dirroot to drop, and reinstall the Behat test site.
{$comandpath}
{$commandpath}
EOF;
self::log_and_stop($message);
@ -362,8 +373,13 @@ EOF;
// The `before_first_scenario_start_session` function will have started the session instead.
return;
}
self::$currentscenarioexception = null;
$this->restart_session();
try {
$this->restart_session();
} catch (Exception $e) {
self::$currentscenarioexception = $e;
}
}
/**
@ -374,6 +390,12 @@ EOF;
*/
public function before_scenario_hook(BeforeScenarioScope $scope) {
global $DB;
if (self::$currentscenarioexception) {
// A BeforeScenario hook triggered an exception and marked this test as failed.
// Skip this hook as it will likely fail.
return;
}
$suitename = $scope->getSuite()->getName();
// Register behat selectors for theme, if suite is changed. We do it for every suite change.
@ -526,6 +548,12 @@ EOF;
* @BeforeStep
*/
public function before_step_javascript(BeforeStepScope $scope) {
if (self::$currentscenarioexception) {
// A BeforeScenario hook triggered an exception and marked this test as failed.
// Skip this hook as it will likely fail.
return;
}
self::$currentstepexception = null;
// Only run if JS.
@ -742,6 +770,11 @@ EOF;
* @see Moodle\BehatExtension\EventDispatcher\Tester\ChainedStepTester
*/
public function i_look_for_exceptions() {
// If the scenario already failed in a hook throw the exception.
if (!is_null(self::$currentscenarioexception)) {
throw self::$currentscenarioexception;
}
// If the step already failed in a hook throw the exception.
if (!is_null(self::$currentstepexception)) {
throw self::$currentstepexception;