Quiz not redirecting to quiz/QUIZ-ID

Will probably need test coverage

@Dhammika I notice that your 2x patches are named 'fixed-quiz-feedback-issue'. That sounds like it relates to a different issue? Can you please confirm that your patches fix this issue?

Only see a test file in the MR. Which if it’s covering a new change would expect to fail.

Also appears to have standard issues.

I have tried applying the patch of @Dhammika but getting errors.

The MR should contain the full fix.

It has a test only feature already built in for testing if it fails without the fix.

@smustgrave I have now included a full fix in the MR. I have updated the issue description.

Why needs work?

@smustgrave Because i thought maybe the reason you had not reviewed for a few days was that I had changed the status to 'needs review' and that that perhaps only maintainers should do that. I believe it IS ready for review.

Nope you are free to put into review.

Will try and get to sometime next week.

@smustgrave Much appreciated.

@smustgrave To make your task a bit easier, you will find one of my last changes was to an apparently random test. It was a test that I found was failing after I completed the task and got my test to pass. I discovered that the 'random' test contained an assertion for a 403 access denied. So I believed that the condition in my event listener (when a certain exception is raised) must be met in this 'random' test, also. So I edited the test to expect not a 403 but a 200 and redirect to quiz/quiz id. That made it pass along with my new test.

Had to fix up the pipeline to have descriptions and used constructor promotion instead so it's all green now.

Thank you for completing this fix!

Automatically closed - issue fixed for 2 weeks with no activity.