From b16f5c1638e83a4e91e574364797ffdefd5f0595 Mon Sep 17 00:00:00 2001
From: Lucas Welscher <ga53foy@mytum.de>
Date: Mon, 4 Dec 2023 10:18:57 +0100
Subject: [PATCH] Programming exercises: Fix visible testid if the task name is
 also a test name (#7703)

---
 .../ProgrammingExerciseTaskService.java       |  7 +++---
 .../ProgrammingExerciseTaskServiceTest.java   | 22 ++++++++++++++++++-
 2 files changed, 24 insertions(+), 5 deletions(-)

diff --git a/src/main/java/de/tum/in/www1/artemis/service/hestia/ProgrammingExerciseTaskService.java b/src/main/java/de/tum/in/www1/artemis/service/hestia/ProgrammingExerciseTaskService.java
index f743cd4a3e11..78555242aea5 100644
--- a/src/main/java/de/tum/in/www1/artemis/service/hestia/ProgrammingExerciseTaskService.java
+++ b/src/main/java/de/tum/in/www1/artemis/service/hestia/ProgrammingExerciseTaskService.java
@@ -432,16 +432,15 @@ private String replaceTaskTests(String problemStatement, Set<ProgrammingExercise
         Matcher matcher = TASK_PATTERN.matcher(problemStatement);
 
         return matcher.replaceAll(matchResult -> {
-            // matchResult is fa full task, e.g., [task][Bubble Sort](testBubbleSort,testClass[BubbleSort])
-            String fullMatch = matchResult.group();
             // group 1: task name, group 2: test names, e.g, testBubbleSort,testClass[BubbleSort]
+            String taskName = matchResult.group(1);
             String testNames = matchResult.group(2);
 
             // converted testids, e.g., <testid>10</testid>,<testid>12</testid>
             String testIds = replacer.apply(testNames, testCases);
 
-            // replace the names with their ids
-            return fullMatch.replace(testNames, testIds);
+            // construct a new task using the testids
+            return "[task][%s](%s)".formatted(taskName, testIds);
         });
     }
 
diff --git a/src/test/java/de/tum/in/www1/artemis/hestia/ProgrammingExerciseTaskServiceTest.java b/src/test/java/de/tum/in/www1/artemis/hestia/ProgrammingExerciseTaskServiceTest.java
index 2e607ae0b996..2eb63fc57e91 100644
--- a/src/test/java/de/tum/in/www1/artemis/hestia/ProgrammingExerciseTaskServiceTest.java
+++ b/src/test/java/de/tum/in/www1/artemis/hestia/ProgrammingExerciseTaskServiceTest.java
@@ -381,10 +381,30 @@ void testNameReplacementOnlyWithinTasks() {
         programmingExerciseTaskService.replaceTestNamesWithIds(programmingExercise);
         String problemStatement = programmingExercise.getProblemStatement();
 
-        assertThat(problemStatement).contains("[task][Taskname](<testid>%s</testid>,<testid>%s</testid>".formatted(test1.getId(), test2.getId()))
+        assertThat(problemStatement).contains("[task][Taskname](<testid>%s</testid>,<testid>%s</testid>)".formatted(test1.getId(), test2.getId()))
                 .contains("This description contains the words test and taskTest, which should not be replaced.");
     }
 
+    @Test
+    void testNameReplacementTaskNameSameAsTestName() {
+        var sort = programmingExerciseUtilService.addTestCaseToProgrammingExercise(programmingExercise, "sort");
+
+        updateProblemStatement("""
+                [task][sort](sort)
+                Sort using the method sort.
+                @startuml
+                class LinkedList<T> {
+                    <color:testsColor(sort)>+ sort()</color>
+                }
+                @enduml""");
+
+        programmingExerciseTaskService.replaceTestNamesWithIds(programmingExercise);
+        String problemStatement = programmingExercise.getProblemStatement();
+
+        assertThat(problemStatement).contains("[task][sort](<testid>%s</testid>)".formatted(sort.getId())).contains("Sort using the method sort.")
+                .contains("<color:testsColor(<testid>%s</testid>)>+ sort()</color>".formatted(sort.getId()));
+    }
+
     @Test
     void testIdReplacementWithNames() {
         var bubbleSort = programmingExerciseTestCaseRepository.findByExerciseIdAndTestName(programmingExercise.getId(), "testClass[BubbleSort]").orElseThrow();