`Programming Exercises`: Add basic service implementation for HadesCI - Githubissues

ls1intum / Artemis

Artemis - Interactive Learning with Automated Feedback

https://docs.artemis.cit.tum.de

MIT License

452 stars 275 forks source link

`Programming Exercises`: Add basic service implementation for HadesCI #8891

Open magkue opened 3 days ago

magkue commented 3 days ago

Checklist

General

[x] 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.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] 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 documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[x] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).
[ ] I tested all changes and their related features with all corresponding user types on a test server configured with Gitlab and Jenkins.

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Testserver States

[!NOTE] These badges show the state of the test servers. Green = Currently available, Red = Currently locked Click on the badges to get to the test servers.

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
Exam Mode Test
[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots