fix: use full output-path on mdapi convert

salesforcecli · Jul 15, 2024 · 357e721 · 357e721
1 parent a04af69
commit 357e721
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/commands/project/convert/mdapi.ts b/src/commands/project/convert/mdapi.ts
@@ -202,8 +202,13 @@ const ensureFlagPath = async (options: EnsureFsFlagOptions): Promise<string> =>
         throw new SfError(messages.getMessage('InvalidFlagPath', [flagName, path, enoent]), 'InvalidFlagPath');
       }
       const dir = type === 'dir' ? resolvedPath : dirname(resolvedPath);
-      // using as because fs promises always returns a string when recursive is true
-      return fs.promises.mkdir(dir, { recursive: true }) as Promise<string>;
+
+      await fs.promises.mkdir(dir, { recursive: true }).catch((err) => {
+        throw SfError.wrap(err);
+      });
+
+      // `fs.mkdir` will return only the first dir in the path so we return the full path here
+      return dir;
     }
   }
 };
diff --git a/test/nuts/convert/mdapi.nut.ts b/test/nuts/convert/mdapi.nut.ts
@@ -90,6 +90,10 @@ describe('project convert mdapi NUTs', () => {
       expect(result.jsonOutput?.status).to.equal(0);
       expect(result.jsonOutput?.result).to.be.an('array').with.length.greaterThan(10);
       expect(fs.existsSync(convertedToSrcPath)).to.be.true;
+      expect(result.jsonOutput).to.not.be.undefined;
+      result.jsonOutput?.result.forEach((md) => {
+        expect(md.filePath.startsWith(convertedToSrcPath)).to.be.true;
+      });
     });
 
     it('should convert the dreamhouse project using metadatapath flag', () => {