Merge branch 'MDL-79202-402' of https://github.com/paulholden/moodle into MOODLE_402_STABLE
This commit is contained in:
@@ -318,7 +318,7 @@ class external extends external_api {
|
||||
$timenow = time();
|
||||
if ($timenow - $last < $mintimereq) {
|
||||
$minutes = $mintimereq / MINSECS;
|
||||
throw new moodle_exception('autologinkeygenerationlockout', 'tool_mobile', $minutes);
|
||||
throw new moodle_exception('autologinkeygenerationlockout', 'tool_mobile', '', $minutes);
|
||||
}
|
||||
set_user_preference('tool_mobile_autologin_request_last', $timenow, $USER);
|
||||
|
||||
|
||||
@@ -404,18 +404,18 @@ class externallib_test extends externallib_advanced_testcase {
|
||||
$result = external::get_autologin_key($token->privatetoken);
|
||||
$result = external_api::clean_returnvalue(external::get_autologin_key_returns(), $result);
|
||||
|
||||
// Change min time between requests to 30 seconds.
|
||||
set_config('autologinmintimebetweenreq', 30, 'tool_mobile');
|
||||
// Change min time between requests to 3 minutes.
|
||||
set_config('autologinmintimebetweenreq', 3 * MINSECS, 'tool_mobile');
|
||||
|
||||
// Mock a previous request, 60 seconds ago.
|
||||
$mocktime = time() - MINSECS;
|
||||
// Mock a previous request, 4 minutes ago.
|
||||
$mocktime = time() - (4 * MINSECS);
|
||||
set_user_preference('tool_mobile_autologin_request_last', $mocktime, $USER);
|
||||
$result = external::get_autologin_key($token->privatetoken); // All good, we were expecint 30 seconds or more.
|
||||
$result = external::get_autologin_key($token->privatetoken);
|
||||
$result = external_api::clean_returnvalue(external::get_autologin_key_returns(), $result);
|
||||
|
||||
// We just requested one token, we must wait.
|
||||
$this->expectException('moodle_exception');
|
||||
$this->expectExceptionMessage(get_string('autologinkeygenerationlockout', 'tool_mobile'));
|
||||
$this->expectExceptionMessage(get_string('autologinkeygenerationlockout', 'tool_mobile', 3));
|
||||
$result = external::get_autologin_key($token->privatetoken);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user