From 205229817139fc5a9f429e7f5e28dcff2dd7e729 Mon Sep 17 00:00:00 2001 From: Edwin Sirko Date: Fri, 26 Jan 2024 20:52:00 -0500 Subject: [PATCH] fix --- __tests__/fs-helper.test.ts | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/__tests__/fs-helper.test.ts b/__tests__/fs-helper.test.ts index 8d47ebc..22d2cf4 100644 --- a/__tests__/fs-helper.test.ts +++ b/__tests__/fs-helper.test.ts @@ -34,8 +34,11 @@ describe('getConsolidatedDirectory', () => { it("returns the directory itself if it is a single directory, and instructed to not clean it up", () => { - // TODO: We're not really distinguishing between the `publish-action-package` directory and the consumer repo directory. - // In real life the consumer repo is differentiated via ... ?? + // TODO: In these tests, we're not really distinguishing between the `publish-action-package` directory and the consumer repo directory, i.e., they share the same space. + // In real life, when the consumer workflow runs, its own javascript is in ., but + // the publish-action-package's code is in ${{github.action_path}}. + // So.... I guess to emulate this, we should create a temp directory (representing the consumer repo) + // and cd there before the test starts? const { consolidatedPath, needToCleanUpDir } = fsHelper.getConsolidatedDirectory(".") expect(needToCleanUpDir).toBe(false)