fix(lambda-nodejs): bundling with nodeModules fails with paths containing spaces (#17632)

Enclose paths with double quotes (`"`).

Closes #17631


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

Author: jogold

packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts

@@ -305,22 +305,22 @@ class OsCommand {
   public writeJson(filePath: string, data: any): string {
     const stringifiedData = JSON.stringify(data);
     if (this.osPlatform === 'win32') {
-      return `echo ^${stringifiedData}^ > ${filePath}`;
+      return `echo ^${stringifiedData}^ > "${filePath}"`;
     }
 
-    return `echo '${stringifiedData}' > ${filePath}`;
+    return `echo '${stringifiedData}' > "${filePath}"`;
   }
 
   public copy(src: string, dest: string): string {
     if (this.osPlatform === 'win32') {
-      return `copy ${src} ${dest}`;
+      return `copy "${src}" "${dest}"`;
     }
 
-    return `cp ${src} ${dest}`;
+    return `cp "${src}" "${dest}"`;
   }
 
   public changeDirectory(dir: string): string {
-    return `cd ${dir}`;
+    return `cd "${dir}"`;
   }
 }
 

packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts

@@ -169,9 +169,9 @@ test('esbuild bundling with externals and dependencies', () => {
         'bash', '-c',
         [
           'esbuild --bundle "/asset-input/test/bundling.test.js" --target=node12 --platform=node --outfile="/asset-output/index.js" --external:abc --external:delay',
-          `echo \'{\"dependencies\":{\"delay\":\"${delayVersion}\"}}\' > /asset-output/package.json`,
-          'cp /asset-input/package-lock.json /asset-output/package-lock.json',
-          'cd /asset-output',
+          `echo \'{\"dependencies\":{\"delay\":\"${delayVersion}\"}}\' > "/asset-output/package.json"`,
+          'cp "/asset-input/package-lock.json" "/asset-output/package-lock.json"',
+          'cd "/asset-output"',
           'npm ci',
         ].join(' && '),
       ],
@@ -550,9 +550,9 @@ test('esbuild bundling with projectRoot and externals and dependencies', () => {
         'bash', '-c',
         [
           'esbuild --bundle "/asset-input/packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.js" --target=node12 --platform=node --outfile="/asset-output/index.js" --external:abc --external:delay',
-          `echo \'{\"dependencies\":{\"delay\":\"${delayVersion}\"}}\' > /asset-output/package.json`,
-          'cp /asset-input/common/package-lock.json /asset-output/package-lock.json',
-          'cd /asset-output',
+          `echo \'{\"dependencies\":{\"delay\":\"${delayVersion}\"}}\' > "/asset-output/package.json"`,
+          'cp "/asset-input/common/package-lock.json" "/asset-output/package-lock.json"',
+          'cd "/asset-output"',
           'npm ci',
         ].join(' && '),
       ],