@appcircle/codepush-cli 0.0.2 → 0.0.3

package/script/command-parser.ts

@@ -34,84 +34,6 @@ export function showHelp(showRootDescription?: boolean): void {
   }
 }
 
-function accessKeyAdd(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " access-key " + commandName + " <accessKeyName>")
-    .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option arguments
-    .example(
-      "access-key " + commandName + ' "VSTS Integration"',
-      'Creates a new access key with the name "VSTS Integration", which expires in 60 days'
-    )
-    .example(
-      "access-key " + commandName + ' "One time key" --ttl 5m',
-      'Creates a new access key with the name "One time key", which expires in 5 minutes'
-    )
-    .option("ttl", {
-      default: "60d",
-      demand: false,
-      description: "Duration string which specifies the amount of time that the access key should remain valid for (e.g 5m, 60d, 1y)",
-      type: "string",
-    });
-
-  addCommonConfiguration(yargs);
-}
-
-function accessKeyPatch(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " access-key " + commandName + " <accessKeyName>")
-    .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option arguments
-    .example(
-      "access-key " + commandName + ' "Key for build server" --name "Key for CI machine"',
-      'Renames the access key named "Key for build server" to "Key for CI machine"'
-    )
-    .example(
-      "access-key " + commandName + ' "Key for build server" --ttl 7d',
-      'Updates the access key named "Key for build server" to expire in 7 days'
-    )
-    .option("name", {
-      default: null,
-      demand: false,
-      description: "Display name for the access key",
-      type: "string",
-    })
-    .option("ttl", {
-      default: null,
-      demand: false,
-      description: "Duration string which specifies the amount of time that the access key should remain valid for (e.g 5m, 60d, 1y)",
-      type: "string",
-    });
-  addCommonConfiguration(yargs);
-}
-
-function accessKeyList(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " access-key " + commandName + " [options]")
-    .demand(/*count*/ 0, /*max*/ 0)
-    .example("access-key " + commandName, "Lists your access keys in tabular format")
-    .example("access-key " + commandName + " --format json", "Lists your access keys in JSON format")
-    .option("format", {
-      default: "table",
-      demand: false,
-      description: 'Output format to display your access keys with ("json" or "table")',
-      type: "string",
-    });
-
-  addCommonConfiguration(yargs);
-}
-
-function accessKeyRemove(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " access-key " + commandName + " <accessKeyName>")
-    .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option arguments
-    .example("access-key " + commandName + ' "VSTS Integration"', 'Removes the "VSTS Integration" access key');
-
-  addCommonConfiguration(yargs);
-}
-
 function addCommonConfiguration(yargs: yargs.Argv): void {
   yargs
     .wrap(/*columnLimit*/ null)
@@ -146,6 +68,16 @@ function appRemove(commandName: string, yargs: yargs.Argv): void {
   addCommonConfiguration(yargs);
 }
 
+function appDeploymentKeyList(commandName: string, yargs: yargs.Argv): void {
+  isValidCommand = true;
+  yargs
+    .usage(USAGE_PREFIX + " app " + commandName + " <appName>")
+    .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option arguments
+    .example("app " + commandName + " MyApp", 'Lists the deployment keys for app "MyApp" in tabular format');
+
+  addCommonConfiguration(yargs);
+}
+
 function listCollaborators(commandName: string, yargs: yargs.Argv): void {
   isValidCommand = true;
   yargs
@@ -173,33 +105,6 @@ function removeCollaborator(commandName: string, yargs: yargs.Argv): void {
   addCommonConfiguration(yargs);
 }
 
-function sessionList(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " session " + commandName + " [options]")
-    .demand(/*count*/ 0, /*max*/ 0)
-    .example("session " + commandName, "Lists your sessions in tabular format")
-    .example("session " + commandName + " --format json", "Lists your login sessions in JSON format")
-    .option("format", {
-      default: "table",
-      demand: false,
-      description: 'Output format to display your login sessions with ("json" or "table")',
-      type: "string",
-    });
-
-  addCommonConfiguration(yargs);
-}
-
-function sessionRemove(commandName: string, yargs: yargs.Argv): void {
-  isValidCommand = true;
-  yargs
-    .usage(USAGE_PREFIX + " session " + commandName + " <machineName>")
-    .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option arguments
-    .example("session " + commandName + ' "John\'s PC"', 'Removes the existing login session from "John\'s PC"');
-
-  addCommonConfiguration(yargs);
-}
-
 function deploymentHistoryClear(commandName: string, yargs: yargs.Argv): void {
   isValidCommand = true;
   yargs
@@ -279,21 +184,6 @@ function deploymentHistory(commandName: string, yargs: yargs.Argv): void {
 yargs
   .usage(USAGE_PREFIX + " <command>")
   .demand(/*count*/ 1, /*max*/ 1) // Require exactly one non-option argument.
-  .command("access-key", "View and manage the access keys associated with your account", (yargs: yargs.Argv) => {
-    isValidCommandCategory = true;
-    yargs
-      .usage(USAGE_PREFIX + " access-key <command>")
-      .demand(/*count*/ 2, /*max*/ 2) // Require exactly two non-option arguments.
-      .command("add", "Create a new access key associated with your account", (yargs: yargs.Argv) => accessKeyAdd("add", yargs))
-      .command("patch", "Update the name and/or TTL of an existing access key", (yargs: yargs.Argv) => accessKeyPatch("patch", yargs))
-      .command("remove", "Remove an existing access key", (yargs: yargs.Argv) => accessKeyRemove("remove", yargs))
-      .command("rm", "Remove an existing access key", (yargs: yargs.Argv) => accessKeyRemove("rm", yargs))
-      .command("list", "List the access keys associated with your account", (yargs: yargs.Argv) => accessKeyList("list", yargs))
-      .command("ls", "List the access keys associated with your account", (yargs: yargs.Argv) => accessKeyList("ls", yargs))
-      .check((argv: any, aliases: { [aliases: string]: string }): any => isValidCommand); // Report unrecognized, non-hyphenated command category.
-
-    addCommonConfiguration(yargs);
-  })
   .command("app", "View and manage your CodePush apps", (yargs: yargs.Argv) => {
     isValidCommandCategory = true;
     yargs
@@ -321,6 +211,7 @@ yargs
       })
       .command("list", "Lists the apps associated with your account", (yargs: yargs.Argv) => appList("list", yargs))
       .command("ls", "Lists the apps associated with your account", (yargs: yargs.Argv) => appList("ls", yargs))
+      .command("deployment-keys","Lists the deployment keys for app", (yargs:yargs.Argv) => appDeploymentKeyList("deployment-keys", yargs))
       .command("transfer", "Transfer the ownership of an app to another account", (yargs: yargs.Argv) => {
         isValidCommand = true;
         yargs
@@ -391,17 +282,6 @@ yargs
 
     addCommonConfiguration(yargs);
   })
-  .command("link", "Link an additional authentication provider (e.g. GitHub) to an existing CodePush account", (yargs: yargs.Argv) => {
-    isValidCommandCategory = true;
-    isValidCommand = true;
-    yargs
-      .usage(USAGE_PREFIX + " link")
-      .demand(/*count*/ 0, /*max*/ 1) //set 'max' to one to allow usage of serverUrl undocument parameter for testing
-      .example("link", "Links an account on the CodePush server")
-      .check((argv: any, aliases: { [aliases: string]: string }): any => isValidCommand); // Report unrecognized, non-hyphenated command category.
-
-    addCommonConfiguration(yargs);
-  })
   .command("login", "Authenticate with the CodePush server in order to begin managing your apps", (yargs: yargs.Argv) => {
     isValidCommandCategory = true;
     isValidCommand = true;
@@ -579,17 +459,6 @@ yargs
 
     addCommonConfiguration(yargs);
   })
-  .command("register", "Register a new CodePush account", (yargs: yargs.Argv) => {
-    isValidCommandCategory = true;
-    isValidCommand = true;
-    yargs
-      .usage(USAGE_PREFIX + " register")
-      .demand(/*count*/ 0, /*max*/ 1) //set 'max' to one to allow usage of serverUrl undocument parameter for testing
-      .example("register", "Registers a new CodePush account")
-      .check((argv: any, aliases: { [aliases: string]: string }): any => isValidCommand); // Report unrecognized, non-hyphenated command category.
-
-    addCommonConfiguration(yargs);
-  })
   .command("release", "Release an update to an app deployment", (yargs: yargs.Argv) => {
     yargs
       .usage(USAGE_PREFIX + " release <appName> <updateContentsPath> <targetBinaryVersion> [options]")
@@ -853,21 +722,6 @@ yargs
 
     addCommonConfiguration(yargs);
   })
-  .command("session", "View and manage the current login sessions associated with your account", (yargs: yargs.Argv) => {
-    isValidCommandCategory = true;
-    yargs
-      .usage(USAGE_PREFIX + " session <command>")
-      .demand(/*count*/ 2, /*max*/ 2) // Require exactly two non-option arguments.
-      .command("remove", "Remove an existing login session", (yargs: yargs.Argv) => sessionRemove("remove", yargs))
-      .command("rm", "Remove an existing login session", (yargs: yargs.Argv) => sessionRemove("rm", yargs))
-      .command("list", "List the current login sessions associated with your account", (yargs: yargs.Argv) =>
-        sessionList("list", yargs)
-      )
-      .command("ls", "List the current login sessions associated with your account", (yargs: yargs.Argv) => sessionList("ls", yargs))
-      .check((argv: any, aliases: { [aliases: string]: string }): any => isValidCommand); // Report unrecognized, non-hyphenated command category.
-
-    addCommonConfiguration(yargs);
-  })
   .alias("v", "version")
   .version(packageJson.version)
   .wrap(/*columnLimit*/ null)
@@ -887,56 +741,6 @@ export function createCommand(): cli.ICommand {
     const arg4: any = argv._[4];
 
     switch (arg0) {
-      case "access-key":
-        switch (arg1) {
-          case "add":
-            if (arg2) {
-              cmd = { type: cli.CommandType.accessKeyAdd };
-              const accessKeyAddCmd = <cli.IAccessKeyAddCommand>cmd;
-              accessKeyAddCmd.name = arg2;
-              const ttlOption: string = argv["ttl"] as any;
-              if (isDefined(ttlOption)) {
-                accessKeyAddCmd.ttl = parseDurationMilliseconds(ttlOption);
-              }
-            }
-            break;
-
-          case "patch":
-            if (arg2) {
-              cmd = { type: cli.CommandType.accessKeyPatch };
-              const accessKeyPatchCmd = <cli.IAccessKeyPatchCommand>cmd;
-              accessKeyPatchCmd.oldName = arg2;
-
-              const newNameOption: string = argv["name"] as any;
-              const ttlOption: string = argv["ttl"] as any;
-              if (isDefined(newNameOption)) {
-                accessKeyPatchCmd.newName = newNameOption;
-              }
-
-              if (isDefined(ttlOption)) {
-                accessKeyPatchCmd.ttl = parseDurationMilliseconds(ttlOption);
-              }
-            }
-            break;
-
-          case "list":
-          case "ls":
-            cmd = { type: cli.CommandType.accessKeyList };
-
-            (<cli.IAccessKeyListCommand>cmd).format = argv["format"] as any;
-            break;
-
-          case "remove":
-          case "rm":
-            if (arg2) {
-              cmd = { type: cli.CommandType.accessKeyRemove };
-
-              (<cli.IAccessKeyRemoveCommand>cmd).accessKey = arg2;
-            }
-            break;
-        }
-        break;
-
       case "app":
         switch (arg1) {
           case "add":
@@ -953,7 +757,10 @@ export function createCommand(): cli.ICommand {
 
             (<cli.IAppListCommand>cmd).format = argv["format"] as any;
             break;
-
+          case "deployment-keys":
+            cmd = {type : cli.CommandType.appDeploymentKeyList};
+            (<cli.IAppDeploymentKeysCommand>cmd).appName = arg2;
+            break;
           case "remove":
           case "rm":
             if (arg2) {
@@ -1065,13 +872,6 @@ export function createCommand(): cli.ICommand {
         }
         break;
 
-      case "link":
-        cmd = <cli.ILinkCommand>{
-          type: cli.CommandType.link,
-          serverUrl: getServerUrl(arg1),
-        };
-        break;
-
       case "login":
         cmd = { type: cli.CommandType.login };
 
@@ -1124,14 +924,6 @@ export function createCommand(): cli.ICommand {
         }
         break;
 
-      case "register":
-        cmd = { type: cli.CommandType.register };
-
-        const registerCommand = <cli.IRegisterCommand>cmd;
-
-        registerCommand.serverUrl = getServerUrl(arg1);
-        break;
-
       case "release":
         if (arg1 && arg2 && arg3) {
           cmd = { type: cli.CommandType.release };
@@ -1195,26 +987,6 @@ export function createCommand(): cli.ICommand {
           rollbackCommand.targetRelease = argv["targetRelease"] as any;
         }
         break;
-
-      case "session":
-        switch (arg1) {
-          case "list":
-          case "ls":
-            cmd = { type: cli.CommandType.sessionList };
-
-            (<cli.ISessionListCommand>cmd).format = argv["format"] as any;
-            break;
-
-          case "remove":
-          case "rm":
-            if (arg2) {
-              cmd = { type: cli.CommandType.sessionRemove };
-
-              (<cli.ISessionRemoveCommand>cmd).machineName = arg2;
-            }
-            break;
-        }
-        break;
     }
 
     return cmd;

package/script/management-sdk.ts

@@ -28,6 +28,7 @@ import {
   ServerAccessKey,
   Session,
 } from "./types";
+import { DeploymentKey } from "./types/rest-definitions";
 
 const packageJson = require("../../package.json");
 
@@ -268,6 +269,10 @@ class AccountManager {
     return this.get(urlEncode([`/apps/${appName}`])).then((res: JsonResponse) => res.body.app);
   }
 
+  public getDeploymentKeys(appName: string): Promise<DeploymentKey[]> {
+    return this.get(urlEncode([`/apps/${appName}/deployment-keys`])).then((res: JsonResponse) => res.body);
+  }
+
   public addApp(appName: string): Promise<App> {
     const app: App = { name: appName };
     return this.post(urlEncode(["/apps"]), JSON.stringify(app), /*expectResponseBody=*/ false).then(() => app);

package/script/types/cli.ts

@@ -4,12 +4,9 @@
 import AccountManager = require("../management-sdk");
 
 export enum CommandType {
-  accessKeyAdd,
-  accessKeyPatch,
-  accessKeyList,
-  accessKeyRemove,
   appAdd,
   appList,
+  appDeploymentKeyList,
   appRemove,
   appRename,
   debug,
@@ -20,17 +17,13 @@ export enum CommandType {
   deploymentMetrics,
   deploymentRemove,
   deploymentRename,
-  link,
   login,
   logout,
   patch,
   promote,
-  register,
   release,
   releaseReact,
-  rollback,
-  sessionList,
-  sessionRemove
+  rollback
 }
 
 export interface ICommand {
@@ -70,6 +63,10 @@ export interface IAppRemoveCommand extends ICommand {
   appName: string;
 }
 
+export interface IAppDeploymentKeysCommand extends ICommand {
+  appName: string;
+}
+
 export interface IAppRenameCommand extends ICommand {
   currentAppName: string;
   newAppName: string;

package/script/types/rest-definitions.ts

@@ -126,6 +126,12 @@ export interface Deployment {
   /*generated*/ package?: Package;
 }
 
+/*out*/
+export interface DeploymentKey {
+  /*generated key*/ deploymentKey?: string;
+  /*key*/ name: string;
+}
+
 /*out*/
 export interface BlobInfo {
   size: number;

package/test/cli.ts

@@ -298,173 +298,6 @@ describe("CLI", () => {
     sandbox.restore();
   });
 
-  it("accessKeyAdd creates access key with name and default ttl", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyAddCommand = {
-      type: cli.CommandType.accessKeyAdd,
-      name: "Test name",
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledTwice(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = `Successfully created the "Test name" access key: key123`;
-      assert.equal(actual, expected);
-
-      actual = log.args[1][0];
-      expected = "Make sure to save this key value somewhere safe, since you won't be able to view it from the CLI again!";
-      assert.equal(actual, expected);
-
-      done();
-    });
-  });
-
-  it("accessKeyAdd creates access key with name and specified ttl", (done: Mocha.Done): void => {
-    var ttl = 10000;
-    var command: cli.IAccessKeyAddCommand = {
-      type: cli.CommandType.accessKeyAdd,
-      name: "Test name",
-      ttl,
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledTwice(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = `Successfully created the "Test name" access key: key123`;
-      assert.equal(actual, expected);
-
-      actual = log.args[1][0];
-      expected = "Make sure to save this key value somewhere safe, since you won't be able to view it from the CLI again!";
-      assert.equal(actual, expected);
-
-      done();
-    });
-  });
-
-  it("accessKeyPatch updates access key with new name", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyPatchCommand = {
-      type: cli.CommandType.accessKeyPatch,
-      oldName: "Test name",
-      newName: "Updated name",
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = `Successfully renamed the access key "Test name" to "Updated name".`;
-
-      assert.equal(actual, expected);
-      done();
-    });
-  });
-
-  it("accessKeyPatch updates access key with new ttl", (done: Mocha.Done): void => {
-    var ttl = 10000;
-    var command: cli.IAccessKeyPatchCommand = {
-      type: cli.CommandType.accessKeyPatch,
-      oldName: "Test name",
-      ttl,
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = `Successfully changed the expiration date of the "Test name" access key to Wednesday, August 17, 2016 12:07 PM.`;
-
-      assert.equal(actual, expected);
-      done();
-    });
-  });
-
-  it("accessKeyPatch updates access key with new name and ttl", (done: Mocha.Done): void => {
-    var ttl = 10000;
-    var command: cli.IAccessKeyPatchCommand = {
-      type: cli.CommandType.accessKeyPatch,
-      oldName: "Test name",
-      newName: "Updated name",
-      ttl,
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = `Successfully renamed the access key "Test name" to "Updated name" and changed its expiration date to Wednesday, August 17, 2016 12:07 PM.`;
-
-      assert.equal(actual, expected);
-      done();
-    });
-  });
-
-  it("accessKeyList lists access key name and expires fields", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyListCommand = {
-      type: cli.CommandType.accessKeyList,
-      format: "json",
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = [
-        {
-          createdTime: 0,
-          name: "Test name",
-          expires: NOW + DEFAULT_ACCESS_KEY_MAX_AGE,
-        },
-      ];
-
-      assertJsonDescribesObject(actual, expected);
-      done();
-    });
-  });
-
-  it("accessKeyRemove removes access key", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyRemoveCommand = {
-      type: cli.CommandType.accessKeyRemove,
-      accessKey: "8",
-    };
-
-    var removeAccessKey: sinon.SinonSpy = sandbox.spy(cmdexec.sdk, "removeAccessKey");
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(removeAccessKey);
-      sinon.assert.calledWithExactly(removeAccessKey, "8");
-      sinon.assert.calledOnce(log);
-      sinon.assert.calledWithExactly(log, 'Successfully removed the "8" access key.');
-
-      done();
-    });
-  });
-
-  it("accessKeyRemove does not remove access key if cancelled", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyRemoveCommand = {
-      type: cli.CommandType.accessKeyRemove,
-      accessKey: "8",
-    };
-
-    var removeAccessKey: sinon.SinonSpy = sandbox.spy(cmdexec.sdk, "removeAccessKey");
-
-    wasConfirmed = false;
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.notCalled(removeAccessKey);
-      sinon.assert.calledOnce(log);
-      sinon.assert.calledWithExactly(log, "Access key removal cancelled.");
-
-      done();
-    });
-  });
-
   it("appAdd reports new app name and ID", (done: Mocha.Done): void => {
     var command: cli.IAppAddCommand = {
       type: cli.CommandType.appAdd,
@@ -1518,88 +1351,6 @@ describe("CLI", () => {
       .done();
   });
 
-  it("sessionList lists session name and expires fields", (done: Mocha.Done): void => {
-    var command: cli.IAccessKeyListCommand = {
-      type: cli.CommandType.sessionList,
-      format: "json",
-    };
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(log);
-      assert.equal(log.args[0].length, 1);
-
-      var actual: string = log.args[0][0];
-      var expected = [
-        {
-          loggedInTime: 0,
-          machineName: TEST_MACHINE_NAME,
-        },
-      ];
-
-      assertJsonDescribesObject(actual, expected);
-      done();
-    });
-  });
-
-  it("sessionRemove removes session", (done: Mocha.Done): void => {
-    var machineName = TEST_MACHINE_NAME;
-    var command: cli.ISessionRemoveCommand = {
-      type: cli.CommandType.sessionRemove,
-      machineName: machineName,
-    };
-
-    var removeSession: sinon.SinonSpy = sandbox.spy(cmdexec.sdk, "removeSession");
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.calledOnce(removeSession);
-      sinon.assert.calledWithExactly(removeSession, machineName);
-      sinon.assert.calledOnce(log);
-      sinon.assert.calledWithExactly(log, `Successfully removed the login session for "${machineName}".`);
-
-      done();
-    });
-  });
-
-  it("sessionRemove does not remove session if cancelled", (done: Mocha.Done): void => {
-    var machineName = TEST_MACHINE_NAME;
-    var command: cli.ISessionRemoveCommand = {
-      type: cli.CommandType.sessionRemove,
-      machineName: machineName,
-    };
-
-    var removeSession: sinon.SinonSpy = sandbox.spy(cmdexec.sdk, "removeSession");
-
-    wasConfirmed = false;
-
-    cmdexec.execute(command).done((): void => {
-      sinon.assert.notCalled(removeSession);
-      sinon.assert.calledOnce(log);
-      sinon.assert.calledWithExactly(log, "Session removal cancelled.");
-
-      done();
-    });
-  });
-
-  it("sessionRemove does not remove current session", (done: Mocha.Done): void => {
-    var machineName = os.hostname();
-    var command: cli.ISessionRemoveCommand = {
-      type: cli.CommandType.sessionRemove,
-      machineName: machineName,
-    };
-
-    wasConfirmed = false;
-
-    cmdexec
-      .execute(command)
-      .then(() => {
-        done(new Error("Did not throw error."));
-      })
-      .catch(() => {
-        done();
-      })
-      .done();
-  });
-
   function releaseHelperFunction(command: cli.IReleaseCommand, done: Mocha.Done, expectedError: string): void {
     cmdexec.execute(command).done(
       (): void => {