feat: parameter overrides support double escape

BREAKING CHANGE
joscha · Sep 3, 2021 · 350cf7e · 350cf7e
1 parent e1abeb9
commit 350cf7e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/__tests__/command.spec.ts b/src/__tests__/command.spec.ts
@@ -383,7 +383,16 @@ describe('buildkite-graph', () => {
               .withParameterOverride('priority', 'MY_PRIORITY')
               .toJson(),
           ).resolves.toEqual(
-            expect.objectContaining({ priority: '${MY_PRIORITY}' }),
+            expect.objectContaining({ priority: '$MY_PRIORITY' }),
+          );
+        });
+        it('supports double escape', async () => {
+          await expect(
+            new CommandStep('noop')
+              .withParameterOverride('priority', '$MY_PRIORITY')
+              .toJson(),
+          ).resolves.toEqual(
+            expect.objectContaining({ priority: '$$MY_PRIORITY' }),
           );
         });
       });

diff --git a/src/steps/command.ts b/src/steps/command.ts
@@ -285,7 +285,7 @@ export class CommandStep extends LabeledStep {
 
   private valueWithOverride<T>(value: T, key: CommandProperty): string | T {
     if (this.overrides.has(key)) {
-      return `$\{${this.overrides.get(key) as string}}`;
+      return ('$' + this.overrides.get(key)) as string;
     }
     return value;
   }