git: resolved should be a git+ssh:// url, not just ssh://

Author: isaacs

lib/git.js

@@ -23,9 +23,11 @@ const _prepareDir = Symbol('_prepareDir')
 
 // get the repository url.  prefer ssh, fall back to git://
 const repoUrl = (hosted, opts) =>
-  hosted.sshurl && hosted.sshurl(opts) ||
+  hosted.sshurl && addGitPlus(hosted.sshurl(opts)) ||
   hosted.git && hosted.git(opts)
 
+const addGitPlus = url => url && `git+${url}`
+
 class GitFetcher extends Fetcher {
   constructor (spec, opts) {
     super(spec, opts)

test/git.js

@@ -396,3 +396,10 @@ t.test('fetch a private repo where the tgz is not a tarball', t => {
     code: 'TAR_BAD_ARCHIVE',
   })
 })
+
+t.test('resolved is a git+ssh url for hosted repos that support it', t => {
+  const hash = '0000000000000000000000000000000000000000'
+  const gf = new GitFetcher(`github:x/y#${hash}`, {})
+  t.equal(gf.resolved, `git+ssh://git@github.com/x/y.git#${hash}`)
+  t.end()
+})