Development: Adjust server tests to restructuring into modules

src/main/java/de/tum/cit/aet/artemis/assessment/repository/ComplaintRepository.java

@@ -209,15 +209,6 @@ SELECT COUNT(c)
     @Modifying
     void deleteByResult_Id(long resultId);
 
-    /**
-     * Given a user id, retrieve all complaints related to assessments made by that assessor
-     *
-     * @param assessorId - the id of the assessor
-     * @return a list of complaints
-     */
-    @EntityGraph(type = LOAD, attributePaths = { "result.participation", "result.submission", "result.assessor" })
-    List<Complaint> getAllByResult_Assessor_Id(Long assessorId);
-
     /**
      * Given an exercise id, retrieve all complaints related to that exercise
      *

src/main/java/de/tum/cit/aet/artemis/assessment/repository/ComplaintResponseRepository.java

@@ -3,7 +3,6 @@
 import static de.tum.cit.aet.artemis.core.config.Constants.PROFILE_CORE;
 
 import java.util.List;
-import java.util.Optional;
 import java.util.Set;
 
 import org.springframework.context.annotation.Profile;
@@ -25,8 +24,6 @@
 @Repository
 public interface ComplaintResponseRepository extends ArtemisJpaRepository<ComplaintResponse, Long> {
 
-    Optional<ComplaintResponse> findByComplaint_Id(Long complaintId);
-
     /**
      * This magic method counts the number of complaints responses by complaint type associated to a course id
      *

src/main/java/de/tum/cit/aet/artemis/assessment/repository/ExampleSubmissionRepository.java

@@ -52,8 +52,6 @@ public interface ExampleSubmissionRepository extends ArtemisJpaRepository<Exampl
             """)
     Optional<ExampleSubmission> findByIdWithResultsAndFeedback(@Param("exampleSubmissionId") long exampleSubmissionId);
 
-    Optional<ExampleSubmission> findBySubmissionId(long submissionId);
-
     @EntityGraph(type = LOAD, attributePaths = { "submission", "submission.results" })
     Optional<ExampleSubmission> findWithResultsBySubmissionId(long submissionId);
 

src/main/java/de/tum/cit/aet/artemis/assessment/repository/ParticipantScoreRepository.java

@@ -10,8 +10,6 @@
 import java.util.Optional;
 import java.util.Set;
 
-import jakarta.validation.constraints.NotNull;
-
 import org.springframework.context.annotation.Profile;
 import org.springframework.data.jpa.repository.EntityGraph;
 import org.springframework.data.jpa.repository.Modifying;
@@ -46,11 +44,6 @@ public interface ParticipantScoreRepository extends ArtemisJpaRepository<Partici
             """)
     List<ParticipantScore> findAllOutdated();
 
-    @NotNull
-    @Override
-    @EntityGraph(type = LOAD, attributePaths = { "exercise", "lastResult", "lastRatedResult" })
-    List<ParticipantScore> findAll();
-
     @EntityGraph(type = LOAD, attributePaths = { "exercise", "lastResult", "lastRatedResult" })
     List<ParticipantScore> findAllByExercise(Exercise exercise);
 

src/main/java/de/tum/cit/aet/artemis/assessment/repository/ResultRepository.java

@@ -147,23 +147,6 @@ default Optional<Result> findFirstWithSubmissionAndFeedbacksAndTestCasesByPartic
     @EntityGraph(type = LOAD, attributePaths = "submission")
     Optional<Result> findResultWithSubmissionsById(long resultId);
 
-    /**
-     * Finds the first result by participation ID, including its submissions, ordered by completion date in descending order.
-     * This method avoids in-memory paging by retrieving the first result directly from the database.
-     *
-     * @param participationId the ID of the participation to find the result for
-     * @return an {@code Optional} containing the first {@code Result} with submissions, ordered by completion date in descending order,
-     *         or an empty {@code Optional} if no result is found
-     */
-    default Optional<Result> findFirstWithSubmissionsByParticipationIdOrderByCompletionDateDesc(long participationId) {
-        var resultOptional = findFirstByParticipationIdOrderByCompletionDateDesc(participationId);
-        if (resultOptional.isEmpty()) {
-            return Optional.empty();
-        }
-        var id = resultOptional.get().getId();
-        return findResultWithSubmissionsById(id);
-    }
-
     Optional<Result> findFirstByParticipationIdAndRatedOrderByCompletionDateDesc(long participationId, boolean rated);
 
     /**
@@ -184,11 +167,6 @@ default Optional<Result> findFirstByParticipationIdAndRatedWithSubmissionOrderBy
         return findResultWithSubmissionsById(id);
     }
 
-    Optional<Result> findDistinctBySubmissionId(long submissionId);
-
-    @EntityGraph(type = LOAD, attributePaths = "feedbacks")
-    Optional<Result> findDistinctWithFeedbackBySubmissionId(long submissionId);
-
     @Query("""
             SELECT r
             FROM Result r
@@ -208,8 +186,6 @@ default Optional<Result> findFirstByParticipationIdAndRatedWithSubmissionOrderBy
             """)
     Optional<Result> findByIdWithEagerFeedbacksAndAssessor(@Param("resultId") long resultId);
 
-    Set<Result> findAllByParticipationExerciseId(long exerciseId);
-
     /**
      * Load a result from the database by its id together with the associated submission, the list of feedback items, its assessor and assessment note.
      *
@@ -352,9 +328,6 @@ SELECT COUNT(r.id)
             """)
     List<Long> countNumberOfFinishedAssessmentsByExamIdIgnoreTestRuns(@Param("examId") long examId);
 
-    @EntityGraph(type = LOAD, attributePaths = { "feedbacks" })
-    Set<Result> findAllWithEagerFeedbackByAssessorIsNotNullAndParticipation_ExerciseIdAndCompletionDateIsNotNull(long exerciseId);
-
     @Query("""
             SELECT COUNT(DISTINCT p)
             FROM Participation p
@@ -805,10 +778,6 @@ default Optional<Result> findLatestResultWithFeedbacksForParticipation(long part
         }
     }
 
-    default Result findFirstWithFeedbacksByParticipationIdOrderByCompletionDateDescElseThrow(long participationId) {
-        return getValueElseThrow(findFirstWithFeedbacksTestCasesByParticipationIdOrderByCompletionDateDesc(participationId));
-    }
-
     default Result findWithBidirectionalSubmissionAndFeedbackAndAssessorAndAssessmentNoteAndTeamStudentsByIdElseThrow(long resultId) {
         return getValueElseThrow(findWithBidirectionalSubmissionAndFeedbackAndAssessorAndAssessmentNoteAndTeamStudentsById(resultId), resultId);
     }

src/main/java/de/tum/cit/aet/artemis/assessment/repository/TutorParticipationRepository.java

@@ -12,7 +12,6 @@
 import org.springframework.transaction.annotation.Transactional;
 
 import de.tum.cit.aet.artemis.assessment.domain.TutorParticipation;
-import de.tum.cit.aet.artemis.core.domain.Course;
 import de.tum.cit.aet.artemis.core.domain.User;
 import de.tum.cit.aet.artemis.core.repository.base.ArtemisJpaRepository;
 import de.tum.cit.aet.artemis.exercise.domain.Exercise;
@@ -24,8 +23,6 @@
 @Repository
 public interface TutorParticipationRepository extends ArtemisJpaRepository<TutorParticipation, Long> {
 
-    List<TutorParticipation> findByAssessedExercise(Exercise assessedExercise);
-
     @EntityGraph(type = LOAD, attributePaths = { "trainedExampleSubmissions", "trainedExampleSubmissions.submission.results" })
     TutorParticipation findWithEagerExampleSubmissionAndResultsByAssessedExerciseAndTutor(Exercise assessedExercise, User tutor);
 
@@ -34,8 +31,6 @@ public interface TutorParticipationRepository extends ArtemisJpaRepository<Tutor
     @EntityGraph(type = LOAD, attributePaths = { "trainedExampleSubmissions", "trainedExampleSubmissions.submission.results" })
     List<TutorParticipation> findAllByAssessedExercise_Course_IdAndTutor_Id(long courseId, long tutorId);
 
-    List<TutorParticipation> findAllByAssessedExercise_Course(Course course);
-
     @EntityGraph(type = LOAD, attributePaths = { "trainedExampleSubmissions", "trainedExampleSubmissions.submission.results" })
     List<TutorParticipation> findAllByAssessedExercise_ExerciseGroup_Exam_IdAndTutor_Id(long examId, long tutorId);
 

src/main/java/de/tum/cit/aet/artemis/atlas/repository/CompetencyProgressRepository.java

@@ -41,10 +41,6 @@ public interface CompetencyProgressRepository extends ArtemisJpaRepository<Compe
             """)
     Optional<CompetencyProgress> findByCompetencyIdAndUserId(@Param("competencyId") long competencyId, @Param("userId") long userId);
 
-    default CompetencyProgress findByCompetencyIdAndUserIdOrElseThrow(long competencyId, long userId) {
-        return getValueElseThrow(findByCompetencyIdAndUserId(competencyId, userId));
-    }
-
     @Query("""
             SELECT cp
             FROM CompetencyProgress cp

src/main/java/de/tum/cit/aet/artemis/atlas/repository/LearningPathRepository.java

@@ -36,10 +36,6 @@ default LearningPath findWithEagerUserByIdElseThrow(long learningPathId) {
     @EntityGraph(type = LOAD, attributePaths = { "competencies" })
     Optional<LearningPath> findWithEagerCompetenciesByCourseIdAndUserId(long courseId, long userId);
 
-    default LearningPath findWithEagerCompetenciesByCourseIdAndUserIdElseThrow(long courseId, long userId) {
-        return getValueElseThrow(findWithEagerCompetenciesByCourseIdAndUserId(courseId, userId));
-    }
-
     @EntityGraph(type = LOAD, attributePaths = { "course", "competencies" })
     Optional<LearningPath> findWithEagerCourseAndCompetenciesById(long learningPathId);
 

src/main/java/de/tum/cit/aet/artemis/atlas/repository/ScienceEventRepository.java

@@ -12,7 +12,6 @@
 import org.springframework.transaction.annotation.Transactional;
 
 import de.tum.cit.aet.artemis.atlas.domain.science.ScienceEvent;
-import de.tum.cit.aet.artemis.atlas.domain.science.ScienceEventType;
 import de.tum.cit.aet.artemis.core.repository.base.ArtemisJpaRepository;
 
 /**
@@ -22,8 +21,6 @@
 @Repository
 public interface ScienceEventRepository extends ArtemisJpaRepository<ScienceEvent, Long> {
 
-    Set<ScienceEvent> findAllByType(ScienceEventType type);
-
     @Transactional // ok because of modifying query
     @Modifying
     @Query("""

src/main/java/de/tum/cit/aet/artemis/atlas/web/AdminStandardizedCompetencyResource.java → src/main/java/de/tum/cit/aet/artemis/atlas/web/admin/AdminStandardizedCompetencyResource.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.artemis.atlas.web;
+package de.tum.cit.aet.artemis.atlas.web.admin;
 
 import static de.tum.cit.aet.artemis.core.config.Constants.PROFILE_CORE;
 

src/main/java/de/tum/cit/aet/artemis/communication/repository/ConversationParticipantRepository.java

@@ -64,10 +64,6 @@ public interface ConversationParticipantRepository extends ArtemisJpaRepository<
 
     Optional<ConversationParticipant> findConversationParticipantByConversationIdAndUserId(Long conversationId, Long userId);
 
-    default ConversationParticipant findConversationParticipantByConversationIdAndUserIdElseThrow(Long conversationId, Long userId) {
-        return getValueElseThrow(findConversationParticipantByConversationIdAndUserId(conversationId, userId));
-    }
-
     @Query("""
             SELECT DISTINCT conversationParticipant
             FROM ConversationParticipant conversationParticipant

src/main/java/de/tum/cit/aet/artemis/communication/repository/PostRepository.java

@@ -25,22 +25,6 @@ public interface PostRepository extends ArtemisJpaRepository<Post, Long>, JpaSpe
 
     List<Post> findPostsByAuthorId(long authorId);
 
-    /**
-     * find all posts of a user in a course
-     * currently only used for testing
-     *
-     * @param authorId id of the user
-     * @param courseId id of the course
-     * @return a list of posts
-     */
-    @Query("""
-            SELECT p
-            FROM Post p
-            WHERE p.author.id =:authorId
-                AND p.conversation.course.id = :courseId
-            """)
-    List<Post> findPostsByAuthorIdAndCourseId(@Param("authorId") long authorId, @Param("courseId") long courseId);
-
     @Transactional // ok because of delete
     @Modifying
     void deleteAllByConversationId(Long conversationId);

