commit b89ae948646625812b308b902a617ea63317a880
author Antoine Musso <hashar@free.fr> Tue Sep 30 19:31:35 2025 +0200
committer Antoine Musso <hashar@free.fr> Tue Sep 30 19:39:52 2025 +0200
tree 5fa02b6b07c98990dfa8815cac0daeb79f3efa4f
parent c0094a91f4022f397396c486b5fdc6f814ae51af

Tests: update path prefix in git urls

When testing URLs, it is helpfull to distinguish between the base URL
(which can contains a path prefix such as `/r`) and the application
paths that is appended to it.

Using `/foo` is a bit meaningless, replace it by `/path-prefix` which is
self describing and helps differentiating between the base URL and the
rest of the path.

Change-Id: I91230750cc67b7a2876e4da7779a0b0e20f62639

javatests/com/google/gitiles/LinkifierTest.java

diff --git a/javatests/com/google/gitiles/LinkifierTest.java b/javatests/com/google/gitiles/LinkifierTest.java
index bcd82a4..a51eccd 100644
--- a/javatests/com/google/gitiles/LinkifierTest.java
+++ b/javatests/com/google/gitiles/LinkifierTest.java
@@ -104,19 +104,19 @@
     assertThat(l.linkify(REQ, "I0123456789"))
         .containsExactly(
             ImmutableMap.of(
-                "text", "I0123456789", "url", "http://test-host-review/foo/#/q/I0123456789"))
+                "text", "I0123456789", "url", "http://test-host-review/path-prefix/#/q/I0123456789"))
         .inOrder();
     assertThat(l.linkify(REQ, "Change-Id: I0123456789"))
         .containsExactly(
             ImmutableMap.of("text", "Change-Id: "),
             ImmutableMap.of(
-                "text", "I0123456789", "url", "http://test-host-review/foo/#/q/I0123456789"))
+                "text", "I0123456789", "url", "http://test-host-review/path-prefix/#/q/I0123456789"))
         .inOrder();
     assertThat(l.linkify(REQ, "Change-Id: I0123456789 exists"))
         .containsExactly(
             ImmutableMap.of("text", "Change-Id: "),
             ImmutableMap.of(
-                "text", "I0123456789", "url", "http://test-host-review/foo/#/q/I0123456789"),
+                "text", "I0123456789", "url", "http://test-host-review/path-prefix/#/q/I0123456789"),
             ImmutableMap.of("text", " exists"))
         .inOrder();
   }
@@ -149,7 +149,7 @@
                 "text", "http://my/url/I0123456789", "url", "http://my/url/I0123456789"),
             ImmutableMap.of("text", " is not change "),
             ImmutableMap.of(
-                "text", "I0123456789", "url", "http://test-host-review/foo/#/q/I0123456789"))
+                "text", "I0123456789", "url", "http://test-host-review/path-prefix/#/q/I0123456789"))
         .inOrder();
   }
 

javatests/com/google/gitiles/TestGitilesUrls.java

diff --git a/javatests/com/google/gitiles/TestGitilesUrls.java b/javatests/com/google/gitiles/TestGitilesUrls.java
index 29cc014..7f838ee 100644
--- a/javatests/com/google/gitiles/TestGitilesUrls.java
+++ b/javatests/com/google/gitiles/TestGitilesUrls.java
@@ -28,12 +28,12 @@
 
   @Override
   public String getBaseGitUrl(HttpServletRequest req) {
-    return "git://" + HOST_NAME + "/foo";
+    return "git://" + HOST_NAME + "/path-prefix";
   }
 
   @Override
   public String getBaseGerritUrl(HttpServletRequest req) {
-    return "http://" + HOST_NAME + "-review/foo/";
+    return "http://" + HOST_NAME + "-review/path-prefix/";
   }
 
   private TestGitilesUrls() {}