biz-a-cli 2.3.67 → 2.3.69

package/bin/app.js

@@ -136,8 +136,20 @@ Object
                     jestCommand.push('--passWithNoTests')
                 }
             }
-            const child = spawn('npx', jestCommand, { shell: true });
-            child.stderr.on('data', data => output += data?.toString())
+            process.chdir(workDir);
+
+            const child = process.platform === 'win32'
+                ? spawn('cmd.exe', ['/d', '/s', '/c', 'npx', ...jestCommand])
+                : spawn('npx', jestCommand)
+            const collectOutput = data => {
+                output += data?.toString() || ''
+            }
+            child.stderr?.on('data', collectOutput)
+            child.stdout?.on('data', () => { }) // SCY BZ 4363, ref : https://nodejs.org/download/release/v22.19.0/docs/api/child_process.html
+
+            child.on('error', e => {
+                output += e?.message ? `\n${e.message}` : ''
+            });
             child.on('close', async code => {
                 console.log('====================');
                 console.log(output);
@@ -149,7 +161,6 @@ Object
                 if ((code == 0) ||
                     ((code == 1) && noTestsFound) ||
                     ((code == 1) && missingJest && passWithNoTests)) {
-                    process.chdir(workDir)
                     await addApp()
                 } else {
                     console.error('Biz-A Add aborted');
@@ -252,4 +263,6 @@ Object
     .strict()
     .parse();
 
-export { options, addCommandOptions, removeCommandOptions }
+export { options, addCommandOptions, removeCommandOptions }
+
+

package/envs/env.dev.js

@@ -5,6 +5,6 @@ export const envDev = {
   CDM_MONGO_LINK: 'mongodb+srv://imm_cdm:' +
     encodeURIComponent('imm@2019') + '@imm-cdm-dev.rf6wr.mongodb.net/?retryWrites=true&w=majority',
   COMPANY_REGISTER: 'companyregister-dev',
-  BIZA_SERVER_LINK: 'https://biz-a-dev-41e7c93a25e5.herokuapp.com',
+  BIZA_SERVER_LINK: 'https://devserver.biz-a.id',
   BIZA_HUB_SERVER_LINK: 'https://hub.biz-a.id'
 };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "biz-a-cli",
   "nameDev": "biz-a-cli-dev",
-  "version": "2.3.67",
+  "version": "2.3.69",
   "versionDev": "0.0.34",
   "description": "",
   "main": "bin/index.js",

package/tests/app.test.js

@@ -5,6 +5,7 @@ import { finished } from 'node:stream/promises'
 import { createDecipheriv } from 'node:crypto'
 import * as tar from "tar"
 import { env } from "../envs/env.js"
+import path from "node:path"
 
 describe('Biz-A Apps CLI', () => {
     let originalOptions, originalCWD, axios, child_process;
@@ -404,6 +405,8 @@ describe('Biz-A Apps CLI', () => {
             }
             const appName = 'compressStressTest1';
             const fullTestPath = mockDataFolder + appName;
+            const absoluteFullTestPath = path.resolve(fullTestPath);
+
             let jestArgs = ['--no-install', 'jest', '--json'];
             let jestArgsWithPassNoTest = jestArgs.slice();
             jestArgsWithPassNoTest.push('--passWithNoTests');
@@ -414,7 +417,12 @@ describe('Biz-A Apps CLI', () => {
             }
 
             function continueExpect(args) {
-                expect(child_process.spawn).toBeCalledWith("npx", args, { "shell": true });
+                expect(process.cwd()).toBe(absoluteFullTestPath);
+                if (process.platform === "win32") {
+                    expect(child_process.spawn).toBeCalledWith("cmd.exe", ["/d", "/s", "/c", "npx", ...args]);
+                } else {
+                    expect(child_process.spawn).toBeCalledWith("npx", args);
+                }
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL - 2][0]).toBe('====================');
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL - 1][0]).toBe(jestMessage);
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL][0]).toBe('====================');
@@ -422,7 +430,12 @@ describe('Biz-A Apps CLI', () => {
             }
 
             function abortExpect(args) {
-                expect(child_process.spawn).toBeCalledWith("npx", args, { "shell": true });
+                expect(process.cwd()).toBe(absoluteFullTestPath);
+                if (process.platform === "win32") {
+                    expect(child_process.spawn).toBeCalledWith("cmd.exe", ["/d", "/s", "/c", "npx", ...args]);
+                } else {
+                    expect(child_process.spawn).toBeCalledWith("npx", args);
+                }
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL][0]).toBe('====================');
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL + 1][0]).toBe(jestMessage);
                 expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL + 2][0]).toBe('====================');
@@ -531,6 +544,51 @@ describe('Biz-A Apps CLI', () => {
 
                 continueExpect(jestArgsWithPassNoTest);
             })
+
+            it('Shall continue add biz a template, stdout exists and must be consumed for many unit test outputs', async () => {
+                returnedCode = 0;
+                const stdoutChunks = Array(300).fill('Passed');
+                let stdoutConsumed = false;
+
+                child_process.spawn.mockImplementationOnce(() => ({
+                    on: jest.fn((event, callback) => {
+                        if ((event === 'close') && stdoutConsumed) callback(returnedCode)
+                    }),
+                    stderr: {
+                        on: jest.fn()
+                    },
+                    stdout: {
+                        on: jest.fn((event, callback) => {
+                            if (event === 'data') {
+                                stdoutConsumed = true;
+                                for (const chunk of stdoutChunks) {
+                                    callback(chunk)
+                                }
+                            }
+                        })
+                    }
+                }))
+
+                mockValidTemplates(appName, mockScripts);
+                fs.mkdirSync(fullTestPath + '/test', { recursive: true })
+                fs.writeFileSync(fullTestPath + '/test/a.test.js', `it('dummy test', () => {
+    expect(true).toBe(true);
+})`)
+
+                await runJest();
+
+                expect(stdoutConsumed).toBe(true);
+                expect(process.cwd()).toBe(absoluteFullTestPath);
+                if (process.platform === "win32") {
+                    expect(child_process.spawn).toBeCalledWith("cmd.exe", ["/d", "/s", "/c", "npx", ...jestArgs]);
+                } else {
+                    expect(child_process.spawn).toBeCalledWith("npx", jestArgs);
+                }
+                expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL - 2][0]).toBe('====================');
+                expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL - 1][0]).toBe('');
+                expect(logSpy.mock.calls[logSpy.mock.calls.length - JEST_MESSAGE_TOTAL][0]).toBe('====================');
+                expect(errorSpy.mock.calls.length).toBe(0);
+            })
         })
     })
 
@@ -638,4 +696,8 @@ describe('Biz-A Apps CLI', () => {
             }
         })
     })
-})
+})
+
+
+
+