[BUGFIX beta] Don't capitalize characters following multiple forward …

…slashes
emberjs · Apr 5, 2019 · 9835b3e · 9835b3e
1 parent ad830c6
commit 9835b3e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/packages/@ember/string/index.ts b/packages/@ember/string/index.ts
@@ -53,7 +53,7 @@ const UNDERSCORE_CACHE = new Cache<string, string>(1000, str =>
     .toLowerCase()
 );
 
-const STRING_CAPITALIZE_REGEXP = /(^|\/)([a-z\u00C0-\u024F])/g;
+const STRING_CAPITALIZE_REGEXP = /(^|(?<!\/)\/(?!\/))([a-z\u00C0-\u024F])/g;
 
 const CAPITALIZE_CACHE = new Cache<string, string>(1000, str =>
   str.replace(STRING_CAPITALIZE_REGEXP, (match /*, separator, chr */) => match.toUpperCase())

diff --git a/packages/@ember/string/tests/capitalize_test.js b/packages/@ember/string/tests/capitalize_test.js
@@ -52,6 +52,12 @@ moduleFor(
         'Private-docs/Owner-invoice',
         'capitalize namespaced dasherized string'
       );
+      test(
+        assert,
+        'url is http://www.emberjs.com, not http:///www.emberjs.com',
+        'Url is http://www.emberjs.com, not http:///www.emberjs.com',
+        'does nothing with multiple slashes'
+      );
       test(assert, 'šabc', 'Šabc', 'capitalize string with accent character');
     }
   }