Adaptive learning: Add endpoint to retrieve student course metrics

[MaximilianAnzinger]

Checklist

General

• I chose a title conforming to the naming conventions for pull requests.

Server

• Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
• I strictly followed the server coding and design guidelines.
• I added multiple integration tests (Spring) related to the features (with a high test coverage).
• I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
• I documented the Java code using JavaDoc style.

Motivation and Context

We want to create a new course-level dashboard view and we need an efficient approach to fetch the displayed information (e.g. exercises and related metrics).

Description

Adds new endpoint to fetch exercise related data for new dashboard.

Testserver States

Note

These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

• I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

• Code Review 1
• Code Review 2

Test Coverage

[JohannesStoehr]

Question in general: Should these metrics take practice participations into account?

[FelixTJDietrich]

I have two exercises, one programming and one text and it looks like this for me now:

• exerciseInformation is fine except maybe the type which is currently something like de.tum.in.www1.artemis.domain.ProgrammingExercise, can't it be simpler?
• averageScore seems fine but is 0 for exercises that are not at their due date yet, maybe it is a good idea to not give an average score if the due date is not over. Also what about exercises where assessment is still in progress?
• averageLatestSubmission is a double instead of a timestamp which is inconsistent with latestSubmission
• Why do we have submissionTimestamps and why does it include the score? I'm also confused why it contains exerciseId: 4 this seems to be wrong. I think submissionTimestamps should be replaced with score to keep it consistent with the rest