test: moved all github url tests together

Author: Michael Perrotte

test/github.js

@@ -292,5 +292,46 @@ test('fromUrl(github url)', function (t) {
 
   t.equal(HostedGit.fromUrl('git+ssh://github.com/foo.git'), undefined)
 
+  t.test('HTTPS GitHub URL with embedded auth -- generally not a good idea', function (tt) {
+    function verify (host, label, branch) {
+      var hostinfo = HostedGit.fromUrl(host)
+      var hash = branch ? '#' + branch : ''
+      tt.ok(hostinfo, label)
+      if (!hostinfo) return
+      tt.is(hostinfo.https(), 'git+https://user:pass@github.com/111/222.git' + hash, label + ' -> https')
+      tt.is(hostinfo.git(), 'git://user:pass@github.com/111/222.git' + hash, label + ' -> git')
+      tt.is(hostinfo.browse(), 'https://github.com/111/222' + (branch ? '/tree/' + branch : ''), label + ' -> browse')
+      tt.is(hostinfo.browse('C'), 'https://github.com/111/222/tree/' + (branch || 'master') + '/C', label + ' -> browse(path)')
+      tt.is(hostinfo.browse('C/D'), 'https://github.com/111/222/tree/' + (branch || 'master') + '/C/D', label + ' -> browse(path)')
+      tt.is(hostinfo.browse('C', 'A'), 'https://github.com/111/222/tree/' + (branch || 'master') + '/C#a', label + ' -> browse(path, fragment)')
+      tt.is(hostinfo.browse('C/D', 'A'), 'https://github.com/111/222/tree/' + (branch || 'master') + '/C/D#a', label + ' -> browse(path, fragment)')
+      tt.is(hostinfo.bugs(), 'https://github.com/111/222/issues', label + ' -> bugs')
+      tt.is(hostinfo.docs(), 'https://github.com/111/222' + (branch ? '/tree/' + branch : '') + '#readme', label + ' -> docs')
+      tt.is(hostinfo.ssh(), 'git@github.com:111/222.git' + hash, label + ' -> ssh')
+      tt.is(hostinfo.sshurl(), 'git+ssh://git@github.com/111/222.git' + hash, label + ' -> sshurl')
+      tt.is(hostinfo.shortcut(), 'github:111/222' + hash, label + ' -> shortcut')
+      tt.is(hostinfo.file('C'), 'https://user:pass@raw.githubusercontent.com/111/222/' + (branch || 'master') + '/C', label + ' -> file')
+      tt.is(hostinfo.file('C/D'), 'https://user:pass@raw.githubusercontent.com/111/222/' + (branch || 'master') + '/C/D', label + ' -> file')
+    }
+
+    // insecure protocols
+    verify('git://user:pass@github.com/111/222', 'git')
+    verify('git://user:pass@github.com/111/222.git', 'git.git')
+    verify('git://user:pass@github.com/111/222#branch', 'git#branch', 'branch')
+    verify('git://user:pass@github.com/111/222.git#branch', 'git.git#branch', 'branch')
+
+    verify('https://user:pass@github.com/111/222', 'https')
+    verify('https://user:pass@github.com/111/222.git', 'https.git')
+    verify('https://user:pass@github.com/111/222#branch', 'https#branch', 'branch')
+    verify('https://user:pass@github.com/111/222.git#branch', 'https.git#branch', 'branch')
+
+    verify('http://user:pass@github.com/111/222', 'http')
+    verify('http://user:pass@github.com/111/222.git', 'http.git')
+    verify('http://user:pass@github.com/111/222#branch', 'http#branch', 'branch')
+    verify('http://user:pass@github.com/111/222.git#branch', 'http.git#branch', 'branch')
+
+    tt.end()
+  })
+
   t.end()
 })