fix: Remove path normalization at arg parser level; now handled in cm…

…d/run.js
mozilla · Jan 23, 2024 · 13af82f · 13af82f
1 parent 9dc5b59
commit 13af82f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 14 deletions.
diff --git a/src/program.js b/src/program.js
@@ -430,7 +430,7 @@ Example: $0 --help run.
       default: process.cwd(),
       requiresArg: true,
       type: 'string',
-      coerce: (arg) => (arg != null ? path.resolve(arg) : undefined),
+      coerce: (arg) => arg ?? undefined,
     },
     'artifacts-dir': {
       alias: 'a',

diff --git a/tests/unit/test.program.js b/tests/unit/test.program.js
@@ -481,19 +481,6 @@ describe('program.main', () => {
     sinon.assert.calledWith(fakeVersionGetter, projectRoot);
   });
 
-  it('turns sourceDir into an absolute path', () => {
-    const fakeCommands = fake(commands, {
-      build: () => Promise.resolve(),
-    });
-    return execProgram(['build', '--source-dir', '..'], {
-      commands: fakeCommands,
-    }).then(() => {
-      sinon.assert.calledWithMatch(fakeCommands.build, {
-        sourceDir: path.resolve(path.join(process.cwd(), '..')),
-      });
-    });
-  });
-
   it('normalizes the artifactsDir path', () => {
     const fakeCommands = fake(commands, {
       build: () => Promise.resolve(),