added test for matching the url. Fixes #68

Author: mirceaalexandru

lib/github.js

@@ -30,6 +30,7 @@ function cmdGet (msg, done) {
   var cache = opts.cache
   var registry = opts.registry + msg.name
 
+  console.log('Processing module: ', msg.name)
   cache.load$(msg.name, (err, github) => {
     if (err) return done(err)
 
@@ -51,19 +52,25 @@ function cmdGet (msg, done) {
       var url = repository.url || ''
 
       if(url.length > 0) {
+        console.log('Matching URL: ', url)
         var matches = /[\/:]([^\/:]+?)[\/:]([^\/]+?)(\.git)*$/.exec(url)
-        var params = {
-          name: msg.name,
-          url: url,
-          user: matches[1] || null,
-          repo: matches[2] || null
-        }
 
-        if (!params.user || !params.repo) {
-          return done(new Error('not found on github'))
+        if (matches && matches.length ===2) {
+          var params = {
+            name: msg.name,
+            url: url,
+            user: matches[1] || null,
+            repo: matches[2] || null
+          }
+
+          if (!params.user || !params.repo) {
+            return done(new Error('not found on github'))
+          }
+
+          queryGithub(params, done)
+        }else{
+          return done(new Error('invalid github url:', url, ' for module:', msg.name))
         }
-
-        queryGithub(params, done)
       }
       else{
         return done(new Error('not found on github'))