biz-a-cli 2.3.63 → 2.3.65

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "biz-a-cli",
   "nameDev": "biz-a-cli-dev",
-  "version": "2.3.63",
+  "version": "2.3.65",
   "versionDev": "0.0.34",
   "description": "",
   "main": "bin/index.js",
@@ -56,8 +56,18 @@
       "!<rootDir>/tests/mockData",
       "!<rootDir>/tests/mockData/**"
     ],
-    "testPathIgnorePatterns": ["<rootDir>/node_modules", "<rootDir>/tests/mockData"],
-    "watchPathIgnorePatterns": ["<rootDir>/node_modules", "<rootDir>/tests/mockData"],
-    "coveragePathIgnorePatterns": ["<rootDir>/node_modules", "<rootDir>/tests/mockData", "<rootDir>/tests/mockData/**"]
+    "testPathIgnorePatterns": [
+      "<rootDir>/node_modules",
+      "<rootDir>/tests/mockData"
+    ],
+    "watchPathIgnorePatterns": [
+      "<rootDir>/node_modules",
+      "<rootDir>/tests/mockData"
+    ],
+    "coveragePathIgnorePatterns": [
+      "<rootDir>/node_modules",
+      "<rootDir>/tests/mockData",
+      "<rootDir>/tests/mockData/**"
+    ]
   }
 }

package/scheduler/datalib.js

@@ -12,6 +12,7 @@ dayjs.default.extend(customParseFormat)
 dayjs.default.extend(timezone)
 dayjs.default.extend(utc)
 
