Continue if oauth creds not found

This commit is contained in:
Girish Ramakrishnan
2014-08-23 20:33:31 -07:00
parent 7b3f715fe4
commit 68fdce4b67
3 changed files with 10 additions and 5 deletions
+7 -1
View File
@@ -22,6 +22,7 @@ var assert = require('assert'),
net = require('net'),
config = require('../config.js'),
database = require('./database.js'),
DatabaseError = require('./databaseerror.js'),
ejs = require('ejs'),
appFqdn = require('./apps').appFqdn;
@@ -311,7 +312,12 @@ function removeOAuthCredentials(app, callback) {
debug('removeOAuthCredentials:', app.id);
clientdb.del(app.id, callback);
clientdb.del(app.id, function (error) {
if (error && error.reason === DatabaseError.NOT_FOUND) return callback(null);
if (error) console.error(error);
callback(null);
});
}
function startContainer(app, callback) {
+1 -1
View File
@@ -2,7 +2,7 @@
'use strict';
var DatabaseError = require('./databaseerror'),
var DatabaseError = require('./databaseerror.js'),
path = require('path'),
debug = require('debug')('box:clientdb'),
database = require('./database.js'),
+2 -3
View File
@@ -133,10 +133,9 @@ describe('apptask', function () {
});
});
it('remove OAuth credentials twice fails', function (done) {
it('remove OAuth credentials twice succeeds', function (done) {
apptask._removeOAuthCredentials(APP, function (error) {
expect(error).to.be.a(DatabaseError);
expect(error.reason).to.equal(DatabaseError.NOT_FOUND);
expect(!error).to.be.ok();
done();
});
});