fix(path): use $GITHUB_PATH

JPBM135 · Jan 18, 2024 · ba52057 · ba52057
1 parent 4a4ca3c
commit ba52057
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/src/index.spec.ts b/src/index.spec.ts
@@ -33,7 +33,7 @@ describe('Self-Hosted Actions Polyfill', () => {
     await main();
 
     expect(core.setFailed).not.toHaveBeenCalled();
-    expect(exec.exec).toHaveBeenCalledTimes(4);
+    expect(exec.exec).toHaveBeenCalledTimes(5);
 
     expect(exec.exec).toHaveBeenCalledWith('sudo', ['apt-get', 'update', '-y'], expect.anything());
 
@@ -53,9 +53,13 @@ describe('Self-Hosted Actions Polyfill', () => {
 
     expect(exec.exec).toHaveBeenCalledWith('/bin/bash', ['-c', POLYFILLS.yarn?.command], expect.anything());
 
-    expect(exec.exec).toHaveBeenLastCalledWith('sudo', ['apt-get', 'autoremove', '-y'], expect.anything());
+    expect(exec.exec).toHaveBeenCalledWith(
+      '/bin/bash',
+      ['-c', `echo "${POLYFILLS.yarn.path}" >> $GITHUB_PATH`],
+      expect.anything(),
+    );
 
-    expect(core.addPath).toHaveBeenCalledWith(POLYFILLS.yarn.path!);
+    expect(exec.exec).toHaveBeenLastCalledWith('sudo', ['apt-get', 'autoremove', '-y'], expect.anything());
   });
 
   it('should throw an error if the platform is not supported', async () => {

diff --git a/src/index.ts b/src/index.ts
@@ -77,7 +77,12 @@ export async function main() {
       });
 
       if (polyfillOptions.path) {
-        core.addPath(polyfillOptions.path);
+        const escapedPath = polyfillOptions.path.replaceAll('"', '\\"');
+
+        await exec.exec('/bin/bash', ['-c', `echo "${escapedPath}" >> $GITHUB_PATH`], {
+          ...createStreams(),
+        });
+
         core.info(`Added ${polyfill} polyfill to PATH.`);
       }