+export const AXIOS_TIMEOUT = 30 * 1000;
 const logger = createLogger({
     level: 'info',
     transports: [
@@ -66,7 +67,8 @@ export function getTableObj(model, tableName) {
 export function options(apiConfig) {
     return {
         headers: { 'content-type': 'text/plain' },
-        params: { subdomain: apiConfig.subdomain }
+        params: { subdomain: apiConfig.subdomain },
+        timeout: apiConfig.timeout ?? AXIOS_TIMEOUT  //SCY BZ 4316
     };
 }
 
@@ -163,17 +165,21 @@ export async function extractFunctionScript(selectedConfig, data) {
     const scriptFn = runScriptInThisContext(data[0].script);
     if (!scriptFn().functions) return
 
+    const axiosInstance = axios.create({ timeout: AXIOS_TIMEOUT });
+
     let lib = {
-        axios: axios.default,
+        // axios: axios.default,
+        axios: axiosInstance,  //SCY BZ 4316
         dayjs: dayjs.default,
         crypto: crypto,
         log: logger,
-        data: {sendModel, queryData, crudData, genId, getUrlAndParam}
+        data: { sendModel, queryData, crudData, genId, getUrlAndParam }
     }
 
     if (scriptFn().functions.useLibrary) {
         Object.assign(lib, await setLibrary(config, scriptFn().functions.useLibrary(), lib));
     }
+
     return scriptFn(lib).functions;
 }
 
@@ -187,7 +193,6 @@ export function getQueryDataPrmsParameters(column, value) {
 
 export async function queryDataPromise(config, column, value) {
     let param = getQueryDataPrmsParameters(column, value);
-    // let param = getQueryDataPrmsParameters(d);
 
     return await queryData(param, config, true);
 }

package/tests/app.test.js

@@ -6,14 +6,15 @@ import { createDecipheriv } from 'node:crypto'
 import * as tar from "tar"
 import { env } from "../envs/env.js"
 
-describe('Biz-A Apps CLI', ()=>{
+describe('Biz-A Apps CLI', () => {
+    let originalOptions, originalCWD, axios, child_process;
+    let returnedCode = 0;
 
-    let originalOptions, originalCWD, axios
     const logSpy = jest.spyOn(global.console, 'log').mockImplementation()
     const errorSpy = jest.spyOn(global.console, 'error').mockImplementation()
 
-    async function runCommand(...options){
-        process.argv[1] = process.cwd()+'\\bin' // mock the key root folder
+    async function runCommand(...options) {
+        process.argv[1] = process.cwd() + '\\bin' // mock the key root folder
         process.argv = [process.argv[0], process.argv[1], ...options]
         await import('../bin/app.js')
 
@@ -25,79 +26,90 @@ describe('Biz-A Apps CLI', ()=>{
         await new Promise(process.nextTick) // wait for all import promise (including ES6 import promises) to be resolved
     }
 
-    beforeEach(async ()=>{
+    beforeEach(async () => {
         originalOptions = process.argv
         originalCWD = process.cwd()
 
         // mock default export first with unstable_mockModule
-        jest.unstable_mockModule('axios', ()=>{return {default: jest.fn()}})
+        jest.unstable_mockModule('axios', () => { return { default: jest.fn() } })
         axios = (await import('axios')).default
 
         // then we can mock classes, functions, variables, and objects inside mocked module when doing dynamic import using jest.fn()
         axios.get = jest.fn()
         axios.post = jest.fn()
+
+        jest.unstable_mockModule('node:child_process', () => ({
+            spawn: jest.fn().mockReturnValue({
+                on: jest.fn((event, callback) => {
+                    if (event === "close") {
+                        callback(returnedCode);
+                    }
+                })
+            })
+        }));
+        child_process = (await import('node:child_process'));
     })
 
-    afterEach(()=>{
+    afterEach(() => {
         jest.resetModules()
         jest.resetAllMocks()
         process.argv = originalOptions
         process.chdir(originalCWD)
     })
 
-    describe('Add App', ()=>{
+    describe('Add App', () => {
         const mockDataFolder = './tests/mockData/'
 
-        function mockValidTemplates(appName, scriptList){
-            fs.rmSync(mockDataFolder+'/'+appName, {force: true, recursive: true})
-            fs.mkdirSync(mockDataFolder+'/'+appName, {recursive: true})
-            for (const [key, value] of Object.entries(scriptList)){
-                fs.writeFileSync(mockDataFolder+'/'+appName + '/' + key, value.act)
+        function mockValidTemplates(appName, scriptList) {
+            fs.rmSync(mockDataFolder + '/' + appName, { force: true, recursive: true })
+            fs.mkdirSync(mockDataFolder + '/' + appName, { recursive: true })
+            for (const [key, value] of Object.entries(scriptList)) {
+                fs.writeFileSync(mockDataFolder + '/' + appName + '/' + key, value.act)
             }
         }
 
-        function mockIssuerKeyResponse(){
+        function mockIssuerKeyResponse() {
             axios.get.mockResolvedValue({
-                data:{
+                data: {
                     // this data containing 'mockAESKey', which is encrypted using CLI Private key
                     data: 'eyJpc3N1ZXIiOnsia2V5IjoidEVmZlRwbGpsRW9seVRVcnpEQUs4b05mZEg0T2p2VE9MOHZ4c0p3MkRISC9LY09WV2t0SG5kWmxxUXJURUdoWGtoWUFtQjBobmw1RU5xdFh2TXNjcHFCMnd6eXZJUy9OdUpxeFdBQmE4Y21QbXYyUHBqRFV3eUd5LzJzbFBybG00aGJjbjJWamFPRVJiMHFpclZFNGhZVDluNlFwSnR1ZUp1NXcrVG9JN0V1eC9wdkZHeWVaOU1ham5hSTVtcVliZkNsT0pEZnV3MkpuU1RGNk1kbWg0MURualNuYk5nWkJhSjF2VWw2U1lmVDREeVJoVTkrUER6UTRMZ0xPZXlVYUlTVHQwWnpQWDIyeGI5Nlo3NlE0ZFNGUzhwRzRJRk0vMFpHbnI5eXB0NmNhRFlseW5VUWVFOHVJMklJMnpZcjFNdE12RnkzaDEyUU5Za0tLS3NaaVNRV00vQXRPTm5sa00rcjFKSFpMZW42b3lKYlV1cEgvRmZxQmViQWdRUThLcUVENnN2UlljSXBKOWh3UlJDb2RvNmlHaVVTMUVZZnh2N3pFMDhwL281Q2JmdFRKTXpISjh3T1JZbkdaTDh5cWttT2V0Y3dJL0p6U2dGVDNST2E5V3ZMRkFHWkIrcWdEOTBNeGhwT3kyVzRlbS9lUVBpeHNadDR2UGl2T1l0QVZYZTBLL0lJR0tlZ3VRZ0NzeUZENkRmajQxYkNXbm9oUEgzNWl1Y2h1cGxKVUhsSmszMTRqLytkbTBieEZ6MWFzdDBKanVaRFp3bkVSeVJVbTNFSWZNU2kxVGRRaXRWRjI2bFJ3a1NwK0VSRHh0ZllHWCs3V1U3OUhyUFF3MHBuU3NzU29FbVFzQ2VXU0dFc1AwbEtpTlBjLzBCeDNzbjdNMnJpMzJmZUZLRHM9In0sImFjcXVpcmVyIjp7ImRhdGEiOiJtT1NabzB0N0ZZb2M2a0tuZlNURk04S3VHcUtzLzU2MGtERGdldDMrRW9UTUZZaW5sbThwRFRqc2xVK2VoWnRidk1TejJ1bmNPck5rQ1phQy9LM0JrbmM5T3BZTDZ2SXVGNmY2alZrbjJDS25xbnRnUXBqUkY4T2dYK3NEKzhLbnBMVll2QmhMdkJydjI2RTBHTnk5YTNwbCt2MHNna3EycXJoeUJCeVc3VnpRL3h1Y2FCdDRrUjE0N2FpaTlSaThvUXEzRU1kZ3BPN2crYmxLMFk3eGt5ekQ2YU1JSDFSR1lSMTgwaU94akVqYVpiVE1uQkUwS1RJUnBEMnkwL25MTU5VTHZTUlBraVMyTUh5cmI1QWJCbWNQR1hzVUVvZFc0OHM0QVdmS2hZRldLbzV5anQwZlhCVCtUbEdaWmYrcDJINnRzSVNUYjZSKzlHWFk5c1EzNE5IdmFCRTZtNzlURUVQWTNkeUl3Vm5WQTIzR0FqL211bzI5M2dMTlp0R090ZWcrR1VlWXIzWXViQUZnL2VrN0NyTThGdGRQZC92R3NjdlJ4bkNEYnhmU3U3YXUzUFd1RFB5djk0ZjhXaXVMcWlKUGFQRmFZaUE4SW1lZklnY2ROKzMrTWZDRkdyR1dsWHlZM1pxVVI5NEhteGVjd1Q5Q256N2g2NHc0L3c0b0c5bU9CWmVsRk14dzlxUVlOWTZ5ZzRrakRBa2xUK29GMUszd213VUpPVTlhc3NCVEQ1SXZONlJRSEV0SU02ZmhKY1R0Zm5tK3l0Y201b3hjY3ZrN2tlVDExR2xjVi9Oa3EwNDRKb0NFL0drbzV2TWovQ29ERklJd0VjSENnQ1RvVGE0SForcWUzTWc2Wm1CazhvL2pwV3VFbTF5anJMcUdMSmFOTEJXNjlTUT0iLCJzaWduYXR1cmUiOiJzZDJXYnNDVy9oeXlMMnIzcmVhSFcyZndzVy85THVKNkp6MW0zUUNkWVA1NkZUU1ovSHM4c0Z6V2hmdVhtS1JJdEd5cnA4RGxMNTJkVkw1eTNnYmRIc0VoMDRITDhiWXIwS2lFU2RlbnVPMnoyZkdBWTlsclVjRC92alpZNlAwOHBVQTBEbXIreTk2UjQzdGVhTnZaMkplMklMWW1GZzdNOGF2cDZJZWk5WllzUmxyMFg2TmlzcUlXTWxDeHZOLzdadGNzcWQ3QndtWEhZRmlQWVkxaC9KTUtJdEREZnBneGgzU3RITlZQb01JU09FcTVkbGVRMjJNaTlPL2trZ2pkcmlBRDZsZ1hCZ214Q0FDaE5VQzVEZW5vUVIwQUlTREMvTW9udVA5ZldoMktwMkV2SlRGakR0eUJFdFZZekZZME82NEcxOVNhSFRnRmdxTFd2eXJDL2gzK1BqREIyTTdxOFBXOWVDNjlwM0ZCdUt1R0U3aTkrTmpKeWh4eFJoWWlxbW5MWjFxN3lFUzNudkIrNjJHRDZrUlR6YU5zdjFCUTRFSVpBR1NSaUF4d3o0aWo0ZldtdWJhaEd2WGlseGUzYzE1S09aa0RzcWRpQklIL1RWYzc1TUVBUVYrVVNlbXNEUnF2eUZ3ZFBobW9oM25KeVdmbFdsMU05eVNhdU1MaUFPV2c1bXhhSkdqQ0NtMGV5SHZaWUFFOFNWUjViWFJOZjIrNkpBQUJ2Mk5iSEJIVEJxM3V1cWpEVFZZdFZTRmQ3QmsvUS9NdVRiQnNjYUNqVUtybmtMcUpDTEw4akV0NnJlQnJLdlNPZll1c0txWHBWRmZMV1dHd1ZWSEViTzZVQ0xYTmIzY0c1MXRuNmpOS3NmN29wQXhBTFhUNFRXVjk0RjB6KzRpMjNYOD0ifX0=',
                     // sign using BizA Server PrivateKey, to be verified by BizA CLI using Server PublicKey
                     signature: 'Y9Vs9mOV5fqjcX1ZQh8N0vubV6J5+rx1Lvcrb8T9nBONMJIS74lqRdnO4a84rDCXdda0QOJs5750OdE69/ognriSqdvJ5qjb4sjYAuzXmbxGOuU4ptWMAl9CVfi7JdISdXaDMn5o2E9SuuQLwBMUvX6a9p93/L+TK/pSLuv3cwZqaCtiyhsZV1Q1mSz5xpEfGZR//0Vdaj7pZYZ7CDjQAiq6tMc1Mm7azDmxEwDmNqTMvPKJusGTTHvIFH9MbGTsHgq7DzhsGBVW0fyQGx4ycqB5u9D9j9YffjdlFZ3xncxgq/NVHJzHPbosrG2blW3yWr0Sg4P34AAOhNMtAbbrZfa5HYQrGLB5MbOY3s8EiXItufKzz5xTKDVdLER7kWk6th+a7c548TfqWkrkepPEvgBgmGoMw/PvXFsMVMKqRoF31lN7tumKnVyVzz/EwrsVVc7QrsJxfODHgU0bNNxi6gEb+prqKy0ZWmWwtf7coLWq/ybC2yFMrqUjymoJBGvNoLBURihD2m645+qudNlcJlLoyTs8B3KRCwFKtBH2X9eW5ZzDpIQAXDge6Bz/668Xg6DB8TzvpNr0sPOkuXXSKhQnb0SkR7QTHzpFDcg88Ur9Nnnh9aSZZRj1nRo5LawNto+9684SzpIfbjzAnbTHLQ6oY28SIdihKVjc0wj9BZk='
                 }
             })
-            axios.post.mockResolvedValue({data: {success: true}, status: 200})
+            axios.post.mockResolvedValue({ data: { success: true }, status: 200 })
         }
 
-        beforeAll(()=>{
-            fs.rmSync(mockDataFolder, {recursive: true, force: true})
+        beforeAll(() => {
+            fs.rmSync(mockDataFolder, { recursive: true, force: true })
         })
 
-        afterAll(()=>{
-            fs.rmSync(mockDataFolder, {recursive: true, force: true})
+        afterAll(() => {
+            fs.rmSync(mockDataFolder, { recursive: true, force: true })
         })
 
         const stressTestCount = 10;
-        it.each(Array(stressTestCount).fill().map((v,i)=>i+1))(`shall compress and encrypt app scripts (stress test %p of ${stressTestCount})`, async (testNo)=>{
+        it.each(Array(stressTestCount).fill().map((v, i) => i + 1))(`shall compress and encrypt app scripts (stress test %p of ${stressTestCount})`, async (testNo) => {
             const mockScripts = {
-                'a.js' : {
+                'a.js': {
                     act: 'get = function () {return {modelA: {}}}',
                     exp: '{"modelA":{}}'
                 },
-                'b.js' : {
+                'b.js': {
                     act: 'const get = function () {\nreturn {"modelB": {id: null}}};',
                     exp: '{"modelB":{"id":null}}'
                 },
-                'c.js' : {
+                'c.js': {
                     act: 'let get = function () {return {modelC: {}, function: {func: () => { \nreturn ["lib"];\n}\n}}};',
                     exp: '{"modelC":{},"function":{"func":["window.Function","","return[\\\"lib\\\"]"]}}'
                 },
-                'd.js' : {
+                'd.js': {
                     act: 'var get = function () {return {model: {}, tableName: "", fields: [], function: {}}};\nmodule.exports = get;',
                     exp: '{"model":{},"tableName":"","fields":[],"function":{}}'
                 },
-                'menu.json' : {
+                'menu.json': {
                     // act: JSON.stringify([
-                        act: `[
+                    act: `[
                         {"menuName": "", "caption": "Home", "link": ["./main"],"subMenu": []},
                         {"menuName": "", "caption": "Personalia","link": [], "subMenu": [
                             {"menuName": "","caption": "Data Master","link": [], "subMenu": []},
@@ -191,7 +203,7 @@ describe('Biz-A Apps CLI', ()=>{
                 tar.t({
                     strict: true
                     , sync: true
-                    , onReadEntry: entry=>{
+                    , onReadEntry: entry => {
                         const chunks = [];
                         entry.on('data', chunk => chunks.push(chunk));
                         entry.on('end', () => {
@@ -205,10 +217,10 @@ describe('Biz-A Apps CLI', ()=>{
                         })
                     }
                 })
-                .on('finish', ()=>stream.end())
+                    .on('finish', () => stream.end())
             )
             await finished(stream) // wait for untar process to finish
-            for (const [fileName, script] of Object.entries(decryptScripts)){
+            for (const [fileName, script] of Object.entries(decryptScripts)) {
                 expect(script).toBe(mockScripts[fileName].exp)
             }
 
@@ -228,7 +240,7 @@ describe('Biz-A Apps CLI', ()=>{
             expect(apiParams[3]).toHaveProperty('menu') // shall save "menu.json" as App.metadata
             expect(apiParams[3].menu).not.toBeNull() // IV (initialize vector) using 16 random bytes
 
-            expect(postArgs[2]).toStrictEqual({"headers": {"Content-Type": "text/plain"}})
+            expect(postArgs[2]).toStrictEqual({ "headers": { "Content-Type": "text/plain" } })
 
             expect(logSpy.mock.calls[0][0]).toContain(`Finished packing ${Object.keys(mockScripts).length} files into "./upload/${appName.toLowerCase()}.tgz"`)
             expect(logSpy.mock.calls[1][0]).toContain(`Finished uploading "./upload/${appName.toLocaleLowerCase()}.tgz"`)
@@ -236,61 +248,61 @@ describe('Biz-A Apps CLI', ()=>{
             // fs.rmSync(mockDataFolder + '/' + appName, {recursive: true, force: true})
         })
 
-/*        it('Invalid script syntax', async ()=>{
-            mockValidTemplates({
-                'a.js' : {act: 'get = function () {return {modelA: {}}'} // missing close closure at the end
-            })
-            mockIssuerKeyResponse()
-            await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
-
-            expect(logSpy.mock.calls.length).toBe(1)
-            expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
-
-            expect(errorSpy.mock.calls.length).toBe(1)
-            expect(errorSpy.mock.calls[0][0]).toBe('a.js : SyntaxError: Unexpected token: eof, expected: punc «}»')
-        })
-
-        it('Failed to compile script with node sandbox and ES6 Import', async ()=>{
-            mockValidTemplates({
-                'a.js' : {act: 'get = function () {return {modelA: {}}}\n modul.expor = get;\n'} // wrong "module.export"
-            })
-            mockIssuerKeyResponse()
-            await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
-
-            expect(logSpy.mock.calls.length).toBe(5)
-            expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
-            expect(logSpy.mock.calls[1][0]).toBe('Minify : \nget=function(){return{modelA:{}}};modul.expor=get;')
-            expect(logSpy.mock.calls[2][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
-            expect(logSpy.mock.calls[3][0]).toBe('a.js : ReferenceError: modul is not defined')
-            expect(logSpy.mock.calls[4][0]).toBe('Running script with Import function')
-
-            expect(errorSpy.mock.calls.length).toBe(1)
-            expect(errorSpy.mock.calls[0][0]).toBe("a.js : ReferenceError: modul is not defined") // ES6 import error
-        })
-
-        it('Shall not allow empty script', async ()=>{
-            mockValidTemplates({
-                'a.js' : {act: 'get = function () {}'} // empty script
-            })
-            mockIssuerKeyResponse()
-            await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
-
-            expect(logSpy.mock.calls.length).toBe(4)
-            expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
-            expect(logSpy.mock.calls[1][0]).toBe('Minify : \nget=function(){};')
-            expect(logSpy.mock.calls[2][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
-            expect(logSpy.mock.calls[3][0]).toBe('Running script with Import function')
-
-            expect(errorSpy.mock.calls.length).toBe(1)
-            expect(errorSpy.mock.calls[0][0]).toBe('a.js : Failed to compile template script.\nPlease make sure the script is correct and not returning empty result') // ES6 import error
-        }) */
+        /*        it('Invalid script syntax', async ()=>{
+                    mockValidTemplates({
+                        'a.js' : {act: 'get = function () {return {modelA: {}}'} // missing close closure at the end
+                    })
+                    mockIssuerKeyResponse()
+                    await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
+        
+                    expect(logSpy.mock.calls.length).toBe(1)
+                    expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
+        
+                    expect(errorSpy.mock.calls.length).toBe(1)
+                    expect(errorSpy.mock.calls[0][0]).toBe('a.js : SyntaxError: Unexpected token: eof, expected: punc «}»')
+                })
+        
+                it('Failed to compile script with node sandbox and ES6 Import', async ()=>{
+                    mockValidTemplates({
+                        'a.js' : {act: 'get = function () {return {modelA: {}}}\n modul.expor = get;\n'} // wrong "module.export"
+                    })
+                    mockIssuerKeyResponse()
+                    await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
+        
+                    expect(logSpy.mock.calls.length).toBe(5)
+                    expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
+                    expect(logSpy.mock.calls[1][0]).toBe('Minify : \nget=function(){return{modelA:{}}};modul.expor=get;')
+                    expect(logSpy.mock.calls[2][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
+                    expect(logSpy.mock.calls[3][0]).toBe('a.js : ReferenceError: modul is not defined')
+                    expect(logSpy.mock.calls[4][0]).toBe('Running script with Import function')
+        
+                    expect(errorSpy.mock.calls.length).toBe(1)
+                    expect(errorSpy.mock.calls[0][0]).toBe("a.js : ReferenceError: modul is not defined") // ES6 import error
+                })
+        
+                it('Shall not allow empty script', async ()=>{
+                    mockValidTemplates({
+                        'a.js' : {act: 'get = function () {}'} // empty script
+                    })
+                    mockIssuerKeyResponse()
+                    await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', appFolderName, "-v")
+        
+                    expect(logSpy.mock.calls.length).toBe(4)
+                    expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
+                    expect(logSpy.mock.calls[1][0]).toBe('Minify : \nget=function(){};')
+                    expect(logSpy.mock.calls[2][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
+                    expect(logSpy.mock.calls[3][0]).toBe('Running script with Import function')
+        
+                    expect(errorSpy.mock.calls.length).toBe(1)
+                    expect(errorSpy.mock.calls[0][0]).toBe('a.js : Failed to compile template script.\nPlease make sure the script is correct and not returning empty result') // ES6 import error
+                }) */
 
         const errorHandlingTestCases = [
             // [test case name, scripts, expectedFn()]
             [
                 'Invalid script syntax',
-                {act: 'get = function () {return {modelA: {}}'}, // missing close closure at the end',
-                ()=>{
+                { act: 'get = function () {return {modelA: {}}' }, // missing close closure at the end',
+                () => {
                     expect(logSpy.mock.calls.length).toBe(5)
                     expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
                     expect(logSpy.mock.calls[1][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
@@ -304,8 +316,8 @@ describe('Biz-A Apps CLI', ()=>{
             ],
             [
                 'Failed to compile script with node sandbox and ES6 Import',
-                {act: 'get = function () {return {modelA: {}}}\n modul.expor = get;\n'}, // wrong "module.export"
-                ()=>{
+                { act: 'get = function () {return {modelA: {}}}\n modul.expor = get;\n' }, // wrong "module.export"
+                () => {
                     expect(logSpy.mock.calls.length).toBe(6)
                     expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
                     expect(logSpy.mock.calls[1][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
@@ -320,8 +332,8 @@ describe('Biz-A Apps CLI', ()=>{
             ],
             [
                 'Shall not allow empty script',
-                {act: 'get = function () {}'}, // empty script
-                ()=>{
+                { act: 'get = function () {}' }, // empty script
+                () => {
                     expect(logSpy.mock.calls.length).toBe(5)
                     expect(logSpy.mock.calls[0][0]).toBe('===================\nA.JS\n===================')
                     expect(logSpy.mock.calls[1][0]).toBe('Running script with VM') // node sandbox (VN) error as console.log
@@ -334,15 +346,15 @@ describe('Biz-A Apps CLI', ()=>{
                 }
             ],
         ]
-        it.each(errorHandlingTestCases)('%p', async (name, scripts, expectedFn)=>{
+        it.each(errorHandlingTestCases)('%p', async (name, scripts, expectedFn) => {
             const appName = 'errorHandlingTest' + errorHandlingTestCases.findIndex(testCase => testCase[0] === name)
-            mockValidTemplates(appName, {'a.js' : scripts})
+            mockValidTemplates(appName, { 'a.js': scripts })
             mockIssuerKeyResponse()
             await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', mockDataFolder + '/' + appName, "-v")
             expectedFn()
         })
 
-        it('Shall not allow app without script template', async ()=>{
+        it('Shall not allow app without script template', async () => {
             mockValidTemplates(expect.getState().currentTestName, {})
             mockIssuerKeyResponse()
             await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', mockDataFolder + '/' + expect.getState().currentTestName, "-v")
@@ -353,89 +365,106 @@ describe('Biz-A Apps CLI', ()=>{
             expect(errorSpy.mock.calls[0][0]).toBe('Nothing to upload. Please recheck your app folder.')
         })
 
-        it('Shall recommend command', async ()=>{
-            const exitSpy = jest.spyOn(process, 'exit').mockImplementation(()=>{})
+        it('Shall recommend command', async () => {
+            const exitSpy = jest.spyOn(process, 'exit').mockImplementation(() => { })
             await runCommand('Add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', mockDataFolder + '/NotExistFolder', "-v")
             expect(logSpy.mock.calls.length).toBe(0)
             expect(exitSpy).toHaveBeenCalledWith(1) // exit code = 1
-            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('Did you mean add?')
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length - 1][0]).toBe('Did you mean add?')
         })
 
-        it('Shall have minimun one command', async ()=>{
+        it('Shall have minimun one command', async () => {
             const exitSpy = jest.spyOn(process, 'exit').mockImplementation()
             await runCommand('-s', 'https://a.b.c', '-i', 2)
             expect(logSpy.mock.calls.length).toBe(0)
             expect(exitSpy).toHaveBeenCalledWith(1)
-            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('You need at least one command before moving on')
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length - 1][0]).toBe('You need at least one command before moving on')
         })
 
-        it('Shall use strict command and options', async ()=>{
+        it('Shall use strict command and options', async () => {
             const exitSpy = jest.spyOn(process, 'exit').mockImplementation()
             await runCommand('Unknown Command', '-s', 'https://a.b.c', '-i', 2)
             expect(logSpy.mock.calls.length).toBe(0)
             expect(exitSpy).toHaveBeenCalledWith(1)
-            expect(errorSpy.mock.calls[errorSpy.mock.calls.length-1][0]).toBe('Unknown argument: Unknown Command')
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length - 1][0]).toBe('Unknown argument: Unknown Command')
+        })
+
+        it('Shall not continue add biz a template if code is not zero', async () => {  //SCY BZ 4331
+            returnedCode = 1;
+
+            const mockScripts = {
+                'a.js': {
+                    act: 'get = function () {return {modelA: {}}}',
+                    exp: '{"modelA":{}}'
+                }
+            }
+            const appName = 'compressStressTest1';
+
+            mockValidTemplates(appName, mockScripts);
+            await runCommand('add', '-s', 'https://a.b.c', '-p', '1205', '-i', '2', '-d', mockDataFolder + '/' + appName)
+
+            expect(errorSpy.mock.calls[errorSpy.mock.calls.length - 1][0]).toBe('Biz-A Add aborted')
         })
     })
 
-    describe('Remove App', ()=>{
-        it('shall remove specific apps', async ()=>{
-            axios.post.mockResolvedValue({data: {success: true}, status: 200})
+    describe('Remove App', () => {
+        it('shall remove specific apps', async () => {
+            axios.post.mockResolvedValue({ data: { success: true }, status: 200 })
 
             await runCommand('remove', '-s', 'https://finaapi.imamatek.com', '-p', '1205', '-i', '1', '-n', 'myApp')
 
             expect(axios.post.mock.calls).toHaveLength(1)
             expect(axios.post).toHaveBeenCalledWith(
                 'https://finaapi.imamatek.com:1205/fina/rest/TOrmMethod/%22deleteApp%22',
-                {_parameters: [1, 'myapp']},
-                {headers: {'Content-Type': 'text/plain'}}
+                { _parameters: [1, 'myapp'] },
+                { headers: { 'Content-Type': 'text/plain' } }
             )
             expect(logSpy).toHaveBeenCalledWith('myapp removed')
         })
 
-        it('shall remove multiple apps', async ()=>{
-            axios.post.mockResolvedValue({data: {success: true, _f: 'notExistApp'}, status: 200})
+        it('shall remove multiple apps', async () => {
+            axios.post.mockResolvedValue({ data: { success: true, _f: 'notExistApp' }, status: 200 })
 
             await runCommand('remove', '-i', '2', '-s', 'https://www.AA.io', '-p', '2708', '-n', 'firstApp, notExistApp, secondApp')
 
             expect(axios.post.mock.calls).toHaveLength(1)
             expect(axios.post).toHaveBeenCalledWith(
                 'https://www.AA.io:2708/fina/rest/TOrmMethod/%22deleteApp%22',
-                {_parameters: [2, 'firstapp,notexistapp,secondapp']},
-                {headers: {'Content-Type': 'text/plain'}}
+                { _parameters: [2, 'firstapp,notexistapp,secondapp'] },
+                { headers: { 'Content-Type': 'text/plain' } }
             )
             expect(logSpy).toHaveBeenCalledWith('firstapp removed')
             expect(logSpy).toHaveBeenCalledWith('secondapp removed')
             expect(logSpy).toHaveBeenCalledWith('notexistapp not found')
         })
 
-        it('shall remove all apps', async ()=>{
-            axios.post.mockResolvedValue({data: {success: true}, _f: '', status: 200})
+        it('shall remove all apps', async () => {
+            axios.post.mockResolvedValue({ data: { success: true }, _f: '', status: 200 })
 
             await runCommand('remove', '-s', 'https://finaapi.imamatek.com', '-p', '1205', '-i', '2')
 
             expect(axios.post.mock.calls).toHaveLength(1)
             expect(axios.post).toHaveBeenCalledWith(
                 'https://finaapi.imamatek.com:1205/fina/rest/TOrmMethod/%22deleteApp%22',
-                {_parameters: [2, '']},
-                {headers: {'Content-Type': 'text/plain'}}
+                { _parameters: [2, ''] },
+                { headers: { 'Content-Type': 'text/plain' } }
             )
             expect(logSpy).toHaveBeenCalledWith('All apps removed')
         })
     })
 
-    it('Shall have options', async ()=>{
+    it('Shall have options', async () => {
         jest.spyOn(process, 'exit').mockImplementation()
         const appModule = await import('../bin/app.js')
         expect(appModule.options).toStrictEqual({
-            "s" : {
+            "s": {
                 alias: "server",
                 describe: `API or Server URL (ex: ${env.BIZA_SERVER_LINK} or http://192.168.1.1 or https://finaapi.imamatek.com)`,
                 type: "string",
                 demandOption: true,
                 default: 'http://localhost'
             },
-            "i" : {
+            "i": {
                 alias: "dbIndex",
                 default: 2,
                 describe: "database index",
@@ -450,7 +479,7 @@ describe('Biz-A Apps CLI', ()=>{
             },
             "p": {
                 alias: "apiPort",
-                default : 212,
+                default: 212,
                 describe: "FINA API Port",
                 type: "number",
                 demandOption: false,
@@ -482,5 +511,4 @@ describe('Biz-A Apps CLI', ()=>{
             }
         })
     })
-
 })