Merge pull request #68 from zowe/fix/67

Fix/67
zowe · Oct 17, 2018 · 462f4ef · 462f4ef
2 parents 83e3cad + 503f6ab
commit 462f4ef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/imperative/__tests__/plugins/utilities/npm-interface/install.test.ts b/packages/imperative/__tests__/plugins/utilities/npm-interface/install.test.ts
@@ -82,7 +82,7 @@ describe("PMF: Install Interface", () => {
    */
   const wasExecSyncCallValid = (expectedPackage: string, expectedRegistry: string) => {
     expect(mocks.execSync).toHaveBeenCalledWith(
-      `npm install "${expectedPackage}" --prefix ${PMFConstants.instance.PLUGIN_INSTALL_LOCATION} -g --registry ${expectedRegistry}`,
+      `npm install "${expectedPackage}" --prefix "${PMFConstants.instance.PLUGIN_INSTALL_LOCATION}" -g --registry "${expectedRegistry}"`,
       {
         cwd  : PMFConstants.instance.PMF_ROOT,
         stdio: ["pipe", "pipe", process.stderr]

diff --git a/packages/imperative/src/plugins/utilities/npm-interface/install.ts b/packages/imperative/src/plugins/utilities/npm-interface/install.ts
@@ -90,8 +90,8 @@ export function install(packageLocation: string, registry: string, installFromFi
         // some form of a half-assed progress bar. This progress bar doesn't have any
         // formatting or colors but at least I can get the output of stdout right.
         iConsole.info("Installing packages...this may take some time.");
-        const execOutput = execSync(`npm install "${npmPackage}" --prefix ${PMFConstants.instance.PLUGIN_INSTALL_LOCATION} ` +
-            `-g --registry ${registry}`, {
+        const execOutput = execSync(`npm install "${npmPackage}" --prefix "${PMFConstants.instance.PLUGIN_INSTALL_LOCATION}" ` +
+            `-g --registry "${registry}"`, {
             cwd: PMFConstants.instance.PMF_ROOT,
             stdio: pipe
         });