ls1intum / Artemis

Artemis - Interactive Learning with Automated Feedback
https://docs.artemis.cit.tum.de
MIT License
452 stars 275 forks source link

`Exam mode`: Add banner to exam results view for instructors before results are published #8892

Open florian-glombik opened 3 days ago

florian-glombik commented 3 days ago

Problem

Can cause confusion as instructors might think the student view does already have the published results. Lead to confusion during testing session

Solution

Add a banner when instructors view a student exam that is displayed if the results have not been published yet.

Could be analogue to the "not submitted" banner shown to instructors when viewing unsubmitted exams image

Initial Bug Description ### Describe the bug With programming exercise results where the release date is set to "AterReleaseDateOfResults" the exercise results shall not be visible before the release date of results. ### To Reproduce 0. `instructor`: prepare an exam and do not set the release date of results 1. `student`: Participate in an exam programming exercise (e.g. Java) where the default test case visibility is "AfterReleaseDateOfResults" 2. `student`: Push shortly before the end of the exam 3. `student`: See that you will receive the results as student 4. `instructor`: Trigger the test runs 5. `student`: see that the result is displayed in the summary for programming exercises ### Expected behavior With the visibility `AfterReleaseDateOfResults` the results for exam programming exercises should be visible for students only after the Release Date of Results (if no release date is set not at all, it is most important not to leak information of tests and weights that might not be finalized yet). ### Screenshots _No response_ ### Which version of Artemis are you seeing the problem on? 7.3.1 ### What browsers are you seeing the problem on? Chrome
florian-glombik commented 3 days ago

I must have had opened the wrong tab, being logged in with an instructor you can see the results as soon as they are available.

I think it might be good to add a banner at the top when viewing the exam as an instructor that indicates that the results are not released yet and this does not 100% reflect the student perspective to prevent confusion in the future.