Fixes impractical comment line breaks

ls1intum · Aug 5, 2024 · 448249c · 448249c
1 parent d5b9d28
commit 448249c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/src/test/java/de/tum/in/www1/artemis/science/ScienceUtilService.java b/src/test/java/de/tum/in/www1/artemis/science/ScienceUtilService.java
@@ -36,8 +36,7 @@ public ScienceEvent createScienceEvent(String identity, ScienceEventType type, L
 
     /**
      * Comparator for comparing two science events.
-     * Allows for a 500ns difference between the timestamps due to the reimport from
-     * the csv export.
+     * Allows for a 500ns difference between the timestamps due to the reimport from the csv export.
      */
     public static Comparator<ScienceEvent> scienceEventComparator = Comparator.comparing(ScienceEvent::getResourceId).thenComparing(ScienceEvent::getType)
             .thenComparing((ScienceEvent e1, ScienceEvent e2) -> {

diff --git a/src/test/java/de/tum/in/www1/artemis/service/DataExportCreationServiceTest.java b/src/test/java/de/tum/in/www1/artemis/service/DataExportCreationServiceTest.java
@@ -230,14 +230,10 @@ private void assertCommunicationDataCsvFile(Path courseDirPath) {
 
     /**
      * Asserts the content of the science events CSV file.
-     * Allows for a 500ns difference between the timestamps due to the reimport from
-     * the csv export.
-     * Might cause the test to be flaky if multiple events are created overlapping
-     * and matched wrongly.
+     * Allows for a 500ns difference between the timestamps due to the reimport from the csv export.
      *
      * @param extractedZipDirPath The path to the extracted zip directory
-     * @param events              The set of science events to compare with the
-     *                                content of the CSV file
+     * @param events              The set of science events to compare with the content of the CSV file
      */
     private void assertScienceEventsCSVFile(Path extractedZipDirPath, Set<ScienceEvent> events) {
         assertThat(extractedZipDirPath).isDirectoryContaining(path -> "science_events.csv".equals(path.getFileName().toString()));
@@ -410,8 +406,7 @@ private void assertCorrectContentForExercise(Path exerciseDirPath, boolean cours
                         .isDirectoryContaining(path -> path.getFileName().toString().contains("plagiarism_case") && path.getFileName().toString().endsWith(FILE_FORMAT_CSV));
             }
         }
-        // only include automatic test feedback if the assessment due date is in the
-        // future
+        // only include automatic test feedback if the assessment due date is in the future
         if (exerciseDirPath.toString().contains("Programming") && assessmentDueDateInTheFuture && courseExercise) {
             var fileContentResult1 = Files.readString(getProgrammingResultsFilePath(exerciseDirPath, true));
             // automatic feedback
@@ -644,7 +639,7 @@ void testDataExportContainsDataAboutCourseStudentUnenrolled() throws Exception {
         var course = prepareCourseDataForDataExportCreation(assessmentDueDateInTheFuture, courseShortName);
         conversationUtilService.addOneMessageForUserInCourse(TEST_PREFIX + "student1", course, "only one post");
         var dataExport = initDataExport();
-        // by setting the course groups to a different value we simulate unenrollment
+        // by setting the course groups to a different value, we simulate unenrollment
         // because the user is no longer part of the user group and hence, the course.
         courseUtilService.updateCourseGroups("abc", course, "");
         dataExportCreationService.createDataExport(dataExport);