src/main/java/de/tum/cit/aet/artemis/communication/repository/ReactionRepository.java

@@ -17,9 +17,5 @@
 @Repository
 public interface ReactionRepository extends ArtemisJpaRepository<Reaction, Long> {
 
-    List<Reaction> findReactionsByPostId(Long postId);
-
     List<Reaction> findReactionsByUserId(long userId);
-
-    List<Reaction> findReactionsByAnswerPostId(Long answerPostId);
 }

src/main/java/de/tum/cit/aet/artemis/communication/repository/conversation/ChannelRepository.java

@@ -58,15 +58,6 @@ public interface ChannelRepository extends ArtemisJpaRepository<Channel, Long> {
             """)
     List<Channel> findChannelsOfUser(@Param("courseId") Long courseId, @Param("userId") Long userId);
 
-    @Query("""
-            SELECT DISTINCT channel
-            FROM Channel channel
-            WHERE channel.course.id = :courseId
-                AND channel.isCourseWide = TRUE
-            ORDER BY channel.name
-            """)
-    List<Channel> findCourseWideChannelsInCourse(@Param("courseId") long courseId);
-
     @Query("""
             SELECT DISTINCT channel
             FROM Channel channel

src/main/java/de/tum/cit/aet/artemis/communication/repository/conversation/ConversationRepository.java

@@ -27,9 +27,6 @@ public interface ConversationRepository extends ArtemisJpaRepository<Conversatio
     @Modifying
     void deleteAllByCourseId(long courseId);
 
-    // This is used only for testing purposes
-    List<Conversation> findAllByCourseId(long courseId);
-
     @EntityGraph(type = LOAD, attributePaths = { "conversationParticipants" })
     Optional<Conversation> findWithParticipantsById(long conversationId);
 

src/main/java/de/tum/cit/aet/artemis/communication/repository/conversation/OneToOneChatRepository.java

@@ -11,7 +11,6 @@
 import org.springframework.stereotype.Repository;
 
 import de.tum.cit.aet.artemis.communication.domain.conversation.OneToOneChat;
-import de.tum.cit.aet.artemis.core.exception.EntityNotFoundException;
 import de.tum.cit.aet.artemis.core.repository.base.ArtemisJpaRepository;
 
 @Profile(PROFILE_CORE)
@@ -73,15 +72,5 @@ AND EXISTS (
     // one-to-one chats userA has in that specific course.
     Optional<OneToOneChat> findWithParticipantsAndUserGroupsInCourseBetweenUsers(@Param("courseId") Long courseId, @Param("userIdA") Long userIdA, @Param("userIdB") Long userIdB);
 
-    @Query("""
-            SELECT DISTINCT oneToOneChat
-            FROM OneToOneChat oneToOneChat
-                LEFT JOIN FETCH oneToOneChat.conversationParticipants p
-                LEFT JOIN FETCH p.user u
-                LEFT JOIN FETCH u.groups
-            WHERE oneToOneChat.id = :oneToOneChatId
-            """)
-    Optional<OneToOneChat> findByIdWithConversationParticipantsAndUserGroups(@Param("oneToOneChatId") Long oneToOneChatId) throws EntityNotFoundException;
-
     Integer countByCreatorIdAndCourseId(Long creatorId, Long courseId);
 }

src/main/java/de/tum/cit/aet/artemis/communication/web/AdminSystemNotificationResource.java → src/main/java/de/tum/cit/aet/artemis/communication/web/admin/AdminSystemNotificationResource.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.artemis.communication.web;
+package de.tum.cit.aet.artemis.communication.web.admin;
 
 import static de.tum.cit.aet.artemis.core.config.Constants.PROFILE_CORE;
 

src/main/java/de/tum/cit/aet/artemis/core/config/CacheConfiguration.java

@@ -241,7 +241,6 @@ public HazelcastInstance hazelcastInstance(JHipsterProperties jHipsterProperties
         }
         config.getMapConfigs().put("default", initializeDefaultMapConfig(jHipsterProperties));
         config.getMapConfigs().put("files", initializeFilesMapConfig(jHipsterProperties));
-        // TODO Ole: investigate if the following * in artemis.*.domain works for database entity caching or if we need to specify all module domain paths here
         config.getMapConfigs().put("de.tum.cit.aet.artemis.*.domain.*", initializeDomainMapConfig(jHipsterProperties));
 
         // Configure split brain protection if the cluster was split at some point

src/main/java/de/tum/cit/aet/artemis/core/repository/CourseRepository.java

@@ -138,9 +138,6 @@ SELECT COUNT(c) > 0
             """)
     Optional<Course> findWithEagerLearningPathsAndLearningPathCompetencies(@Param("courseId") long courseId);
 
-    @EntityGraph(type = LOAD, attributePaths = { "competencies", "prerequisites", "learningPaths", "learningPaths.competencies" })
-    Optional<Course> findWithEagerLearningPathsAndCompetenciesAndPrerequisitesById(long courseId);
-
     // Note: we load attachments directly because otherwise, they will be loaded in subsequent DB calls due to the EAGER relationship
     @EntityGraph(type = LOAD, attributePaths = { "lectures", "lectures.attachments" })
     Optional<Course> findWithEagerLecturesById(long courseId);
@@ -488,11 +485,6 @@ default Course findWithEagerLearningPathsAndLearningPathCompetenciesByIdElseThro
         return getValueElseThrow(findWithEagerLearningPathsAndLearningPathCompetencies(courseId), courseId);
     }
 
-    @NotNull
-    default Course findWithEagerLearningPathsAndCompetenciesAndPrerequisitesByIdElseThrow(long courseId) {
-        return getValueElseThrow(findWithEagerLearningPathsAndCompetenciesAndPrerequisitesById(courseId), courseId);
-    }
-
     Page<Course> findByTitleIgnoreCaseContaining(String partialTitle, Pageable pageable);
 
     /**

src/main/java/de/tum/cit/aet/artemis/core/repository/DataExportRepository.java

@@ -67,11 +67,4 @@ public interface DataExportRepository extends ArtemisJpaRepository<DataExport, L
             ORDER BY dataExport.createdDate DESC
             """)
     List<DataExport> findAllDataExportsByUserIdOrderByRequestDateDesc(@Param("userId") long userId);
-
-    @Query("""
-            SELECT dataExport
-            FROM DataExport dataExport
-            WHERE dataExport.dataExportState = 2
-            """)
-    Set<DataExport> findAllSuccessfullyCreatedDataExports();
 }