feat: ACNA-1602 - add new flag to prevent action code deployment

src/commands/app/deploy.js

@@ -147,12 +147,15 @@ class Deploy extends BuildCommand {
           if (filterActions) {
             filterEntities = { actions: filterActions }
           }
+
+          const options = { actionCode: flags['action-code'] }
+
           const message = `Deploying actions for '${name}'`
           spinner.start(message)
           try {
             const script = await runScript(config.hooks['deploy-actions'])
             if (!script) {
-              deployedRuntimeEntities = await rtLib.deployActions(config, { filterEntities }, onProgress)
+              deployedRuntimeEntities = await rtLib.deployActions(config, { filterEntities }, onProgress, options)
             }
 
             if (deployedRuntimeEntities.actions && deployedRuntimeEntities.actions.length > 0) {
@@ -337,6 +340,11 @@ Deploy.flags = {
     description: '[default: true] Update log forwarding configuration on server',
     default: true,
     allowNo: true
+  }),
+  'action-code': flags.boolean({
+    description: '[default: true] --no-action-code skips deployment of action code',
+    allowNo: true,
+    default: true
   })
 }
 

test/commands/app/deploy.test.js

@@ -215,6 +215,11 @@ test('flags', async () => {
   expect(TheCommand.flags.actions.default).toEqual(true)
   expect(TheCommand.flags.actions.allowNo).toEqual(true)
   expect(TheCommand.flags.actions.exclusive).toEqual(['action'])
+
+  expect(typeof TheCommand.flags['action-code']).toBe('object')
+  expect(typeof TheCommand.flags['action-code'].description).toBe('string')
+  expect(TheCommand.flags['action-code'].default).toEqual(true)
+  expect(TheCommand.flags['action-code'].allowNo).toEqual(true)
 })
 
 describe('run', () => {
@@ -316,7 +321,8 @@ describe('run', () => {
     expect(mockRuntimeLib.deployActions).toHaveBeenCalledWith(appConfig.application, {
       filterEntities: { actions: ['a', 'b', 'c'] }
     },
-    expect.any(Function))
+    expect.any(Function),
+    expect.objectContaining({ actionCode: true }))
   })
 
   test('build & deploy actions with no actions folder and no manifest', async () => {
@@ -480,6 +486,23 @@ describe('run', () => {
     expect(command.buildOneExt).toHaveBeenCalledWith('application', appConfig.application, expect.objectContaining({ 'force-build': false }), expect.anything()) // force-build is true by default for build cmd
   })
 
+  test('--no-action-code', async () => {
+    const appConfig = createAppConfig(command.appConfig)
+    command.getAppExtConfigs.mockReturnValueOnce(appConfig)
+
+    command.argv = ['--no-action-code', '--action', 'a']
+    await command.run()
+    expect(command.error).toHaveBeenCalledTimes(0)
+    expect(mockRuntimeLib.deployActions).toHaveBeenCalledTimes(1)
+    expect(mockWebLib.deployWeb).toHaveBeenCalledTimes(0)
+    expect(command.buildOneExt).toHaveBeenCalledTimes(1)
+    expect(mockRuntimeLib.deployActions).toHaveBeenCalledWith(appConfig.application, {
+      filterEntities: { actions: ['a'] }
+    },
+    expect.any(Function),
+    expect.objectContaining({ actionCode: false }))
+  })
+
   test.each([
     [['--no-log-forwarding-update']],
     [['--no-actions']],