Only ignore dfu-util errors when 'File download successfully' is part…

… of stdout

Fixes #206

commands/UpdateCommand.js

@@ -84,14 +84,7 @@ UpdateCommand.prototype = extend(BaseCommand.prototype, {
 				steps.push(function (cb) {
 					var binary = path.resolve(__dirname, '..', 'updates', updates[part]);
 					whenNode.bindCallback(
-						dfu.write(binary, part, leave)
-						.then(null, function (err) {
-							// don't reject for get_status error
-							if (err.indexOf('Error during download get_status') >= 0) {
-								return '';
-							}
-						})
-						.delay(2000)
+						dfu.write(binary, part, leave).delay(2000)
 					, cb);
 				});
 				first = false;

lib/dfu.js

@@ -142,7 +142,15 @@ var that = module.exports = {
 		}
 
 		that.checkBinaryAlignment('-D ' + binaryPath);
-		return utilities.deferredSpawnProcess(cmd, args);
+		return utilities.deferredSpawnProcess(cmd, args).then(function(output) {
+			return when.resolve(output.stdout.join('\n'));
+		}).catch(function(output) {
+			// If this line is printed, it actually worked. Ignore other errors.
+			if (output.stdout.indexOf('File downloaded successfully') >= 0) {
+				return when.resolve(output.stdout.join('\n'));
+			}
+			return when.reject(output.stderr.join('\n'));
+		});
 	},
 
 	getCommand: function () {

lib/utilities.js

@@ -111,10 +111,11 @@ var that = module.exports = {
 			}
 
 			child.on('close', function (code) {
+				var output = { stdout: stdout, stderr: errors };
 				if (!code) {
-					tmp.resolve(stdout.join('\n'));
+					tmp.resolve(output);
 				} else {
-					tmp.reject(errors.join('\n'));
+					tmp.reject(output);
 				}
 			});
 		} catch (ex) {