Fixed missing error checks

We were not checking for errors and passing them to callbacks in
api.js in three places:

In
  get()
  removeDeployKey()
  deleteHooks()

This was causing 8 of the unit tests to fail.
Fixed.

Author: saraf

lib/api.js

@@ -2,6 +2,7 @@ var _ = require('lodash');
 var async = require('async');
 var superagent = require('superagent');
 var debug = require('debug')('strider-gitlab:api');
+var util = require('util');
 
 module.exports = {
   get: get,
@@ -39,8 +40,13 @@ function get(config, uri, done) {
     .set('User-Agent', 'StriderCD (http://stridercd.com)')
     .end(function (error, res) {
       res = res || {};
+
+      if (error) {
+        return done(error, null, null);
+      }
+
       debug('Response body type: ' + typeof res.body);
-      debug('Response body received: ' + JSON.stringify(res.body));
+      debug('Response body received: ' + util.inspect(res.body, false, 10, true));
       if (res.error) {
         return done(res.error, null, res);
       }
@@ -103,6 +109,10 @@ function deleteHooks(config, repo_id, url, callback) {
     .end(function (err, res) {
       var deleted = false;
 
+      if (err) {
+        return callback(err);
+      }
+
       if (!res) {
         return callback(new Error("Empty response."));
       }
@@ -156,6 +166,11 @@ function removeDeployKey(config, repo_id, title, callback) {
     .query(qpm)
     .set('User-Agent', 'StriderCD (http://stridercd.com)')
     .end(function (err, res) {
+
+      if (err) {
+        return callback(err);
+      }
+
       var deleted = false;
 
       if (!res) {

lib/webapp.js

@@ -30,8 +30,8 @@ module.exports = {
     api.get(config, 'projects', function (err, repos) {
       if (err) return done(err);
 
-      if(!repos) {
-          return done(new Error("Could not get a list of projects from the GitLab repository. Please check the configuration in Account->GitLab."));
+      if (!repos) {
+        return done(new Error("Could not get a list of projects from the GitLab repository. Please check the configuration in Account->GitLab."));
       }
 
       // Parse and filter only git repos
@@ -59,7 +59,7 @@ module.exports = {
 
     debug("Getting URI " + uri);
     api.get(account, uri, function (err, branches) {
-    debug("We get the following as branches: " + JSON.stringify(branches));
+      debug("We get the following as branches: " + util.inspect(branches, false, 10, true));
 
       done(err, _.map(branches || [], function (branch) {
         return branch.name;