Development: Refactor ExamIntegrationTest

[laadvo]

Checklist

General

• 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

• I followed the coding and design guidelines.
• I documented the Java code using JavaDoc style.

Motivation and Context

The class ExamIntegrationTest has over 3000 lines of code and around 150 tests and can be considered to be a blob. This antipattern also causes the integration test to be slow, since many attributes (sometimes only used in a few tests) to be initialized and the setup is expensive. In this PR I moved some of the tests, previously found in `ExamIntegrationTest to new and already existing integration tests.

Description

The following new classes were created:

• ExamRegistrationIntegrationTest: student registration for exams, and removing students from exams
• ExamParticipationIntegrationTest: testing related to participation
• TestExamIntegrationTest: handels test exam related tests
• ProgrammingExamIntegrationTest: Houses tests directly linked to pro- gramming exercises incorporated within exams.
• ExamStartTest: Testing exam exercise start for different types of exercises.

Tests which fit into one of these classes have been moved, in ExamIntegrationTest remain exam creation, access, importing and other tests.

Review Progress

Code Review

• Code Review 1
• Code Review 2

Test Coverage

unchanged

[DominikRemo]

Test changes look mostly good to me. Left one small suggestion ^^

[DominikRemo]

Changes LGTM. Tests succeed on bamboo and GitHub Actions 👍
The number of tests on bamboo did not change, indicating that no test was lost

[terlan98]

Changes look good in general 👍 Left some small comments/suggestions:

• Shouldn't it be moved to ExamRegistrationIntegrationTest?

  Artemis/src/test/java/de/tum/in/www1/artemis/exam/ExamIntegrationTest.java

  Lines 755 to 760 in 69cb7cd

  	@Test
  	@WithMockUser(username = TEST_PREFIX + "instructor1", roles = "INSTRUCTOR")
  	void testDeleteStudentThatDoesNotExist() throws Exception {
  	Exam exam = examUtilService.setupExamWithExerciseGroupsExercisesRegisteredStudents(TEST_PREFIX, course1, 1);
  	request.delete("/api/courses/" + course1.getId() + "/exams/" + exam.getId() + "/students/nonExistingStudent", HttpStatus.NOT_FOUND);
  	}

[terlan98]

Code looks good. Thanks for the changes 👍

[DominikRemo]

LGTM 👍