SONAR-3718 Add support for '/' character in branch names

diff --git a/sonar-core/src/main/java/org/sonar/core/component/ComponentKeys.java b/sonar-core/src/main/java/org/sonar/core/component/ComponentKeys.java
index 67d2f112b3cffb9bb6653342ab196bd221c8c7d1..615fe1304b43c6eac81fad7cda3fd422fcd58f39 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/component/ComponentKeys.java
+++ b/sonar-core/src/main/java/org/sonar/core/component/ComponentKeys.java
@@ -35,9 +35,9 @@ public final class ComponentKeys {
    */
   private static final String VALID_MODULE_KEY_REGEXP = "[\\p{Alnum}\\-_.:]*[\\p{Alpha}\\-_.:]+[\\p{Alnum}\\-_.:]*";
   /*
-   * Allowed characters are alphanumeric, '-', '_' and '.'
+   * Allowed characters are alphanumeric, '-', '_', '.' and '/'
    */
-  private static final String VALID_BRANCH_REGEXP = "[\\p{Alnum}\\-_.]*";
+  private static final String VALID_BRANCH_REGEXP = "[\\p{Alnum}\\-_./]*";
 
   private static final String KEY_WITH_BRANCH_FORMAT = "%s:%s";
 
@@ -101,7 +101,7 @@ public final class ComponentKeys {
    *      <li>Uppercase ASCII letters A-Z</li>
    *      <li>Lowercase ASCII letters a-z</li>
    *      <li>ASCII digits 0-9</li>
-   *      <li>Punctuation signs dash '-', underscore '_', and period '.'</li>
+   *      <li>Punctuation signs dash '-', underscore '_', period '.', and '/'</li>
    *    </ul>
    *  </li>
    * </ul>

diff --git a/sonar-core/src/test/java/org/sonar/core/component/ComponentKeysTest.java b/sonar-core/src/test/java/org/sonar/core/component/ComponentKeysTest.java
index 3c4d0e78063dc47e87ece6f8442d808c94952de3..8b8c33025cccd47e26671a7b8803537b1c1f6732 100644 (file)
--- a/sonar-core/src/test/java/org/sonar/core/component/ComponentKeysTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/component/ComponentKeysTest.java
@@ -62,7 +62,8 @@ public class ComponentKeysTest {
     assertThat(ComponentKeys.isValidBranch("0123")).isTrue();
     assertThat(ComponentKeys.isValidBranch("ab 12")).isFalse();
     assertThat(ComponentKeys.isValidBranch("ab_12")).isTrue();
-    assertThat(ComponentKeys.isValidBranch("ab/12")).isFalse();
+    assertThat(ComponentKeys.isValidBranch("ab/12")).isTrue();
+    assertThat(ComponentKeys.isValidBranch("ab\\12")).isFalse();
   }
 
 }