MORE-Platform#385: Looks like this bug was possible because the assertion in the relevant unit test was the wrong way!

Author: Rupert Westenthaler

studymanager-services/src/test/java/io/redlink/more/studymanager/repository/goals/GoalConfigurationRepositoryTest.java

@@ -322,10 +322,12 @@ void testDoImportGoalTopic() {
         GoalTopic reImported = goalConfigurationRepository.doImport(studyB, updatedVersion);
 
         assertThat(reImported.getStudyId()).isEqualTo(studyB);
-        assertThat(reImported.getTitle()).isEqualTo("Updated - Drink More Water");
+        //it is expected that it is NOT overridden if already existing. So the old title is expected
+        assertThat(reImported.getTitle()).isEqualTo("Drink Water");
 
+        //validate also directly in the DB
         assertThat(goalConfigurationRepository.getTopic(studyB, "hydration").getTitle())
-                .isEqualTo("Updated - Drink More Water");
+                .isEqualTo("Drink Water");
     }
 
     @Test