Make confirmation prompt more severe on app/deployment rm (#402)

* Make the prompt defaults to 'no' if no input is entered
Make the message more severe than it currently for app rm and deployment rm

* Fix upper/lower-case issues

* Make prompt messages consistent

Author: max-mironov

cli/script/command-executor.ts

@@ -72,7 +72,8 @@ export var execSync = childProcess.execSync;
 
 var connectionInfo: ILoginConnectionInfo;
 
-export var confirm = (): Promise<boolean> => {
+export var confirm = (message: string = "Are you sure?"): Promise<boolean> => {
+    message += " (y/N):";
     return Promise<boolean>((resolve, reject, notify): void => {
         prompt.message = "";
         prompt.delimiter = "";
@@ -82,14 +83,19 @@ export var confirm = (): Promise<boolean> => {
         prompt.get({
             properties: {
                 response: {
-                    description: chalk.cyan("Are you sure? (Y/n):")
+                    description: chalk.cyan(message)
                 }
             }
         }, (err: any, result: any): void => {
-            if (!result.response || result.response === "" || result.response === "Y") {
+            var accepted = result.response && result.response.toLowerCase() === "y";
+            var rejected = !result.response || result.response.toLowerCase() === "n";
+
+            if (accepted){
                 resolve(true);
             } else {
-                if (result.response !== "n") console.log("Invalid response: \"" + result.response + "\"");
+                if (!rejected){
+                    console.log("Invalid response: \"" + result.response + "\"");               
+                } 
                 resolve(false);
             }
         });
@@ -179,7 +185,7 @@ function appList(command: cli.IAppListCommand): Promise<void> {
 }
 
 function appRemove(command: cli.IAppRemoveCommand): Promise<void> {
-    return confirm()
+    return confirm("Are you sure you want to remove this app? Note that its deployment keys will be PERMANENTLY unrecoverable.")
         .then((wasConfirmed: boolean): Promise<void> => {
             if (wasConfirmed) {
                 return sdk.removeApp(command.appName)
@@ -337,7 +343,7 @@ export var deploymentList = (command: cli.IDeploymentListCommand, showPackage: b
 }
 
 function deploymentRemove(command: cli.IDeploymentRemoveCommand): Promise<void> {
-    return confirm()
+    return confirm("Are you sure you want to remove this deployment? Note that its deployment key will be PERMANENTLY unrecoverable.")
         .then((wasConfirmed: boolean): Promise<void> => {
             if (wasConfirmed) {
                 return sdk.removeDeployment(command.appName, command.deploymentName)