fix: copy when punctuation marks in sketch path

Changed the `source` and `cwd` args to avoid accidentally creating an
invalid `glob` patterns when doing the brace expansion by `cpy`.

Closes #2043

Signed-off-by: Akos Kitta <[email protected]>

Author: Akos Kitta

Diff for: arduino-ide-extension/src/node/sketches-service-impl.ts

@@ -438,7 +438,7 @@ export class SketchesServiceImpl
    * For example, on Windows, instead of getting an [8.3 filename](https://en.wikipedia.org/wiki/8.3_filename), callers will get a fully resolved path.
    * `C:\\Users\\KITTAA~1\\AppData\\Local\\Temp\\.arduinoIDE-unsaved2022615-21100-iahybb.yyvh\\sketch_jul15a` will be `C:\\Users\\kittaakos\\AppData\\Local\\Temp\\.arduinoIDE-unsaved2022615-21100-iahybb.yyvh\\sketch_jul15a`
    */
-  createTempFolder(): Promise<string> {
+  private createTempFolder(): Promise<string> {
     return new Promise<string>((resolve, reject) => {
       temp.mkdir({ prefix: TempSketchPrefix }, (createError, dirPath) => {
         if (createError) {
@@ -520,13 +520,14 @@ export class SketchesServiceImpl
 
     const cpyModule = await import('cpy');
     const cpy = cpyModule.default;
-    await cpy(source, destination, {
+    await cpy(sourceFolderBasename, destination, {
       rename: (basename) =>
         sourceFolderBasename !== destinationFolderBasename &&
         basename === `${sourceFolderBasename}.ino`
           ? `${destinationFolderBasename}.ino`
           : basename,
       filter,
+      cwd: path.dirname(source),
     });
     const copiedSketch = await this.doLoadSketch(destinationUri, false);
     return copiedSketch;

Diff for: arduino-ide-extension/src/test/node/sketches-service-impl.slow-test.ts

@@ -226,16 +226,72 @@ describe('sketches-service-impl', () => {
       expect(mainFileContentOneAfterCopy).to.be.equal(contentOne);
       expect(mainFileContentTwoAfterCopy).to.be.equal(contentOne);
     });
+
+    [
+      ['(', ')', 'parentheses'],
+      ['[', ']', 'brackets'],
+      ['{', '}', 'braces'],
+      ['<', '>', 'chevrons'],
+    ].map(([open, close, name]) =>
+      it(`should copy a sketch when the path contains ${name} in the sketch folder path: '${open},${close}'`, async function () {
+        this.timeout(testTimeout);
+        const sketchesService =
+          container.get<SketchesServiceImpl>(SketchesService);
+        const content = `// special content when ${name} are in the path`;
+        const tempRoot = await sketchesService['createTempFolder']();
+        toDispose.push(disposeFolder(tempRoot));
+        const sketch = await sketchesService.createNewSketch(
+          'punctuation_marks',
+          content
+        );
+        toDispose.push(disposeSketch(sketch));
+
+        // the destination path contains punctuation marks
+        const tempRootUri = FileUri.create(tempRoot);
+        const testSegment = `path segment with ${open}${name}${close}`;
+        const firstDestinationUri = tempRootUri
+          .resolve(testSegment)
+          .resolve('first')
+          .resolve(sketch.name);
+
+        const firstSketchCopy = await sketchesService.copy(sketch, {
+          destinationUri: firstDestinationUri.toString(),
+        });
+        expect(firstSketchCopy).to.be.not.undefined;
+        expect(firstSketchCopy.mainFileUri).to.be.equal(
+          firstDestinationUri.resolve(`${sketch.name}.ino`).toString()
+        );
+        const firstCopyContent = await mainFileContentOf(firstSketchCopy);
+        expect(firstCopyContent).to.be.equal(content);
+
+        // the source path contains punctuation marks. yes, the target too, but it does not matter
+        const secondDestinationUri = tempRootUri
+          .resolve(testSegment)
+          .resolve('second')
+          .resolve(sketch.name);
+        const secondSketchCopy = await sketchesService.copy(firstSketchCopy, {
+          destinationUri: secondDestinationUri.toString(),
+        });
+        expect(secondSketchCopy).to.be.not.undefined;
+        expect(secondSketchCopy.mainFileUri).to.be.equal(
+          secondDestinationUri.resolve(`${sketch.name}.ino`).toString()
+        );
+        const secondCopyContent = await mainFileContentOf(secondSketchCopy);
+        expect(secondCopyContent).to.be.equal(content);
+      })
+    );
   });
 });
 
 function disposeSketch(...sketch: Sketch[]): Disposable {
+  return disposeFolder(...sketch.map(({ uri }) => FileUri.fsPath(uri)));
+}
+
+function disposeFolder(...paths: string[]): Disposable {
   return new DisposableCollection(
-    ...sketch
-      .map(({ uri }) => FileUri.fsPath(uri))
-      .map((path) =>
-        Disposable.create(() => rimrafSync(path, { maxBusyTries: 5 }))
-      )
+    ...paths.map((path) =>
+      Disposable.create(() => rimrafSync(path, { maxBusyTries: 5 }))
+    )
   );
 }