Commit 2d708a73 authored by Tim Hunt's avatar Tim Hunt

MDL-57588 questions: regrading should not lose flags

parent e7059ec5
......@@ -1311,6 +1311,8 @@ class question_attempt {
if ($finished) {
// This file is part of Moodle -
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// GNU General Public License for more details.
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <>.
* This file contains tests that walks a question through a whole attempt.
* @package core_question
* @copyright 2017 The Open University
* @license GNU GPL v3 or later
defined('MOODLE_INTERNAL') || die();
global $CFG;
require_once(__DIR__ . '/..//lib.php');
require_once(__DIR__ . '/helpers.php');
* End-to-end tests of attempting a question.
* @copyright 2017 The Open University
* @license GNU GPL v3 or later
class core_question_walkthrough_testcase extends qbehaviour_walkthrough_test_base {
public function test_regrade_does_not_lose_flag() {
// Create a true-false question with correct answer true.
$tf = test_question_maker::make_question('truefalse', 'true');
$this->start_attempt_at_question($tf, 'deferredfeedback', 2);
// Process a true answer.
$this->process_submission(array('answer' => 1));
// Finish the attempt.
// Flag the question.
// Now change the correct answer to the question, and regrade.
$tf->rightanswer = false;
// Verify the flag has not been lost.
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment