Development: Store shorter repository URI in database

src/main/java/de/tum/cit/aet/artemis/programming/domain/AbstractBaseProgrammingExerciseParticipation.java

@@ -9,6 +9,7 @@
 
 import de.tum.cit.aet.artemis.exercise.domain.Exercise;
 import de.tum.cit.aet.artemis.exercise.domain.participation.Participation;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 import de.tum.cit.aet.artemis.programming.service.localvc.LocalVCRepositoryUri;
 
 @MappedSuperclass
@@ -23,16 +24,16 @@ public abstract class AbstractBaseProgrammingExerciseParticipation extends Parti
 
     @Override
     public String getRepositoryUri() {
-        return repositoryUri;
+        return RepositoryUriConversionUtil.toFullRepositoryUri(repositoryUri);
     }
 
     @Override
     public void setRepositoryUri(String repositoryUri) {
-        this.repositoryUri = repositoryUri;
+        this.repositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
     }
 
     public void setRepositoryUri(@NotNull LocalVCRepositoryUri repositoryUri) {
-        this.repositoryUri = repositoryUri.getURI().toString();
+        this.repositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri.getURI().toString());
     }
 
     @Override

src/main/java/de/tum/cit/aet/artemis/programming/domain/AuxiliaryRepository.java

@@ -18,6 +18,7 @@
 
 import de.tum.cit.aet.artemis.core.domain.DomainObject;
 import de.tum.cit.aet.artemis.core.exception.localvc.LocalVCInternalException;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 import de.tum.cit.aet.artemis.programming.service.localvc.LocalVCRepositoryUri;
 
 @Entity
@@ -73,11 +74,11 @@ public class AuxiliaryRepository extends DomainObject {
     private ProgrammingExercise exercise;
 
     public String getRepositoryUri() {
-        return repositoryUri;
+        return RepositoryUriConversionUtil.toFullRepositoryUri(repositoryUri);
     }
 
     public void setRepositoryUri(String repositoryUri) {
-        this.repositoryUri = repositoryUri;
+        this.repositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
     }
 
     public String getCheckoutDirectory() {

src/main/java/de/tum/cit/aet/artemis/programming/domain/ProgrammingExercise.java

@@ -46,6 +46,7 @@
 import de.tum.cit.aet.artemis.programming.domain.build.BuildPlanType;
 import de.tum.cit.aet.artemis.programming.domain.submissionpolicy.SubmissionPolicy;
 import de.tum.cit.aet.artemis.programming.service.ProgrammingLanguageFeature;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 import de.tum.cit.aet.artemis.programming.service.localvc.LocalVCRepositoryUri;
 
 /**
@@ -190,11 +191,11 @@ public void setSolutionRepositoryUri(String solutionRepositoryUri) {
     }
 
     public void setTestRepositoryUri(String testRepositoryUri) {
-        this.testRepositoryUri = testRepositoryUri;
+        this.testRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(testRepositoryUri);
     }
 
     public String getTestRepositoryUri() {
-        return testRepositoryUri;
+        return RepositoryUriConversionUtil.toFullRepositoryUri(testRepositoryUri);
     }
 
     public List<AuxiliaryRepository> getAuxiliaryRepositories() {
@@ -471,10 +472,10 @@ public LocalVCRepositoryUri getVcsTestRepositoryUri() {
         }
 
         try {
-            return new LocalVCRepositoryUri(testRepositoryUri);
+            return new LocalVCRepositoryUri(getTestRepositoryUri());
         }
         catch (LocalVCInternalException e) {
-            log.warn("Cannot create URI for testRepositoryUri: {} due to the following error: {}", testRepositoryUri, e.getMessage());
+            log.warn("Cannot create URI for testRepositoryUri: {} due to the following error: {}", getTestRepositoryUri(), e.getMessage());
         }
         return null;
     }

src/main/java/de/tum/cit/aet/artemis/programming/domain/ProgrammingExerciseStudentParticipation.java

@@ -10,6 +10,7 @@
 
 import de.tum.cit.aet.artemis.exercise.domain.Exercise;
 import de.tum.cit.aet.artemis.exercise.domain.participation.StudentParticipation;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 import de.tum.cit.aet.artemis.programming.service.localvc.LocalVCRepositoryUri;
 
 @Entity
@@ -36,16 +37,16 @@ public ProgrammingExerciseStudentParticipation(String branch) {
 
     @Override
     public String getRepositoryUri() {
-        return repositoryUri;
+        return RepositoryUriConversionUtil.toFullRepositoryUri(repositoryUri);
     }
 
     @Override
     public void setRepositoryUri(String repositoryUri) {
-        this.repositoryUri = repositoryUri;
+        this.repositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
     }
 
     public void setRepositoryUri(@NotNull LocalVCRepositoryUri repositoryUri) {
-        this.repositoryUri = repositoryUri.getURI().toString();
+        this.repositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri.getURI().toString());
     }
 
     @Override

src/main/java/de/tum/cit/aet/artemis/programming/repository/ProgrammingExerciseStudentParticipationRepository.java

@@ -22,6 +22,7 @@
 
 import de.tum.cit.aet.artemis.core.repository.base.ArtemisJpaRepository;
 import de.tum.cit.aet.artemis.programming.domain.ProgrammingExerciseStudentParticipation;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 
 /**
  * Spring Data JPA repository for the Participation entity.
@@ -88,13 +89,15 @@ Optional<ProgrammingExerciseStudentParticipation> findByIdWithAllResultsAndRelat
     Optional<ProgrammingExerciseStudentParticipation> findWithSubmissionsByRepositoryUri(String repositoryUri);
 
     default ProgrammingExerciseStudentParticipation findWithSubmissionsByRepositoryUriElseThrow(String repositoryUri) {
-        return getValueElseThrow(findWithSubmissionsByRepositoryUri(repositoryUri));
+        String shortRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
+        return getValueElseThrow(findWithSubmissionsByRepositoryUri(shortRepositoryUri));
     }
 
     Optional<ProgrammingExerciseStudentParticipation> findByRepositoryUri(String repositoryUri);
 
     default ProgrammingExerciseStudentParticipation findByRepositoryUriElseThrow(String repositoryUri) {
-        return getValueElseThrow(findByRepositoryUri(repositoryUri));
+        String shortRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
+        return getValueElseThrow(findByRepositoryUri(shortRepositoryUri));
     }
 
     @EntityGraph(type = LOAD, attributePaths = { "team.students" })

src/main/java/de/tum/cit/aet/artemis/programming/repository/TemplateProgrammingExerciseParticipationRepository.java

@@ -25,6 +25,7 @@
 import de.tum.cit.aet.artemis.exercise.domain.Submission_;
 import de.tum.cit.aet.artemis.programming.domain.TemplateProgrammingExerciseParticipation;
 import de.tum.cit.aet.artemis.programming.domain.TemplateProgrammingExerciseParticipation_;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 
 /**
  * Spring Data JPA repository for the Participation entity.
@@ -49,13 +50,15 @@ public interface TemplateProgrammingExerciseParticipationRepository
     Optional<TemplateProgrammingExerciseParticipation> findWithSubmissionsByRepositoryUri(String repositoryUri);
 
     default TemplateProgrammingExerciseParticipation findWithSubmissionsByRepositoryUriElseThrow(String repositoryUri) {
-        return getValueElseThrow(findWithSubmissionsByRepositoryUri(repositoryUri));
+        String shortRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
+        return getValueElseThrow(findWithSubmissionsByRepositoryUri(shortRepositoryUri));
     }
 
     Optional<TemplateProgrammingExerciseParticipation> findByRepositoryUri(String repositoryUri);
 
     default TemplateProgrammingExerciseParticipation findByRepositoryUriElseThrow(String repositoryUri) {
-        return getValueElseThrow(findByRepositoryUri(repositoryUri));
+        String shortRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
+        return getValueElseThrow(findByRepositoryUri(shortRepositoryUri));
     }
 
     @EntityGraph(type = LOAD, attributePaths = { "submissions", "submissions.results", "submissions.results.feedbacks", "submissions.results.feedbacks.testCase" })

src/main/java/de/tum/cit/aet/artemis/programming/repository/VcsAccessLogRepository.java

@@ -14,6 +14,7 @@
 
 import de.tum.cit.aet.artemis.core.repository.base.ArtemisJpaRepository;
 import de.tum.cit.aet.artemis.programming.domain.VcsAccessLog;
+import de.tum.cit.aet.artemis.programming.service.RepositoryUriConversionUtil;
 
 /**
  * Spring Data JPA repository for the User entity.<br>
@@ -46,7 +47,7 @@ public interface VcsAccessLogRepository extends ArtemisJpaRepository<VcsAccessLo
     /**
      * Retrieves the most recent {@link VcsAccessLog} for a specific repository URI of a participation.
      *
-     * @param repositoryUri the URI of the participation to filter by.
+     * @param repositoryShortUri the short URI of the participation to filter by.
      * @return an Optional containing the newest {@link VcsAccessLog} of the participation, or empty if none exists.
      */
     @Query("""
@@ -57,7 +58,12 @@ LEFT JOIN TREAT (vcsAccessLog.participation AS ProgrammingExerciseStudentPartici
             ORDER BY vcsAccessLog.id DESC
             LIMIT 1
             """)
-    Optional<VcsAccessLog> findNewestByRepositoryUri(@Param("repositoryUri") String repositoryUri);
+    Optional<VcsAccessLog> findNewestByRepositoryUri(@Param("repositoryUri") String repositoryShortUri);
+
+    default Optional<VcsAccessLog> findNewestByFullRepositoryUri(String repositoryUri) {
+        String shortRepositoryUri = RepositoryUriConversionUtil.toShortRepositoryUri(repositoryUri);
+        return findNewestByRepositoryUri(shortRepositoryUri);
+    }
 
     /**
      * Retrieves a list of {@link VcsAccessLog} entities associated with the specified participation ID.

src/main/java/de/tum/cit/aet/artemis/programming/service/RepositoryUriConversionUtil.java

@@ -0,0 +1,143 @@
+package de.tum.cit.aet.artemis.programming.service;
+
+import static de.tum.cit.aet.artemis.core.config.Constants.PROFILE_LOCALVC;
+
+import jakarta.annotation.PostConstruct;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.springframework.beans.factory.annotation.Value;
+import org.springframework.context.annotation.Lazy;
+import org.springframework.context.annotation.Profile;
+import org.springframework.stereotype.Component;
+
+/**
+ * Utility class for converting between full (as expected in the domain) and short (stored in the database) repository URIs.
+ * Parallel-test safe:
+ * - No globally mutable server URL.
+ * - Supports per-thread overrides for tests that run multiple Spring contexts in parallel.
+ */
+@Component
+@Profile(PROFILE_LOCALVC)
+@Lazy
+public final class RepositoryUriConversionUtil {
+
+    private static final Logger log = LoggerFactory.getLogger(RepositoryUriConversionUtil.class);
+
+    private static final String GIT_PREFIX = "/git/";
+
+    private static final String GIT_SUFFIX = ".git";
+
+    /**
+     * Immutable default server URL captured from the (first) Spring context that initializes this bean.
+     * It is intentionally final and never mutated after init.
+     */
+    private static volatile String defaultServerUrl;
+
+    /**
+     * Per-thread override for tests. If set, static methods will use this value
+     * instead of {@link #defaultServerUrl}. This prevents cross-test leakage when
+     * different test threads have different Spring contexts / properties.
+     * Using {@link InheritableThreadLocal} so that child threads (e.g. in thread pools) also inherit the override.
+     * To be compatible with @Async methods, we introduced a test-profile decorator for the AsyncTaskExecutor {@link TestAsyncTaskExecutorDecoratorConfig}
+     */
+    private static final InheritableThreadLocal<String> threadServerUrlOverride = new InheritableThreadLocal<>();
+
+    @Value("${artemis.version-control.url}")
+    private String injectedServerUrl;
+
+    @PostConstruct
+    private void init() {
+        String normalized = normalizeBaseUrl(injectedServerUrl);
+        // Capture the first initialized value only; don't overwrite on later contexts
+        // (production has one context; in tests, use per-thread override below).
+        if (defaultServerUrl == null) {
+            defaultServerUrl = normalized;
+            log.debug("RepositoryUriConversionUtil default server URL set to {}", defaultServerUrl);
+        }
+        else {
+            log.debug("RepositoryUriConversionUtil already initialized with default server URL: {}", defaultServerUrl);
+        }
+    }
+
+    /**
+     * Converts a full repository URI to a short URI for database storage.
+     * Removes the server URL, /git/ prefix, and .git suffix.
+     *
+     * @param fullUri the full repository URI (e.g., "https://artemis.tum.de/git/project/exercise-name.git")
+     * @return the short URI (e.g., "project/exercise-name") or null if the input is null/empty
+     */
+
+    public static String toShortRepositoryUri(String fullUri) {
+        if (isNullOrEmpty(fullUri)) {
+            return null;
+        }
+        String uri = fullUri;
+
+        int gitPrefix = uri.indexOf(GIT_PREFIX);
+        if (gitPrefix == -1) {
+            throw new IllegalArgumentException("The provided repository URI does not contain the expected '" + GIT_PREFIX + "' segment: " + fullUri);
+        }
+        uri = uri.substring(gitPrefix + GIT_PREFIX.length());
+
+        if (uri.endsWith(GIT_SUFFIX)) {
+            uri = uri.substring(0, uri.length() - GIT_SUFFIX.length());
+        }
+
+        return uri;
+    }
+
+    /**
+     * Converts a short repository URI to a full URI for domain logic usage.
+     * Adds the server URL, /git/ prefix, and .git suffix.
+     *
+     * @param shortUri the short repository URI (e.g., "project/exercise-name")
+     * @return the full URI (e.g., "https://artemis.tum.de/git/project/exercise-name.git") or null if the input is null/empty
+     */
+    public static String toFullRepositoryUri(String shortUri) {
+        if (isNullOrEmpty(shortUri)) {
+            return null;
+        }
+        String base = currentServerUrl();
+        if (base == null) {
+            throw new IllegalStateException("Server URL is not configured. Cannot convert to full repository URI.");
+        }
+        return base + GIT_PREFIX + shortUri + GIT_SUFFIX;
+    }
+
+    /**
+     * Override the server URL for the current thread (e.g., in tests that run in parallel).
+     * Call in @BeforeEach (and clear in @AfterEach) of your test class.
+     *
+     * @param serverUrl the server URL to use for this thread (e.g., "http://localhost:49152")
+     * @see #clearServerUrlOverrideForCurrentThread()
+     */
+    public static void overrideServerUrlForCurrentThread(String serverUrl) {
+        threadServerUrlOverride.set(normalizeBaseUrl(serverUrl));
+    }
+
+    /**
+     * Clear the per-thread override. Call in @AfterEach.
+     *
+     * @see #overrideServerUrlForCurrentThread(String)
+     */
+    public static void clearServerUrlOverrideForCurrentThread() {
+        threadServerUrlOverride.remove();
+    }
+
+    private static String currentServerUrl() {
+        String override = threadServerUrlOverride.get();
+        return override != null ? override : defaultServerUrl;
+    }
+
+    private static String normalizeBaseUrl(String url) {
+        if (url == null) {
+            return null;
+        }
+        return url.replaceAll("/+$", "");
+    }
+
+    private static boolean isNullOrEmpty(String uriString) {
+        return uriString == null || uriString.isEmpty();
+    }
+}

src/main/java/de/tum/cit/aet/artemis/programming/service/localvc/VcsAccessLogService.java

@@ -83,7 +83,7 @@ public void updateCommitHash(ProgrammingExerciseParticipation participation, Str
     @Async
     public void updateRepositoryActionType(LocalVCRepositoryUri localVCRepositoryUri, RepositoryActionType repositoryActionType) {
         var repositoryURL = localVCRepositoryUri.toString().replace("/git-upload-pack", "").replace("/git-receive-pack", "");
-        var vcsAccessLog = vcsAccessLogRepository.findNewestByRepositoryUri(repositoryURL);
+        var vcsAccessLog = vcsAccessLogRepository.findNewestByFullRepositoryUri(repositoryURL);
         if (vcsAccessLog.isPresent()) {
             vcsAccessLog.get().setRepositoryActionType(repositoryActionType);
             vcsAccessLogRepository.save(vcsAccessLog.get());