[MDL-35744] Wrong review options for 'Never submitted' quiz attempts - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_23_STABLE, MOODLE_24_STABLE
Fixed Branches:
MOODLE_23_STABLE
Pull from Repository:
git://github.com/timhunt/moodle.git
Pull Main Branch:
~~MDL-35744~~
Pull Main Diff URL:
https://github.com/timhunt/moodle/compare/master...MDL-35744
Testing Instructions:

Hide

The unit tests cover this quite well, but ...

1. Set up a quiz so that students are not allowed to review anything.

2. Get a quiz attempt into the never submitted state.

(When time expires -> Attempts must be submitted before time expires, and set a time limit. Start a quiz attempt as a student. Log out. Wait until time has expired. Log back in, and try to access the quiz attempt.)

3. Ensure that the student cannot review the attempt.

Show
The unit tests cover this quite well, but ... 1. Set up a quiz so that students are not allowed to review anything. 2. Get a quiz attempt into the never submitted state. (When time expires -> Attempts must be submitted before time expires, and set a time limit. Start a quiz attempt as a student. Log out. Wait until time has expired. Log back in, and try to access the quiz attempt.) 3. Ensure that the student cannot review the attempt.

It looks like the first line of quiz_attempt_state is wrong.

is a regression caused by

MDL-3030 Late quiz attempts should be closed automatically

Participants:: Andrew Davis, Dan Poltawski, Eloy Lafuente (stronk7), Tim Hunt