From 755bf9f83354dbd01026018d18a897df58d0918f Mon Sep 17 00:00:00 2001 From: James Ives Date: Thu, 18 Nov 2021 07:49:23 -0500 Subject: [PATCH] Folder Permissions (#925) * Unit Tests * Update main.test.ts --- __tests__/git.test.ts | 20 ++++++++++---------- __tests__/main.test.ts | 4 ++-- src/git.ts | 6 ++++++ 3 files changed, 18 insertions(+), 12 deletions(-) diff --git a/__tests__/git.test.ts b/__tests__/git.test.ts index 69e9bfac..469556cb 100644 --- a/__tests__/git.test.ts +++ b/__tests__/git.test.ts @@ -168,7 +168,7 @@ describe('git', () => { const response = await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(12) + expect(execute).toBeCalledTimes(13) expect(rmRF).toBeCalledTimes(1) expect(response).toBe(Status.SUCCESS) }) @@ -191,7 +191,7 @@ describe('git', () => { const response = await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(11) + expect(execute).toBeCalledTimes(12) expect(rmRF).toBeCalledTimes(1) expect(response).toBe(Status.SUCCESS) }) @@ -216,7 +216,7 @@ describe('git', () => { await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(12) + expect(execute).toBeCalledTimes(13) expect(rmRF).toBeCalledTimes(1) }) @@ -240,7 +240,7 @@ describe('git', () => { await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(11) + expect(execute).toBeCalledTimes(12) expect(rmRF).toBeCalledTimes(1) }) @@ -265,7 +265,7 @@ describe('git', () => { await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(11) + expect(execute).toBeCalledTimes(12) expect(rmRF).toBeCalledTimes(1) }) @@ -296,7 +296,7 @@ describe('git', () => { const response = await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(12) + expect(execute).toBeCalledTimes(13) expect(rmRF).toBeCalledTimes(1) expect(fs.existsSync).toBeCalledTimes(2) expect(response).toBe(Status.SUCCESS) @@ -328,7 +328,7 @@ describe('git', () => { await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(9) + expect(execute).toBeCalledTimes(10) expect(rmRF).toBeCalledTimes(1) }) }) @@ -353,7 +353,7 @@ describe('git', () => { await deploy(action) // Includes the call to generateWorktree - expect(execute).toBeCalledTimes(9) + expect(execute).toBeCalledTimes(10) expect(rmRF).toBeCalledTimes(1) }) @@ -373,7 +373,7 @@ describe('git', () => { await deploy(action) - expect(execute).toBeCalledTimes(9) + expect(execute).toBeCalledTimes(10) expect(rmRF).toBeCalledTimes(1) expect(mkdirP).toBeCalledTimes(1) }) @@ -393,7 +393,7 @@ describe('git', () => { }) const response = await deploy(action) - expect(execute).toBeCalledTimes(9) + expect(execute).toBeCalledTimes(10) expect(rmRF).toBeCalledTimes(1) expect(response).toBe(Status.SKIPPED) }) diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts index 82dfee9e..510e7c65 100644 --- a/__tests__/main.test.ts +++ b/__tests__/main.test.ts @@ -49,7 +49,7 @@ describe('main', () => { debug: true }) await run(action) - expect(execute).toBeCalledTimes(14) + expect(execute).toBeCalledTimes(15) expect(rmRF).toBeCalledTimes(1) expect(exportVariable).toBeCalledTimes(1) }) @@ -69,7 +69,7 @@ describe('main', () => { isTest: TestFlag.HAS_CHANGED_FILES }) await run(action) - expect(execute).toBeCalledTimes(17) + expect(execute).toBeCalledTimes(18) expect(rmRF).toBeCalledTimes(1) expect(exportVariable).toBeCalledTimes(1) }) diff --git a/src/git.ts b/src/git.ts index cdf69761..915543d7 100644 --- a/src/git.ts +++ b/src/git.ts @@ -226,6 +226,12 @@ export async function deploy(action: ActionInterface): Promise { action.silent ) + await execute( + `chmod -R 777 ${temporaryDeploymentDirectory}`, + action.workspace, + action.silent + ) + await execute( `git worktree remove ${temporaryDeploymentDirectory} --force`, action.workspace,