From 6a8ce0718db6e3449088159bf2daf8b937f7bd62 Mon Sep 17 00:00:00 2001 From: Sam Hiatt Date: Thu, 11 Sep 2014 19:30:58 -0700 Subject: [PATCH 1/2] Improved error checking for mkcouchdb and rmcouchdb tasks. --- tasks/couchapp.js | 34 ++++++++++++++++++++++------------ 1 file changed, 22 insertions(+), 12 deletions(-) diff --git a/tasks/couchapp.js b/tasks/couchapp.js index 30482c3..94db5b5 100755 --- a/tasks/couchapp.js +++ b/tasks/couchapp.js @@ -55,13 +55,14 @@ module.exports = function(grunt) { grunt.log.writeln("Database " + db.name + " not present... skipping."); return done(null, null) ; } else { - grunt.warn("Database " + db.name + " does not exist."); + throw ("Database " + db.name + " does not exist."); } } else { - grunt.warn(err); + throw err; } } - return done(err, null); + // remove db was a success + return done(); }); } else { grunt.log.writeln("No database specified... skipping."); @@ -69,7 +70,7 @@ module.exports = function(grunt) { } } catch (e) { grunt.warn(e); - done(e, null); + return done(e, null); } return null; }); @@ -86,26 +87,35 @@ module.exports = function(grunt) { nano = require('nano')(db.url); nano.db.create(db.name, function(err, res) { if (err) { - if (_this.data.options && _this.data.options.okay_if_exists) { + if (err.error && err.error=="unauthorized") { + grunt.warn(err.reason); + throw err; + } else if (err.code && err.description) { // probably connection error + throw err; + } + else if (_this.data.options && _this.data.options.okay_if_exists) { grunt.log.writeln("Database " + db.name + " exists, skipping"); return done(null, null); } else { - grunt.warn("Database " + db.name + " exists, aborting."); - return done(err, null); + grunt.warn("Database " + db.name + " exists and okay_if_exists set to false. Aborting."); + throw err; } - } else { - grunt.log.writeln("Database " + db.name + " created."); + } else if (res && res.ok==true) { + grunt.log.ok("Database " + db.name + " created."); return done(null, null); + } else { + console.log("Unexpected response received."); + console.dir(res); + throw "Unexpected response"; } }); } else { var err_msg = "No database specified to create!"; - grunt.warn(err_msg); - return done(new Error(err_msg), null); + throw err_msg; } } catch (e) { grunt.warn(e); - done(e, null); + return done(e, null); } return null; }); From 541539cd68544fef3806f4914d342a4310553077 Mon Sep 17 00:00:00 2001 From: Sam Hiatt Date: Thu, 11 Sep 2014 19:58:43 -0700 Subject: [PATCH 2/2] Added support for new-style (directory-based) couchapps. --- tasks/couchapp.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tasks/couchapp.js b/tasks/couchapp.js index 94db5b5..ebbc3d9 100755 --- a/tasks/couchapp.js +++ b/tasks/couchapp.js @@ -32,8 +32,11 @@ module.exports = function(grunt) { grunt.registerMultiTask("couchapp", "Install Couchapp", function() { var appobj, done; done = this.async(); - - appobj = require(path.join(process.cwd(), path.normalize(this.data.app))); + if (require("fs").lstatSync(this.data.app).isDirectory()) { // if new-style (directory-based) couchapp app + appobj = couchapp.loadFiles(this.data.app); + } else { // otherwise, fall back to old style. + appobj = require(path.join(process.cwd(), path.normalize(this.data.app))) + } return couchapp.createApp(appobj, this.data.db, function(app) { return app.push(done);