feat: add emptyDist param to buildActions (#192)

adobe · Aug 15, 2024 · 6eede3a · 6eede3a
1 parent da90917
commit 6eede3a
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/src/build-actions.js b/src/build-actions.js
@@ -285,7 +285,7 @@ const zipActions = async (buildsList, lastBuildsPath, distFolder, skipCheck) =>
   return builtList
 }
 
-const buildActions = async (config, filterActions, skipCheck = false) => {
+const buildActions = async (config, filterActions, skipCheck = false, emptyDist = true) => {
   if (!config.app.hasBackend) {
     throw new Error('cannot build actions, app has no backend')
   }
@@ -299,7 +299,9 @@ const buildActions = async (config, filterActions, skipCheck = false) => {
   const distFolder = config.actions.dist
 
   // clear out dist dir
-  fs.emptyDirSync(distFolder)
+  if (emptyDist) {
+    fs.emptyDirSync(distFolder)
+  }
   const toBuildList = []
   const lastBuiltActionsPath = path.join(config.root, 'dist', 'last-built-actions.json')
   for (const [pkgName, pkg] of Object.entries(modifiedConfig.manifest.full.packages)) {

diff --git a/test/build.actions.test.js b/test/build.actions.test.js
@@ -53,6 +53,7 @@ beforeEach(() => {
 
   execa.mockReset()
   utils.zip.mockReset()
+  fs.emptyDirSync = jest.fn()
 })
 
 describe('build by zipping js action folder', () => {
@@ -1156,6 +1157,20 @@ test('should not zip files if the action was built before (skipCheck=false)', as
   expect(utils.zip).not.toHaveBeenCalled()
 })
 
+test('should not delete dist folder when emptyDist=false', async () => {
+  addSampleAppFiles()
+  const config = deepClone(global.sampleAppConfig)
+  await buildActions(config, ['action'], false /* skipCheck */, false /* emptyDist */)
+  expect(fs.emptyDirSync).not.toHaveBeenCalled()
+})
+
+test('should delete dist folder when emptyDist=true', async () => {
+  addSampleAppFiles()
+  const config = deepClone(global.sampleAppConfig)
+  await buildActions(config, ['action'], false /* skipCheck */, true /* emptyDist */)
+  expect(fs.emptyDirSync).toHaveBeenCalled()
+})
+
 test('No backend is present', async () => {
   addSampleAppFiles()
   const config = deepClone(global.sampleAppConfig)