General: Include more information in course archive

[tobias-lippert]

Checklist

General

• I tested all changes and their related features with all corresponding user types on a test server.
• Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
• 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 followed the coding and design guidelines.
• I added multiple integration tests (Spring) related to the features (with a high test coverage).
• I documented the Java code using JavaDoc style.

Motivation and Context

Course archives do not contain exercise details or the problem statement for all exercise types. This PR adds this information to course archives.

Description

Refactored the SubmissionExportService. Fixed the exports of File Upload submissions. Introduced a new ExerciseWithSubmissionsExportService that exports a JSON file with information about the exercise, a markdown file with the problem statement and all submissions or student repos for an exercise. For programming exercise, we also export template, solution, test, and aux repositories.

Steps for Testing

Prerequisites:

• 1 Instructor

1. Log in to Artemis
2. Navigate to Course Administration
3. Select a course that has an end date in the past or set the end date of a course to a value in the past.
4. Click on archive
5. Once finished, download the archive and verify its content.
6. Every exercise should have its own directory with the problem statement as markdown file, the exercise details json file and all submisisons.
7. Also test the export of submissions (for text, modeling, and file upload) available on the scores page of an exercise (empty submissions are always excluded)
8. Also test that the export of a single programming exercise (using the download exercise button) still works.
9. Run a plagiarism check for a programming exercise and check it still works (because it's affected by the changes in FileService)

Review Progress

Performance Review

• I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
• 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

Manual Tests

• Test 1
• Test 2

Test Coverage

Server

Class/File	Line Coverage	Confirmation (assert/expect)
CourseExamExportService.java	90%	✅
ExerciseWithSubmissionsExportService.java	91%	✅
FileUploadExerciseWithSubmissionsExportService.java	100%	✅
ModelingExerciseWithSubmissionsExportService.java	100%	✅
TextExerciseWithSubmissionsExportService.java	100%	✅
ProgrammingExerciseExportService.java	85%	✅

Screenshots

[JohannesStoehr]

Approve code

[Xinyannnnnn]

Tested on TS3 and it works as expected.

[MaximilianAnzinger]

Code

[MaximilianAnzinger]

Code

[MaximilianAnzinger]

Manual: Tested on TS1. The only thing that didn't work was the archive of solution/template/test repositories. Everything else works like a charm.

[Strohgelaender]

code lgtm.

Tested on ts1, the issue mentioned above is fixed and everything else worked as expected.

[MaximilianAnzinger]

Code and manual on TS1

Thx for the quick fix 👍

[laurenzfb]

Tested on ts1, worked as described

[Strohgelaender]

the archive contains the correct information and the plagiarism checks also still work.

code looks good as well.

[jakubriegel]

Tested on ts4. Everything including export of the course, single prog. exercise and plagiarism detection works fine 👍🏻

[DominikRemo]

Tested on ts2. Works as expected