fix(gitlab): add support for gitlab subgroups + escape user path

Author: canercandan

index.js

@@ -37,6 +37,17 @@ module.exports = function (string) {
   var m = /^\/([\w-]+)\/([\w-.]+)\/archive\/(.+)\.tar\.gz?$/.exec(path)
   if (m) return m.slice(1, 4)
 
+  // https://docs.gitlab.com/ce/user/group/subgroups/
+  if (~url.host.indexOf('gitlab')) {
+    var m = /^\/((?:[\w-.]+\/)+)([\w-.]+)$/.exec(path)
+    if (m) {
+      m = m.slice(1, 3);
+      // remove slash at the end
+      m[0] = m[0].slice(0, -1);
+      return m.concat((url.hash || '').slice(1));
+    }
+  }
+
   return false
 }
 

test.js

@@ -87,10 +87,40 @@ describe('gitlab urls', function () {
   })
 
   it('parses git gitlab url', function () {
-    var url = 'git@gitlab.team.com:user/test1.git'
+    var url = 'git@gitlab.com:user/test1.git'
     var parsed = parse(url)
     assert.deepEqual(['user', 'test1', ''], parsed)
   })
+
+  it('parses git gitlab url with one subgroup', function () {
+    var url = 'git@gitlab.com:user/subgroup/test1.git'
+    var parsed = parse(url)
+    assert.deepEqual(['user/subgroup', 'test1', ''], parsed)
+  })
+
+  it('parses git gitlab url with two nested subgroups', function () {
+    var url = 'git@gitlab.com:user/subgroup1/subgroup2/test1.git'
+    var parsed = parse(url)
+    assert.deepEqual(['user/subgroup1/subgroup2', 'test1', ''], parsed)
+  })
+
+  it('parses git gitlab url with three nested subgroups', function () {
+    var url = 'git@gitlab.com:user/subgroup1/subgroup2/subgroup3/test1.git'
+    var parsed = parse(url)
+    assert.deepEqual(['user/subgroup1/subgroup2/subgroup3', 'test1', ''], parsed)
+  })
+
+  it('parses git hosted gitlab url with subgroups', function () {
+    var url = 'git@gitlab.team.com:user/subgroup1/subgroup2/subgroup3/test1.git'
+    var parsed = parse(url)
+    assert.deepEqual(['user/subgroup1/subgroup2/subgroup3', 'test1', ''], parsed)
+  })
+
+  it('cannot parse subgroups in non-gitlab URLs', function () {
+    var url = 'git@stash.local:user/subgroup1/subgroup2/subgroup3/test1.git'
+    var parsed = parse(url)
+    assert.equal(false, parsed)
+  })
 })
 
 describe('git @ syntax', function () {